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

Add rubygem-smart_proxy_dhcp_bluecat package #4197

Merged
merged 1 commit into from
Oct 18, 2019

Conversation

mdellweg
Copy link
Member

No description provided.

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

Looks like the scripts to add packages need some updates. It'd be awesome if you could submit a PR for that as well and regenerate this PR to test it but I'll also accept just fixing it manually in this PR.

@@ -414,6 +414,7 @@ foreman_nonscl_packages:
rubygem-kafo: {}
rubygem-kafo_parsers: {}
rubygem-kafo_wizards: {}
rubygem-smart_proxy_dhcp_bluecat: {}
Copy link
Member

Choose a reason for hiding this comment

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

I think we need to update our add script to the updated structure. Could you move it to the foreman_proxy_plugin_packages group?

Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe the scripts are ok. I think if you only specify the gem name and the template, they end up in the right category. But The usage message does not suggest that.

@ehelms ehelms merged commit 6c71b25 into theforeman:rpm/develop Oct 18, 2019
@mdellweg mdellweg deleted the add_dhcp_bluecat branch October 19, 2019 09:16
@adamruzicka adamruzicka mentioned this pull request Jul 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants