Skip to content

Conversation

00ssh
Copy link

@00ssh 00ssh commented Nov 14, 2018

Added feature for repomd_gpg_url (required for subscription-manager versions >= 1.21.10-2)
With the latest release of RHEL 7.6, subscription manager got updated and the module doesn't support the new feature (it fails at puppet run).

I appreciate if you merge the pull request as soon as possible. Please let me know if there is anything else I could do.

Thanks!

@waveclaw
Copy link
Owner

You need to provide tests for the new feature that demonstrates it is functioning. Until this this is blocked on no tests.

@waveclaw waveclaw added bug enhancement NO TESTS Push Request has new code but no validating tests labels Nov 22, 2018
Copy link
Owner

@waveclaw waveclaw left a comment

Choose a reason for hiding this comment

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

Needs documentation if it is required by anything.

@waveclaw waveclaw merged commit 66131aa into waveclaw:master Nov 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug enhancement NO TESTS Push Request has new code but no validating tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants