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

Fix typo in hive role #570

Conversation

lewisdenny
Copy link
Collaborator

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:

  • Appropriate testing is done and actually running
  • Appropriate documentation exists and/or is up-to-date:
    • README in the role
    • Content of the docs/source is reflecting the changes

This PR fixes a warning reported by ansible-lint [1]

[1] `WARNING  Unable to resolve FQCN for module ansible.builtin.include_task`
@openshift-ci openshift-ci bot requested review from afazekas and dpinhas September 19, 2023 01:54
@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://review.rdoproject.org/zuul/buildset/13487cd870a14f1ba03c0d9d031fb1ea

✔️ openstack-k8s-operators-content-provider SUCCESS in 51m 55s
podified-multinode-edpm-deployment-crc FAILURE in 36m 38s
✔️ noop SUCCESS in 0s
podified-multinode-edpm-e2e-nobuild-tagged-crc FAILURE in 39m 55s
✔️ cifmw-molecule-hive SUCCESS in 4m 48s

Copy link
Contributor

@psathyan psathyan left a comment

Choose a reason for hiding this comment

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

/lgtm

@lewisdenny
Copy link
Collaborator Author

I think we have an issue here, I'm getting the same failure locally as well:

TASK [edpm_kustomize : Apply the already existing kustomization if present output_dir={{ cifmw_edpm_kustomize_artifacts_dir }}, chdir={{ cifmw_edpm_kustomize_cr_path | dirname }}, script=oc kustomize  -o {{ cifmw_edpm_kustomize_cr_path }}] ***
task path: /ldenny/ci-framework/ci_framework/roles/edpm_kustomize/tasks/kustomize.yml:37
The full traceback is:
NoneType: None
fatal: [localhost]: FAILED! => {
    "changed": true,
    "msg": "non-zero return code",
    "rc": 1,
    "stderr": "",
    "stderr_lines": [],
    "stdout": "~/ci-framework-data/artifacts/manifests/openstack/dataplane/cr ~/ci-framework-data/artifacts\nerror: replace operation does not apply: doc is missing path: /spec/deployStrategy/deploy: missing value\n",
    "stdout_lines": [
        "~/ci-framework-data/artifacts/manifests/openstack/dataplane/cr ~/ci-framework-data/artifacts",
        "error: replace operation does not apply: doc is missing path: /spec/deployStrategy/deploy: missing value"
    ]
}

Just leaving this hear until later.

@cjeanner
Copy link
Collaborator

/approve

It seems to need an install_yamls change:
openstack-k8s-operators/install_yamls#534
(as well as openstack-k8s-operators/install_yamls#538 for BM)

IIRC we also have an open PR for that one, probably #560.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 19, 2023

[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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@bshephar
Copy link
Contributor

bshephar commented Sep 19, 2023

/approve

It seems to need an install_yamls change: openstack-k8s-operators/install_yamls#534 (as well as openstack-k8s-operators/install_yamls#538 for BM)

IIRC we also have an open PR for that one, probably #560.

Yeah, the two install_yamls PR's are needed to fix that. It's because the deployStrategy was removed by:
openstack-k8s-operators/dataplane-operator@ff7bbd6#diff-1135163039ac7c4a64ba5b6a225ad7225316c9ed0e7d42f1079d3d630a25120aL127

The PR you referenced of mine for ci-framework is removing the NetworkConfig part of the API, so unrelated to this specific failure.

@pablintino
Copy link
Collaborator

recheck

@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://review.rdoproject.org/zuul/buildset/b40c1f92be374009a79095fe28e43731

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 22m 42s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 57m 27s
✔️ noop SUCCESS in 0s
podified-multinode-edpm-e2e-nobuild-tagged-crc FAILURE in 52m 54s
✔️ cifmw-molecule-hive SUCCESS in 5m 06s

@lewisdenny
Copy link
Collaborator Author

recheck

@openshift-merge-robot openshift-merge-robot merged commit 211c4b4 into openstack-k8s-operators:main Sep 20, 2023
1 check passed
@lewisdenny lewisdenny deleted the resolve_anisble_lint_warnings branch April 30, 2024 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants