-
Notifications
You must be signed in to change notification settings - Fork 15
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 wanda demo to pr env #1707
Add wanda demo to pr env #1707
Conversation
63b6034
to
45e64be
Compare
83c9c91
to
bb9786e
Compare
What should I be looking at here @nelsonkopliku ? |
That the checks execution at https://1707.prenv.trento.suse.com/ works. Previously it always timed out because there are no agents returning gathered facts. |
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.
Code apart, it 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.
Nice @nelsonkopliku
PD: Not your fault, and not need to add any change (as it was already this way), but not having any space before the ending }}
make my eyes bleed XD
bb9786e
to
a40ba4c
Compare
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.
❤️
Description
With this little change we have also the possibility to interact with checks execution.
https://1707.prenv.trento.suse.com/
Besides this also a specific rabbit user for the PR has been specified, otherwise different PRs interfere with each other with the default user being deleted.