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

Setup role to initialize openshift-tests-private repo #30

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AdityaHonkalas
Copy link
Contributor

Signed-off-by: Aditya Honkalas [email protected]

@ppc64le-cloud-bot ppc64le-cloud-bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Dec 27, 2022
@AdityaHonkalas AdityaHonkalas changed the title Added new role for initializing the automation and e2e run New role for initializing the automation and e2e run Dec 27, 2022
examples/init_setup_vars.yaml Outdated Show resolved Hide resolved
@AdityaHonkalas AdityaHonkalas force-pushed the init-setup-role branch 3 times, most recently from b6be0a8 to 45b29df Compare January 12, 2023 03:50
@AdityaHonkalas AdityaHonkalas changed the title New role for initializing the automation and e2e run Setup role to initialize openshift-tests-private repo Jan 23, 2023
| golang_install_tarball | no | `https://go.dev/dl/go1.19.4.linux-ppc64le.tar.gz` | Golang tarball for `ppc64le` arch with version `>= 1.18`, the default is given for version `1.19.4`. |
| openshift_tests_private_basedir | no | `~/ocp-validation` | Base directory path to clone the `openshift-tests-private` repo. It will get created if path not exists or not provided. |
| openshift_tests_private_clone_url | no | `https://github.com/openshift/openshift-tests-private` | Git clone URL, the default has been set for `openshift-tests-private` repository. |
| openshift_tests_private_branch | no | master | Git branch name, the default branch name has been set as `master`. |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: no need to say "the default is...." in the comments as it is already provided in Default column.

when: github_username == "" or github_personal_access_token == ""

- name: Install the development tools
yum:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is'nt it better to use dnf instead? @pravin-dsilva

@ppc64le-cloud-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: AdityaHonkalas
Once this PR has been reviewed and has the lgtm label, please assign yussufsh for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@yussufsh
Copy link
Contributor

@AdityaHonkalas sorry for keeping this PR for long... could you please rebase and validate if this code is still ok to merge?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants