Skip to content

Note on using the oc adm commands on first master#7781

Merged
mburke5678 merged 1 commit intoopenshift:masterfrom
mburke5678:BZ-1432978
Feb 26, 2018
Merged

Note on using the oc adm commands on first master#7781
mburke5678 merged 1 commit intoopenshift:masterfrom
mburke5678:BZ-1432978

Conversation

@mburke5678
Copy link
Copy Markdown
Contributor

@mburke5678 mburke5678 commented Feb 13, 2018

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 13, 2018
@mburke5678
Copy link
Copy Markdown
Contributor Author

mburke5678 commented Feb 13, 2018

@sferich888 The oadm command has been deprecated since you files the original BZ/ Does the same restrictions apply to the oc adm commands?

@mburke5678
Copy link
Copy Markdown
Contributor Author

mburke5678 commented Feb 19, 2018 •
edited
Loading

@sferich888 PTAL
@sdodson
When running oc adm commands, you should run them only from only the first master listed in the Ansible host inventory file, by default /etc/ansible/hosts.

For example, if you execute certificate commands oc adm ca, to create a certificate on the one master, then run the same (or similar) command on a second master the serial numbers in the certificates might not be be unique (because {product-title} uses a flat file for the serial numbers).

@mburke5678
Copy link
Copy Markdown
Contributor Author

mburke5678 commented Feb 21, 2018

@xingxingxia PTAL

@openshift-ci-robot openshift-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Feb 22, 2018
@xingxingxia
Copy link
Copy Markdown
Contributor

xingxingxia commented Feb 26, 2018

FYI, https://bugzilla.redhat.com/show_bug.cgi?id=1432978#c10 , comment from QE of Master sub team who knows CA related background

adellape
adellape reviewed Feb 26, 2018
View reviewed changes
Comment thread getting_started/configure_openshift.adoc Outdated
Copy link
Copy Markdown
Contributor

@adellape adellape Feb 26, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs filename markup, here and throughout.

@mburke5678 mburke5678 force-pushed the BZ-1432978 branch 2 times, most recently from 8bafc4f to e119637 Compare February 26, 2018 16:23
@mburke5678
Copy link
Copy Markdown
Contributor Author

mburke5678 commented Feb 26, 2018

@adellape Changed /etc/ansible/hosts to /etc/ansible/hosts.

@mburke5678 mburke5678 merged commit e77f7db into openshift:master Feb 26, 2018
This was referenced Feb 26, 2018
Closed
Merged
This was referenced Feb 26, 2018
Merged
Merged
@mburke5678 mburke5678 deleted the BZ-1432978 branch August 2, 2018 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sdodson sdodson sdodson approved these changes

+1 more reviewer

@adellape adellape adellape left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

branch/enterprise-3.6 branch/enterprise-3.7 branch/enterprise-3.9 size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@mburke5678 @xingxingxia @sdodson @adellape @openshift-ci-robot