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

Discover bots based on the entry points #2413

Merged
merged 14 commits into from
Nov 14, 2023

Conversation

kamil-certat
Copy link
Contributor

@kamil-certat kamil-certat commented Sep 21, 2023

The bot discovery algorithm was rewritten to use entry point
names instead of the file structure. This allow really easy
developing of packages with custom bots.

Note that the 'group' feature of entry points wasn't used
because we require bots to register callable scripts that
are later used to start bot's processes.

The bot discovery algorithm was rewritten to use entry point
names instead of the file structure. This allow really easy
developing of packages with custom bots.

Not that the 'group' feature of entry points wasn't used
because we require bots to register callable scripts that
are later used to start bot's processes.
@kamil-certat kamil-certat marked this pull request as ready for review September 21, 2023 12:58
@sebix sebix added feature Indicates new feature requests or new features component: core labels Sep 26, 2023
@sebix sebix added this to the 3.3.0 milestone Sep 26, 2023
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
contrib/example-extension-package/setup.py Outdated Show resolved Hide resolved
docs/dev/guide.rst Outdated Show resolved Hide resolved
docs/dev/guide.rst Outdated Show resolved Hide resolved
docs/dev/guide.rst Outdated Show resolved Hide resolved
intelmq/lib/utils.py Show resolved Hide resolved
setup.py Show resolved Hide resolved
docs/user/installation.rst Outdated Show resolved Hide resolved
@kamil-certat
Copy link
Contributor Author

After testing in our environment - fixes to places, where bot modules are imported

@sebix
Copy link
Member

sebix commented Oct 16, 2023

For further tests on more platforms I pushed your branch (with #2416) to the "unstable" package repo: https://build.opensuse.org/package/show/home:sebix:intelmq:unstable/intelmq

debian/control Show resolved Hide resolved
@kamil-certat
Copy link
Contributor Author

Thanks for the unstable package - I'll try to test the integration with the DEB package in the next two weeks. If you had your own use cases for tests, it would be great if you also could do some tests. From my side - the change is now used and works great for in our systems.

@kamil-certat
Copy link
Contributor Author

@sebix I have just tested the integration with DEB package (IntelMQ from deb, additional bots from pip) - it works well! As it also works good in our instances for w few weeks already, I think we can merge it :)

@sebix sebix merged commit f366698 into certtools:develop Nov 14, 2023
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: core feature Indicates new feature requests or new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants