Knative operator job added for ppc64le #486
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: valen-mascarenhas14 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 |
|
Hi @valen-mascarenhas14. Thanks for your PR. I'm waiting for a ppc64le-cloud member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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-sigs/prow repository. |
ppc64le-cloud-bot
added
needs-ok-to-test
|
|
||
| export KUBECONFIG="$(pwd)/knative-$TIMESTAMP/kubeconfig" | ||
|
|
||
| git clone https://$(GITHUB_TOKEN)@github.ibm.com/ppc64le-automation/knative-upstream-ci.git |
There was a problem hiding this comment.
what are these scripts? right now in this repo, we are not dealing any of the code in github.ibm.com.
| @@ -0,0 +1,67 @@ | |||
| periodics: | |||
| - name: knative-operator-main-periodic | |||
| cluster: k8s-ppc64le-cluster | |||
There was a problem hiding this comment.
I suppose quay.io/powercloud/knative-prow-tests:latest is for x86 and we thought of running this job on IKS cluster.
You might have to remove this ppc64le build cluster mention here.
| decorate: true | ||
| decoration_config: | ||
| gcs_configuration: | ||
| bucket: s3://ppc64le-prow-logs |
There was a problem hiding this comment.
Need not explicitly mention the S3 bucket details here. It goes to default IBM COS if no decoration_config
| export KUBECONFIG="$(pwd)/knative-$TIMESTAMP/kubeconfig" | ||
|
|
||
| git clone https://$(GITHUB_TOKEN)@github.ibm.com/ppc64le-automation/knative-upstream-ci.git | ||
| cd knative-upstream-ci |
There was a problem hiding this comment.
Use pushd and popd while switching directories.
|
|
||
| ./test/e2e-tests.sh --run-tests | ||
|
|
||
| kubetest2 tf --powervs-region syd --powervs-zone syd05 \ |
There was a problem hiding this comment.
You must have to run the kubetest2 --down command from the same directory where you ran the --up from.
Also, did you do a test-pj run of this job yaml against any other local k8s cluster?
| - name: config-json-volume | ||
| secret: | ||
| secretName: config-json-secret | ||
| defaultMode: 420 |
There was a problem hiding this comment.
Get rid of this no newline at end of file
|
I see no activities here, @valen-mascarenhas14 any plan for completion? |
|
/ok-to-test |
ppc64le-cloud-bot
added
ok-to-test
|
@valen-mascarenhas14: The following test failed, say
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-sigs/prow repository. I understand the commands that are listed here. |
Seems like they have created a RTC task to complete OSSC approval process for migrating https://github.ibm.com/ppc64le-automation/knative-upstream-ci to public as per our suggestion. |
|
Hi @mkumatag , @Rajalakshmi-Girish , |
With the Knative community's recent decision to disable jobs for P & Z architectures in the upstream Prow CI (knative/infra#497), we are migrating the Knative CI jobs to a local Prow server to ensure continued support for Power architecture.
Here's the Slack thread for discussions & planning for this migration (discussion thread) to align with the community's needs while addressing the specific requirements for Power .
This PR marks the first step in a broader effort to ensure that Knative's CI infrastructure remains robust and inclusive for power arch.
Key Actions:
]Initiated the migration process to our local Prow CI setup to maintain CI coverage for Power architecture, starting with the knative/operator tests, which are relatively straightforward to implement.
Further migrations will follow as we continue