Skip to content

Multiarch-1310, 1391: OCP 4.9 RNs for P/Z #35828

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

Merged
merged 1 commit into from
Sep 22, 2021
Merged

Multiarch-1310, 1391: OCP 4.9 RNs for P/Z #35828

merged 1 commit into from
Sep 22, 2021

Conversation

@netlify
Copy link

netlify bot commented Aug 26, 2021

✔️ Deploy Preview for osdocs ready!

🔨 Explore the source changes: 8368076

🔍 Inspect the deploy log: https://app.netlify.com/sites/osdocs/deploys/6149624892e63c00080289dd

😎 Browse the preview: https://deploy-preview-35828--osdocs.netlify.app

@openshift-ci openshift-ci bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 26, 2021
@alkleinrh
Copy link

lgtm

@lsmcfadden
Copy link

lsmcfadden commented Sep 16, 2021

@ktania46 please ensure "Power10" matching naming guidelines and double check if you omitted "Encrypting data stored in etcd" by mistake.

Copy link

@manojnkumar manojnkumar left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 16, 2021
@lsmcfadden
Copy link

/lgtm

@openshift-ci
Copy link

openshift-ci bot commented Sep 16, 2021

@lsmcfadden: changing LGTM is restricted to collaborators

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci
Copy link

openshift-ci bot commented Sep 17, 2021

New changes are detected. LGTM label has been removed.

@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Sep 17, 2021
@ktania46
Copy link
Contributor Author

@ktania46 please ensure "Power10" matching naming guidelines and double check if you omitted "Encrypting data stored in etcd" by mistake.

reverted as per the Branding Guidelines

@ktania46 ktania46 changed the title [WIP]: Multiarch-1310, 1391: OCP 4.9 RNs for P/Z Multiarch-1310, 1391: OCP 4.9 RNs for P/Z Sep 17, 2021
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 17, 2021
@lsmcfadden
Copy link

/lgtm

@openshift-ci
Copy link

openshift-ci bot commented Sep 17, 2021

@lsmcfadden: changing LGTM is restricted to collaborators

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@holgwolf
Copy link

/lgtm

@openshift-ci
Copy link

openshift-ci bot commented Sep 17, 2021

@holgwolf: changing LGTM is restricted to collaborators

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Copy link
Contributor

@kalexand-rh kalexand-rh left a comment

Choose a reason for hiding this comment

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

This is looking good!

@kalexand-rh kalexand-rh added peer-review-done Signifies that the peer review team has reviewed this PR branch/enterprise-4.9 labels Sep 17, 2021
@kalexand-rh kalexand-rh added this to the Future Release milestone Sep 17, 2021
@SNiemann15
Copy link
Contributor

lgtm

@kalexand-rh kalexand-rh merged commit 19a6372 into openshift:enterprise-4.9 Sep 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-4.9 peer-review-done Signifies that the peer review team has reviewed this PR size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants