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.

Sign up for GitHub

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

Jump to bottom

plugins/inventory/cobbler: Add exclude/include_mgmt_classes #7184

Merged
merged 1 commit into from
Sep 5, 2023
Merged

plugins/inventory/cobbler: Add exclude/include_mgmt_classes #7184

merged 1 commit into from
Sep 5, 2023

Conversation

opoplawski
Copy link
Contributor

SUMMARY

Add exclude/include_mgmt_classes options to be able to include or exclude hosts based on their management class.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/inventory/cobbler.py

@opoplawski opoplawski force-pushed the cobbler_mgmt_classes branch from e9e7092 to f8e9b09 Compare August 31, 2023 20:39
@ansibullbot ansibullbot added feature This issue/PR relates to a feature request inventory inventory plugin plugins plugin (any type) labels Aug 31, 2023
@felixfontein felixfontein added check-before-release PR will be looked at again shortly before release and merged if possible. backport-7 labels Sep 1, 2023
felixfontein
felixfontein reviewed Sep 3, 2023
View reviewed changes
Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@felixfontein felixfontein removed the check-before-release PR will be looked at again shortly before release and merged if possible. label Sep 5, 2023
@felixfontein felixfontein merged commit 0c03f34 into ansible-collections:main Sep 5, 2023
@patchback
Copy link

patchback bot commented Sep 5, 2023 •
edited
Loading

Backport to stable-7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-7/0c03f34f54a4e826483f524571ae7afc3952538f/pr-7184

Backported as #7208

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Sep 5, 2023
@opoplawski @patchback
plugins/inventory/cobbler: Add exclude/include_mgmt_classes (#7184)
cd5fa7d 
(cherry picked from commit 0c03f34)
@patchback patchback bot mentioned this pull request Sep 5, 2023
Merged
@felixfontein
Copy link
Collaborator

@opoplawski thanks for your contribution!

felixfontein pushed a commit that referenced this pull request Sep 5, 2023
@patchback @opoplawski
[PR #7184/0c03f34f backport][stable-7] plugins/inventory/cobbler: Add…
aa2a5d9 
… exclude/include_mgmt_classes (#7208)

plugins/inventory/cobbler: Add exclude/include_mgmt_classes (#7184)

(cherry picked from commit 0c03f34)

Co-authored-by: Orion Poplawski <orion@nwra.com>
etrombly pushed a commit to etrombly/community.general that referenced this pull request Oct 25, 2023
@opoplawski
plugins/inventory/cobbler: Add exclude/include_mgmt_classes (ansible-…
a3f878f 
…collections#7184)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@felixfontein felixfontein felixfontein approved these changes

Assignees
No one assigned
Labels
feature This issue/PR relates to a feature request inventory inventory plugin plugins plugin (any type)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@opoplawski @felixfontein @ansibullbot