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

[minor] Support install with manual certificates #272

Merged
merged 18 commits into from
May 16, 2023
Merged

Conversation

alequint
Copy link
Contributor

@alequint alequint commented May 3, 2023

Besides including suite_cert as part of install-with-fvt pipeline, important to note also the new workspace created to hold the certificates (based on a secret created in ansible-fvt) and the correspondent procedure that copy it from this workspace to the final workspace/config/certs, used by suite_certs role.

Reminder: before merging, mas_devops zip file must be removed (it is here while further test is being performed). So this PR can only be merged after ibm-mas/ansible-devops#501

@alequint alequint changed the title [minor] PIpeline changes to cover manual certificate + custom version… [minor] PIpeline changes to cover manual certificates May 13, 2023
@alequint alequint changed the base branch from master to MASISMIG32554 May 13, 2023 15:20
@alequint alequint changed the base branch from MASISMIG32554 to master May 13, 2023 15:20
@alequint alequint marked this pull request as ready for review May 13, 2023 15:36
@alequint alequint marked this pull request as draft May 13, 2023 18:48
@alequint alequint marked this pull request as ready for review May 13, 2023 18:48
@durera durera changed the title [minor] PIpeline changes to cover manual certificates [minor] Support install with manual certificates May 16, 2023
@durera durera merged commit 482bd4e into master May 16, 2023
@durera durera deleted the manualcertmgmt branch May 16, 2023 14:09
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