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

Test version upon oc/kubectl version #6551

Merged

Conversation

feloy
Copy link
Contributor

@feloy feloy commented Jan 31, 2023

What type of PR is this:

/kind tests
What does this PR do / why we need it:

On OpenShift platforms used by prow tests, oc version does not return the OCP version.
The tests get the value from oc/kubectl version to compare this value with the one found by odo

Which issue(s) this PR fixes:

Fixes #

PR acceptance criteria:

  • Unit test

  • Integration test

  • Documentation

How to test changes / Special notes to the reviewer:

@feloy feloy requested review from rm3l and valaparthvi January 31, 2023 08:57
@netlify
Copy link

netlify bot commented Jan 31, 2023

Deploy Preview for odo-docusaurus-preview canceled.

Name Link
🔨 Latest commit 68dae1d
🔍 Latest deploy log https://app.netlify.com/sites/odo-docusaurus-preview/deploys/63d90f74918a7800093cfaff

@odo-robot
Copy link

odo-robot bot commented Jan 31, 2023

NoCluster Tests on commit a64fb97 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jan 31, 2023

OpenShift Unauthenticated Tests on commit a64fb97 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jan 31, 2023

Unit Tests on commit a64fb97 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jan 31, 2023

Kubernetes Docs Tests on commit a64fb97 finished with errors.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jan 31, 2023

Validate Tests on commit a64fb97 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jan 31, 2023

Windows Tests (OCP) on commit a64fb97 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jan 31, 2023

Kubernetes Tests on commit a64fb97 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jan 31, 2023

OpenShift Tests on commit a64fb97 finished successfully.
View logs: TXT HTML

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. Required by Prow. label Jan 31, 2023
@feloy
Copy link
Contributor Author

feloy commented Jan 31, 2023

/override Kubernetes-Integration-Tests/Kubernetes-Docs-Integration-Tests

@openshift-ci
Copy link

openshift-ci bot commented Jan 31, 2023

@feloy: Overrode contexts on behalf of feloy: Kubernetes-Integration-Tests/Kubernetes-Docs-Integration-Tests

In response to this:

/override Kubernetes-Integration-Tests/Kubernetes-Docs-Integration-Tests

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.

@feloy
Copy link
Contributor Author

feloy commented Jan 31, 2023

/override ci/prow/v4.11-integration-e2e
Passed twice

@openshift-ci
Copy link

openshift-ci bot commented Jan 31, 2023

@feloy: Overrode contexts on behalf of feloy: ci/prow/v4.11-integration-e2e

In response to this:

/override ci/prow/v4.11-integration-e2e
Passed twice

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged. Required by Prow.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants