Skip to content
This repository has been archived by the owner on Jul 23, 2019. It is now read-only.

Merge openshift/installer 4.1.0 rc.5 #83

Merged
merged 16 commits into from
May 20, 2019

Conversation

stbenjam
Copy link
Member

No description provided.

deads2k and others added 16 commits May 8, 2019 12:47
This change adds sudo when gathering rendered-assets within
/usr/local/bin/installer-gather.sh. It also changes the owner and
permissions on the copied files so that the core user has access to
read them. In addition, the method used to retrieve assets from the
control-plane nodes has been modified to make it less noisy on error.
Bug 1708307: bootstrap/files: installer-gather.sh to gather more rendered-assets
Fix the UPI's Readme deploy-packet url
Bug 1707096: gather: suffix container logs with container ids for uniqueness
[openstack] More robust deprovision strategy
…1750-to-release-4.1

Bug 1709869: tls: add openshift service to DNS names
…lete

PR for cluster-machine-approver [1] is taking over the approval of CSRs for client certificates for Machines that end up
as Nodes in Openshift clusters.

But during bootstrapping, cluster-machine-approver is not available and therefore, this service is required to approve CSRs until
we have successfully bootstrapped the control plane, after which cluster-machine-approver or users take over the role of approving any new CSRs.

Currently, all CSRs are automatically approved without any condition, this PR scopes it to only during bootstrapping phase, securing the endpoint for later use.

[1]: openshift/cluster-machine-approver#26
…1747-to-release-4.1

[release-4.1] data: add approve-csr service to approve CSRs until bootstrap is complete
@russellb russellb merged commit f0b27c5 into openshift-metal3:master May 20, 2019
@stbenjam stbenjam deleted the latest-upstream branch May 20, 2019 18:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants