-
-
Notifications
You must be signed in to change notification settings - Fork 45
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
Conman : Improve molecule tests #463
Conversation
@oxedions Did you build conman for el7 on purpose? Should it replace the package from the OS? I would rather prefer to keep the conman from upstream. |
@btravouillon No, it is a mistake. I did it too fast. Conman should not be provided on BB repositories on EL7. I will remove it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you revert all your changes but the verify.yml and update of the baseurl?
@oxedions, as discussed please do not forget to remove this package : Thanks in advance ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok for me
thank you very much @osmocl
Regards
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dear @osmocl
Thanks a lot for this PR.
Ok for me, merging 😊
Adding molecule tests for conman role.