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

seperate ansible scripts #6484

Merged
merged 6 commits into from
May 11, 2023
Merged

seperate ansible scripts #6484

merged 6 commits into from
May 11, 2023

Conversation

anandrkskd
Copy link
Contributor

@anandrkskd anandrkskd commented Jan 11, 2023 •
edited
Loading

Signed-off-by: anandrkskd [email protected]

What type of PR is this:

What does this PR do / why we need it:

Which issue(s) this PR fixes:

Fixes #6455

PR acceptance criteria:

  • Unit test

  • Integration test

  • Documentation

How to test changes / Special notes to the reviewer:

@netlify
Copy link

netlify bot commented Jan 11, 2023 •
edited
Loading

✅ Deploy Preview for odo-docusaurus-preview canceled.

Name Link
🔨 Latest commit 31d1d88
🔍 Latest deploy log https://app.netlify.com/sites/odo-docusaurus-preview/deploys/645ccf1d9ef92b00082086f7

@odo-robot
Copy link

odo-robot bot commented Jan 11, 2023 •
edited
Loading

Windows Tests (OCP) on commit daea529 finished with errors.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jan 11, 2023 •
edited
Loading

OpenShift Tests on commit 6b44c10 finished with errors.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jan 11, 2023 •
edited
Loading

NoCluster Tests on commit daea529 finished with errors.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jan 11, 2023 •
edited
Loading

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

@odo-robot
Copy link

odo-robot bot commented Jan 11, 2023 •
edited
Loading

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

@odo-robot
Copy link

odo-robot bot commented Jan 11, 2023 •
edited
Loading

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

@anandrkskd anandrkskd force-pushed the test/6455 branch 5 times, most recently from dd46a0c to 0a574b9 Compare January 19, 2023 05:51
@feloy
Copy link
Contributor

feloy commented Feb 14, 2023

The README file at the root (scripts/ansible/README.md) is empty. Could you please document how to run the script manually (if one wants to deploy the infra into its own account to test it).

@odo-robot
Copy link

odo-robot bot commented Feb 28, 2023 •
edited
Loading

OpenShift Unauthenticated Tests on commit daea529 finished with errors.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Feb 28, 2023 •
edited
Loading

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

@feloy
Copy link
Contributor

feloy commented Mar 22, 2023

Error: ENOENT: no such file or directory, open 'scripts/ansible/requirements.yml'

The github action seems to fail

@anandrkskd
fix
3c67d1e 
Signed-off-by: anandrkskd <[email protected]>
@anandrkskd
fix
716bcb4 
Signed-off-by: anandrkskd <[email protected]>
@anandrkskd
rename requirements.yaml
8efd594 
Signed-off-by: anandrkskd <[email protected]>
@anandrkskd
change permissions
8787a15 
Signed-off-by: anandrkskd <[email protected]>
@anandrkskd
update cluster version
31d1d88 
Signed-off-by: anandrkskd <[email protected]>
@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. Required by Prow. label May 11, 2023
@feloy feloy closed this May 11, 2023
@feloy feloy reopened this May 11, 2023
@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
No Duplication information No Duplication information

@feloy
Copy link
Contributor

feloy commented May 11, 2023

/override Kubernetes-Integration-Tests/Kubernetes-Docs-Integration-Tests
/override NoCluster-Integration-tests/NoCluster-Integration-tests
/override OpenShift-Integration-tests/OpenShift-Integration-tests
/override OpenShift-Integration-tests/OpenShift-Unauth-Integration-tests
/override windows-integration-test/Windows-test
Not related tests

@openshift-ci
Copy link

openshift-ci bot commented May 11, 2023

@feloy: Overrode contexts on behalf of feloy: Kubernetes-Integration-Tests/Kubernetes-Docs-Integration-Tests, NoCluster-Integration-tests/NoCluster-Integration-tests, OpenShift-Integration-tests/OpenShift-Integration-tests, OpenShift-Integration-tests/OpenShift-Unauth-Integration-tests, windows-integration-test/Windows-test

In response to this:

/override Kubernetes-Integration-Tests/Kubernetes-Docs-Integration-Tests
/override NoCluster-Integration-tests/NoCluster-Integration-tests
/override OpenShift-Integration-tests/OpenShift-Integration-tests
/override OpenShift-Integration-tests/OpenShift-Unauth-Integration-tests
/override windows-integration-test/Windows-test
Not related 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

feloy commented May 11, 2023

/override odo-infra-stage-test / kubernetes-infra-stage-test
Failing on checking NFS server (to be debugged)

@openshift-ci
Copy link

openshift-ci bot commented May 11, 2023

@feloy: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • /
  • odo-infra-stage-test

Only the following failed contexts/checkruns were expected:

  • Kubernetes-Integration-Tests/Kubernetes-Docs-Integration-Tests
  • Kubernetes-Integration-Tests/Kubernetes-Integration-Tests
  • NoCluster-Integration-tests/NoCluster-Integration-tests
  • OpenShift-Integration-tests/OpenShift-Integration-tests
  • OpenShift-Integration-tests/OpenShift-Unauth-Integration-tests
  • Unit-Tests/Unit-Tests
  • ci/prow/odo-ocp4.13-lp-interop-images
  • ci/prow/v4.10-images
  • ci/prow/v4.11-images
  • ci/prow/v4.12-images
  • ci/prow/v4.13-images
  • kubernetes-infra-stage-test
  • netlify/odo-docusaurus-preview/deploy-preview
  • pull-ci-redhat-developer-odo-main-odo-ocp4.13-lp-interop-images
  • pull-ci-redhat-developer-odo-main-v4.10-images
  • pull-ci-redhat-developer-odo-main-v4.11-images
  • pull-ci-redhat-developer-odo-main-v4.12-images
  • pull-ci-redhat-developer-odo-main-v4.13-images
  • tide
  • validator/Validate
  • windows-integration-test/Windows-test

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

In response to this:

/override odo-infra-stage-test / kubernetes-infra-stage-test
Failing on checking NFS server (to be debugged)

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

feloy commented May 11, 2023

/override odo-infra-stage-test

@openshift-ci
Copy link

openshift-ci bot commented May 11, 2023

@feloy: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • odo-infra-stage-test

Only the following failed contexts/checkruns were expected:

  • Kubernetes-Integration-Tests/Kubernetes-Docs-Integration-Tests
  • Kubernetes-Integration-Tests/Kubernetes-Integration-Tests
  • NoCluster-Integration-tests/NoCluster-Integration-tests
  • OpenShift-Integration-tests/OpenShift-Integration-tests
  • OpenShift-Integration-tests/OpenShift-Unauth-Integration-tests
  • Unit-Tests/Unit-Tests
  • ci/prow/odo-ocp4.13-lp-interop-images
  • ci/prow/v4.10-images
  • ci/prow/v4.11-images
  • ci/prow/v4.12-images
  • ci/prow/v4.13-images
  • kubernetes-infra-stage-test
  • netlify/odo-docusaurus-preview/deploy-preview
  • pull-ci-redhat-developer-odo-main-odo-ocp4.13-lp-interop-images
  • pull-ci-redhat-developer-odo-main-v4.10-images
  • pull-ci-redhat-developer-odo-main-v4.11-images
  • pull-ci-redhat-developer-odo-main-v4.12-images
  • pull-ci-redhat-developer-odo-main-v4.13-images
  • tide
  • validator/Validate
  • windows-integration-test/Windows-test

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

In response to this:

/override odo-infra-stage-test

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

feloy commented May 11, 2023

/override kubernetes-infra-stage-test

@openshift-ci
Copy link

openshift-ci bot commented May 11, 2023

@feloy: Overrode contexts on behalf of feloy: kubernetes-infra-stage-test

In response to this:

/override kubernetes-infra-stage-test

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.

@openshift-merge-robot openshift-merge-robot merged commit 7ff460b into redhat-developer:main May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@feloy feloy feloy approved these changes

@rnapoles-rh rnapoles-rh Awaiting requested review from rnapoles-rh

@valaparthvi valaparthvi Awaiting requested review from valaparthvi

Assignees

@feloy feloy

Labels
lgtm Indicates that a PR is ready to be merged. Required by Prow.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Organize and clean-up IBMCloud Infra-As-A-Code
3 participants
@anandrkskd @feloy @openshift-merge-robot