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

Delete secrets from previous installation #531

Closed
wants to merge 1 commit into from

Conversation

holser
Copy link
Contributor

@holser holser commented Jul 16, 2024

The secrets may have old passwords that are not compatible with current installation. This patch removes secrets that are left from previous installation

Closes: OSPRH-8599

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/0650d2953f06402ea593bfd9572683da

data-plane-adoption-osp-17-to-extracted-crc FAILURE in 1h 26m 11s
data-plane-adoption-osp-17-to-extracted-crc-minimal-no-ceph FAILURE in 1h 35m 45s

@jistr
Copy link
Contributor

jistr commented Jul 17, 2024

Looks like the CI failure is relevant.

The secrets may have old passwords that are not compatible with current
installation. This patch removes secrets that are left from previous
installation

Closes: OSPRH-8599
@holser
Copy link
Contributor Author

holser commented Jul 22, 2024

@jistr It should be OK now

jistr
jistr previously approved these changes Jul 23, 2024
@jistr jistr dismissed their stale review July 24, 2024 12:52

Per discussion on Slack, removing finalizers prior to deleting a resource is incorrect

@holser holser closed this Aug 20, 2024
@holser holser deleted the fix_cleanup branch August 20, 2024 14:24
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