Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ctl: fix managment of bot groups #2086

Merged
1 commit merged into from
Aug 31, 2021
Merged

ctl: fix managment of bot groups #2086

1 commit merged into from
Aug 31, 2021

Conversation

ghost
Copy link

@ghost ghost commented Aug 31, 2021

the global block of the runtime configuration was not excluded in one
block, raising an exception

fixes #2085

@ghost ghost added bug Indicates an unexpected problem or unintended behavior component: intelmqctl labels Aug 31, 2021
@ghost ghost added this to the 3.0.1 milestone Aug 31, 2021
@ghost ghost self-requested a review August 31, 2021 06:35
@ghost ghost self-assigned this Aug 31, 2021
bots = sorted(k_v[0] for k_v in filter(lambda x: x[1]["group"] == BOT_GROUP[group], self.runtime_configuration.items()))
for bot_id, bot in self.runtime_configuration.items():
if "group" not in bot:
print(bot_id)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whats that print for?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

... nothing :)

the global block of the runtime configuration was not excluded in one
block, raising an exception

fixes #2085
@ghost ghost force-pushed the wagner/fix-2085 branch from 3d73016 to d21176c Compare August 31, 2021 06:45
@ghost ghost merged commit d21176c into maintenance Aug 31, 2021
@ghost ghost deleted the wagner/fix-2085 branch August 31, 2021 06:45
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior component: intelmqctl
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants