-
Notifications
You must be signed in to change notification settings - Fork 19
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
Knative operator job added for ppc64le #486
base: master
Are you sure you want to change the base?
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. |
config/jobs/periodic/knative/operator/main/operator-main.gen.yaml
Outdated
Show resolved
Hide resolved
@@ -0,0 +1,67 @@ | |||
periodics: | |||
- name: knative-operator-main-periodic | |||
cluster: k8s-ppc64le-cluster |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
config/jobs/periodic/knative/operator/main/operator-main.gen.yaml
Outdated
Show resolved
Hide resolved
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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?
config/prow/config.yaml
Outdated
- name: config-json-volume | ||
secret: | ||
secretName: config-json-secret | ||
defaultMode: 420 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Get rid of this no newline at end of file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
I see no activities here, @valen-mascarenhas14 any plan for completion? |
/ok-to-test |
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 , |
@mkumatag I have created copied their enterprise repo to https://github.com/ppc64le-cloud/knative-upstream-ci as they mentioned to have acquired OSSC approvals to do so. |
@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. |
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