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

Rename modules dbparser to correlation #4294

Merged

Conversation

bazsi
Copy link
Collaborator

@bazsi bazsi commented Jan 28, 2023

This PR renames the modules/dbparser directory (and module) to modules/correlation to indicate that all of our correlation related plugins are stored there and not just dbparser.

This is coming from #4225 but can be merged separately. No NEWS file is required in my opinion, the module is auto-loaded anyway, but we can have a note in the NEWS if someone needs it to be necessary.

@github-actions
Copy link
Contributor

No news file has been detected. Please write one, if applicable.

@kira-syslogng
Copy link
Contributor

Build FAILURE

@bazsi bazsi force-pushed the rename-modules-dbparser-to-correlation branch from 57dc28a to b9c262d Compare January 29, 2023 06:59
@kira-syslogng
Copy link
Contributor

Build FAILURE

Signed-off-by: Balazs Scheidler <bazsi77@gmail.com>
…orrelation" prefix

Signed-off-by: Balazs Scheidler <bazsi77@gmail.com>
Signed-off-by: Balazs Scheidler <bazsi77@gmail.com>
Signed-off-by: Balazs Scheidler <bazsi77@gmail.com>
Signed-off-by: Balazs Scheidler <bazsi77@gmail.com>
@alltilla alltilla force-pushed the rename-modules-dbparser-to-correlation branch from 9fb246b to 5d8cb4b Compare February 1, 2023 12:49
@alltilla
Copy link
Collaborator

alltilla commented Feb 1, 2023

Rebased to master, to get the python fix.

@kira-syslogng
Copy link
Contributor

Build FAILURE

Copy link
Collaborator

@alltilla alltilla left a comment

Choose a reason for hiding this comment

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

Kira build is failing on master, too, so it is not related to this PR.

alltilla added a commit that referenced this pull request Feb 1, 2023
Signed-off-by: Attila Szakacs <szakacs.attila96@gmail.com>
@alltilla
Copy link
Collaborator

alltilla commented Feb 1, 2023

I have created a PR for the news entry about the so rename: #4299 I think it can be interesting for a package maintainer.

@alltilla alltilla merged commit 5827eac into syslog-ng:master Feb 1, 2023
Genfood pushed a commit to Genfood/syslog-ng that referenced this pull request Jun 14, 2023
Signed-off-by: Attila Szakacs <szakacs.attila96@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants