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

validate: check for empty url (ceph_origin=custom) #190

Merged
merged 2 commits into from
Feb 8, 2023
Merged

Conversation

guits
Copy link
Collaborator

@guits guits commented Feb 8, 2023

This checks if custom_repo_url is set to an empty string and makes the playbook fail in that case.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=2097680

Signed-off-by: Guillaume Abrioux [email protected]

@guits guits added backport-quincy backport quincy backport-pacific backport pacific labels Feb 8, 2023
guits added 2 commits February 8, 2023 10:23
This checks if `custom_repo_url` is set to an empty string
and makes the playbook fail in that case.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=2097680

Signed-off-by: Guillaume Abrioux <[email protected]>
it makes troubleshooting easier in the CI.

Signed-off-by: Guillaume Abrioux <[email protected]>
@guits guits force-pushed the custom-repo-empty branch from e81727b to c0db0fd Compare February 8, 2023 09:24
@guits guits merged commit 119b5cf into devel Feb 8, 2023
@guits guits deleted the custom-repo-empty branch February 8, 2023 09:40
guits added a commit that referenced this pull request Feb 8, 2023
validate: check for empty url (ceph_origin=custom) (backport #190)
guits added a commit that referenced this pull request Feb 8, 2023
validate: check for empty url (ceph_origin=custom) (backport #190)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-pacific backport pacific backport-quincy backport quincy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant