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.

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

cleanup: spell check getImageMirroingStatus #2247

Merged
merged 1 commit into from
Jul 14, 2021
Merged

cleanup: spell check getImageMirroingStatus #2247

merged 1 commit into from
Jul 14, 2021

Conversation

yati1998
Copy link
Contributor

@yati1998 yati1998 commented Jul 6, 2021

This commit corrects the spelling for
getImageMirroingStatus() -> getImageMirroringStatus

Signed-off-by: Yati Padia ypadia@redhat.com

@mergify mergify bot added the cleanup label Jul 6, 2021
@Madhu-1 Madhu-1 added the ready-to-merge This PR is ready to be merged and it doesn't need second review (backports only) label Jul 6, 2021
@nixpanic
Copy link
Member

nixpanic commented Jul 6, 2021

/retest ci/centos/mini-e2e/k8s-1.19

@nixpanic
Copy link
Member

nixpanic commented Jul 6, 2021

/retest ci/centos/mini-e2e/k8s-1.19

Failed to deploy Vault (logs):

Jul  6 13:57:20.947: INFO: Running '/usr/bin/kubectl --server=https://192.168.39.97:8443 --kubeconfig=/root/.kube/config --namespace=cephcsi-e2e-ede5a1e7 create --namespace=cephcsi-e2e-ede5a1e7 -f -'
Jul  6 13:57:31.387: INFO: rc: 1
Jul  6 13:57:31.387: FAIL: failed to create vault statefulset error running /usr/bin/kubectl --server=https://192.168.39.97:8443 --kubeconfig=/root/.kube/config --namespace=cephcsi-e2e-ede5a1e7 create --namespace=cephcsi-e2e-ede5a1e7 -f -:
Command stdout:
clusterrole.rbac.authorization.k8s.io/rbd-csi-vault-token-review created
clusterrolebinding.rbac.authorization.k8s.io/rbd-csi-vault-token-review created

stderr:
Error from server: error when creating "STDIN": etcdserver: request timed out

@yati1998
Copy link
Contributor Author

yati1998 commented Jul 6, 2021

/retest ci/centos/mini-e2e/k8s-1.19

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Jul 7, 2021

/retest ci/centos/mini-e2e-helm/k8s-1.20

1 similar comment
@Madhu-1
Copy link
Collaborator

Madhu-1 commented Jul 7, 2021

/retest ci/centos/mini-e2e-helm/k8s-1.20

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Jul 7, 2021

/retest ci/centos/mini-e2e-helm/k8s-1.20

X Exiting due to GUEST_PROVISION: Failed to start host: creating host: create: Error creating machine: Error in driver during machine creation: IP not available after waiting: machine minikube didn't return IP after 1 minute

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Jul 7, 2021

/retest ci/centos/mini-e2e-helm/k8s-1.21

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Jul 7, 2021

/retest ci/centos/mini-e2e-helm/k8s-1.21

X Exiting due to GUEST_PROVISION: Failed to start host: creating host: create: Error creating machine: Error in driver during machine creation: IP not available after waiting: machine minikube didn't return IP after 1 minute

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Jul 12, 2021

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented Jul 12, 2021

Command rebase: success

Branch has been successfully rebased

@nixpanic
Copy link
Member

/retest ci/centos/mini-e2e/k8s-1.21

@nixpanic
Copy link
Member

/retest ci/centos/mini-e2e/k8s-1.21

Failed to start minikube like #2264

@yati1998
Copy link
Contributor Author

/retest ci/centos/k8s-e2e-external-storage/1.21

@yati1998
Copy link
Contributor Author

/retest ci/centos/mini-e2e-helm/k8s-1.19

1 similar comment
@yati1998
Copy link
Contributor Author

/retest ci/centos/mini-e2e-helm/k8s-1.19

@yati1998
Copy link
Contributor Author

/retest ci/centos/k8s-e2e-external-storage/1.21

@yati1998
cleanup: spell check getImageMirroingStatus
574ee46 
This commit corrects the spelling for
getImageMirroingStatus() -> getImageMirroringStatus

Signed-off-by: Yati Padia <ypadia@redhat.com>
@mergify mergify bot merged commit f210d57 into ceph:devel Jul 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nixpanic nixpanic nixpanic approved these changes

@Madhu-1 Madhu-1 Madhu-1 approved these changes

@Yuggupta27 Yuggupta27 Yuggupta27 approved these changes

Assignees
No one assigned
Labels
cleanup ready-to-merge This PR is ready to be merged and it doesn't need second review (backports only)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@yati1998 @nixpanic @Madhu-1 @Yuggupta27