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

Pulled in velero main updates as of 21 Sept 2022. #825

Merged
merged 2 commits into from
Sep 21, 2022

Conversation

sseago
Copy link
Contributor

@sseago sseago commented Sep 15, 2022

This includes vsl credentials, which results in CRD changes for both velero vsl and oadp DPA.

@sseago sseago force-pushed the velero-update-20220915 branch 2 times, most recently from d3c05a2 to 9d90a89 Compare September 19, 2022 13:20
@sseago sseago changed the title Pulled in velero main updates as of 15 Sept 2022. Pulled in velero main updates as of 19 Sept 2022. Sep 19, 2022
@sseago sseago force-pushed the velero-update-20220915 branch from 9d90a89 to f1acc28 Compare September 20, 2022 14:27
@sseago sseago changed the title Pulled in velero main updates as of 19 Sept 2022. Pulled in velero main updates as of 20 Sept 2022. Sep 20, 2022
@sseago
Copy link
Contributor Author

sseago commented Sep 20, 2022

/retest

1 similar comment
@kaovilai
Copy link
Member

/retest

@kaovilai
Copy link
Member

time="2022-09-20T16:14:37Z" level=error msg="Error backing up item" backup=openshift-adp/mongo-restic-e2e-e1adb3ec-38fe-11ed-ac16-0a580a81c38e error="pod is not running, name=todolist-1-build, namespace=mongo-persistent, phase=Succeeded" error.file="/go/src/github.com/vmware-tanzu/velero/pkg/util/kube/pod.go:31" error.function=github.com/vmware-tanzu/velero/pkg/util/kube.IsPodRunning logSource="pkg/backup/backup.go:425" name=todolist-1-build

@kaovilai
Copy link
Member

#825 (comment)

I think this is expected. This pod after a time will have a status of complete. It is not a pod that is always in running status.

@kaovilai
Copy link
Member

Need to append fix to vmware-tanzu/velero#5319

@kaovilai

This comment was marked as resolved.

@kaovilai

This comment was marked as outdated.

@kaovilai
Copy link
Member

sseago#3

@sseago
Copy link
Contributor Author

sseago commented Sep 20, 2022

/retest

2 similar comments
@sseago
Copy link
Contributor Author

sseago commented Sep 21, 2022

/retest

@kaovilai
Copy link
Member

/retest

This includes vsl credentials, which results in CRD
changes for both velero vsl and oadp DPA.
@sseago sseago force-pushed the velero-update-20220915 branch from 96d0dc5 to d6ed8e3 Compare September 21, 2022 13:37
@sseago sseago changed the title Pulled in velero main updates as of 20 Sept 2022. Pulled in velero main updates as of 21 Sept 2022. Sep 21, 2022
@sseago
Copy link
Contributor Author

sseago commented Sep 21, 2022

/retest

@kaovilai
Copy link
Member

This PR was lost.
https://github.com/sseago/oadp-operator/pull/3/files
Cherrypick to re-resolve

time=\"2022-09-21T14:53:14Z\" level=error msg=\"Error backing up item\" backup=openshift-adp/mongo-restic-e2e-c503a60a-39bc-11ed-b809-0a580a81782c error=\"pod is not running, name=todolist-1-build, namespace=mongo-persistent, phase=Succeeded\" error.file=\"/go/src/github.com/vmware-tanzu/velero/pkg/util/kube/pod.go:31\" error.function=github.com/vmware-tanzu/velero/pkg/util/kube.IsPodRunning logSource=\"pkg/backup/backup.go:425\" name=todolist-1-build

@kaovilai
Copy link
Member

sseago#4

@sseago
Copy link
Contributor Author

sseago commented Sep 21, 2022

/retest

@hhpatel14
Copy link
Contributor

/test 4.11-operator-e2e-aws

@sseago
Copy link
Contributor Author

sseago commented Sep 21, 2022

/retest

1 similar comment
@sseago
Copy link
Contributor Author

sseago commented Sep 21, 2022

/retest

@openshift-ci
Copy link

openshift-ci bot commented Sep 21, 2022

@sseago: all tests passed!

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

@sseago sseago merged commit d6f0886 into openshift:master Sep 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants