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

Use is not None instead of != null. #321

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

freyes
Copy link
Member

@freyes freyes commented Feb 15, 2024

The value passed in the when stanza uses None instead of null.

Fixes the error: 'null' is undefined

The value passed in the `when` stanza uses None instead of null.

Fixes the error: `'null' is undefined`
@freyes
Copy link
Member Author

freyes commented Feb 15, 2024

Fixes this error seen at the gate:


2024-02-15 15:19:20.880299 | TASK [handle-constraints-url : GET the constraints file]
2024-02-15 15:19:20.987221 | focal-medium | ERROR
2024-02-15 15:19:20.987644 | focal-medium | {
2024-02-15 15:19:20.987718 | focal-medium |   "msg": "The conditional check 'constraints_file is defined and pip_constraints_url is defined and pip_constraints_url != null and pip_constraints_url|length > 0' failed. The error was: error while evaluating conditional (constraints_file is defined and pip_constraints_url is defined and pip_constraints_url != null and pip_constraints_url|length > 0): 'null' is undefined\n\nThe error appears to be in '/var/lib/zuul/builds/e46ddf012092472a9aa826d844e4117d/trusted/project_0/github.com/openstack-charmers/zosci-config/roles/handle-constraints-url/tasks/main.yaml': line 8, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: GET the constraints file\n  ^ here\n"
2024-02-15 15:19:20.987780 | focal-medium | }

https://openstack-ci-reports.ubuntu.com/artifacts/e46/908775/3/check/jammy-zed/e46ddf0/job-output.txt

@freyes freyes requested a review from ajkavanagh February 15, 2024 18:38
@wolsen wolsen merged commit 2143b89 into openstack-charmers:master Feb 15, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants