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

No way to represent a success with warnings scenario at present #96

Open
jmontleon opened this issue Jul 12, 2022 · 6 comments
Open

No way to represent a success with warnings scenario at present #96

jmontleon opened this issue Jul 12, 2022 · 6 comments
Assignees
Labels
needs-kind priority/critical Must be staffed and worked on either currently, or very soon, ideally in time for the next release. triage/accepted Indicates an issue or PR is ready to be actively worked on.

Comments

@jmontleon
Copy link
Contributor

There are some cases where a migration might fail to create some resources, but otherwise the migration succeeded.

One scenario we're currently aware of is migrating from a DevSandbox instance to your own OpenShift cluster as a non-admin. In this scenario you will be able to read some resources that can only be created as a cluster admin, such as limitranges.

As a namespace admin on both clusters you will be able to read and export the limitrange, but unable to create it on the destination.

This doesn't affect the application from migrating or running, but at present will cause the pipeline to fail.

In MTC we had the notion of a success with warnings, which would have covered the case where some resources could not be recreated.

@github-actions
Copy link

This issue is currently awaiting triage.
If contributors determine this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.
The triage/accepted label can be added by org members by writing /triage accepted in a comment.

@github-actions github-actions bot added needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. needs-kind needs-priority labels Jul 12, 2022
@eriknelson
Copy link

/priority critical
/type feature
/triage accepted

@openshift-ci openshift-ci bot added priority/critical Must be staffed and worked on either currently, or very soon, ideally in time for the next release. triage/accepted Indicates an issue or PR is ready to be actively worked on. labels Jul 12, 2022
@github-actions github-actions bot removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. needs-priority labels Jul 12, 2022
@github-actions
Copy link

This issue synced with: https://issues.redhat.com/browse/MTRHO-91

@mturley
Copy link
Contributor

mturley commented Jul 12, 2022

Do we know if this is something supported by the existing Pipelines UI / Tekton status reporting? As it stands our plugin doesn't do any kind of progress/status reporting (just kicks the user over to the PipelineRun page) and with the new management page I'm adding in #88 the goal is to only very minimally report status (ideally reusing code from the Pipelines UI) and link to that PipelineRun page for detailed status.

@eriknelson
Copy link

@mturley I don't know if pipelines has any real support for this. We should check with that team perhaps in slack and update with the answer here.

@shawn-hurley
Copy link

If there is something that a user can not do, it should fail, and the user should make a choice.

We should have transforms/options for them to re-run if they just want to remove the objects that failed, but I don't think that hiding this information should be the goal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-kind priority/critical Must be staffed and worked on either currently, or very soon, ideally in time for the next release. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
None yet
Development

No branches or pull requests

4 participants