-
Notifications
You must be signed in to change notification settings - Fork 110
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
Fix typo in hive role #570
Fix typo in hive role #570
Conversation
This PR fixes a warning reported by ansible-lint [1] [1] `WARNING Unable to resolve FQCN for module ansible.builtin.include_task`
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/13487cd870a14f1ba03c0d9d031fb1ea ✔️ openstack-k8s-operators-content-provider SUCCESS in 51m 55s |
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.
/lgtm
I think we have an issue here, I'm getting the same failure locally as well:
Just leaving this hear until later. |
/approve It seems to need an install_yamls change: IIRC we also have an open PR for that one, probably #560. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cjeanner The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Yeah, the two install_yamls PR's are needed to fix that. It's because the The PR you referenced of mine for ci-framework is removing the |
recheck |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/b40c1f92be374009a79095fe28e43731 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 22m 42s |
recheck |
211c4b4
into
openstack-k8s-operators:main
This PR fixes a warning reported by ansible-lint [1]
[1]
WARNING Unable to resolve FQCN for module ansible.builtin.include_task
As a pull request owner and reviewers, I checked that: