Skip to content

Fix migration-mtc-workflow.adoc #52969

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Fix migration-mtc-workflow.adoc #52969

wants to merge 1 commit into from

Conversation

ptux
Copy link

@ptux ptux commented Nov 17, 2022

Version(s):

All

Issue:

PVs are cluster-scoped and PVCs are namespace-scoped resources.

@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Nov 17, 2022
@ocpdocs-previewbot
Copy link

🤖 Build preview is available at:
https://52969--docspreview.netlify.app

Build log: https://circleci.com/gh/ocpdocs-previewbot/openshift-docs/3769

@ptux
Copy link
Author

ptux commented Feb 1, 2023

@RichardHoch

this is a friendly reminder.
Cloud you review this?

@RichardHoch
Copy link
Contributor

@ptux My apologies; I must not have seen the notification from @kalexand-rh . Let me check this out and, if needed, add it to our queue. Thanks for bringing it to our attention.

@anarnold97
Copy link
Contributor

JIRAPROJECT-MIG-1300

stillalearner
stillalearner approved these changes Feb 2, 2023
View reviewed changes
Copy link

@stillalearner stillalearner left a comment

Choose a reason for hiding this comment

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

lgtm

@RichardHoch
Copy link
Contributor

/label MTC
/abel peer-review-needed

@openshift-ci openshift-ci bot added the MTC Label for all MTC PRs label Feb 5, 2023
@RichardHoch
Copy link
Contributor

MTC 1.x; OCP 4.9+ -- that is, ready for immediate addition to MTC documentation.

@ptux
Copy link
Author

ptux commented Mar 4, 2023

@RichardHoch
This is a friendly reminder.

@RichardHoch
Copy link
Contributor

/label MTC
/label peer-review-needed

@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label Mar 5, 2023
@RichardHoch
Copy link
Contributor

MTC 1.x; OCP 4.9+ -- that is, ready for immediate addition to MTC documentation.

@RichardHoch
Copy link
Contributor

MTC 1.x; OCP 4.9+ -- that is, ready for immediate addition to MTC documentation.
QE passed.

@RichardHoch
Copy link
Contributor

/label remove-label peer-review-needed

@RichardHoch
Copy link
Contributor

RichardHoch commented Mar 5, 2023 •
edited
Loading

@ptux This error was fixed in #55466. Please let me know if that PR did not solve the problem. If everything is OK, please close this PR.

@RichardHoch
Copy link
Contributor

OCP docs team -- this PR does not need review.

@gaurav-nelson gaurav-nelson added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Mar 6, 2023
@gaurav-nelson
Copy link
Contributor

Closing this one, already done via #55466

@RichardHoch
Copy link
Contributor

Thanks @gaurav-nelson -- sorry for the confusion. I didn't have an active Close button for some reason or I would have closed it myself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@stillalearner stillalearner stillalearner approved these changes

@RichardHoch RichardHoch RichardHoch approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

MTC Label for all MTC PRs peer-review-done Signifies that the peer review team has reviewed this PR size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@ptux @ocpdocs-previewbot @RichardHoch @anarnold97 @gaurav-nelson @stillalearner