-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add new jenkins job for upgrade and to run weekly #245
Conversation
Anandkumar26
commented
Jun 6, 2023
•
edited
Loading
edited
- The weekly jobs are re-named as test-name-cron
- Also update the trigger phrase for existing jobs.
- Fix alignment.
ci/jenkins/README.md
Outdated
@@ -80,3 +80,7 @@ cloud will be automatically updated with the new changes. | |||
- `nephe-test-e2e-azure-for-pull-request`: Run end-to-end Azure tests on a Kind cluster with Azure VMs | |||
- `nephe-test-e2e-eks-for-pull-request`: Run end-to-end AWS tests on EKS cluster with AWS VMs | |||
- `nephe-test-e2e-aks-for-pull-request`: Run end-to-end Azure tests on AKS cluster with Azure VMs | |||
- `nephe-test-e2e-aks-linux-agent-for-pull-request`: Run end-to-end AWS tests on EKS cluster with AWS VMs with Linux agent installed. |
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.
/nephe-test-e2e-agentless. It should test aws, azure, eks, and aks
/nephe-test-e2e-agentless-upgrade. It should test aws, azure
/nephe-test-e2e-agented. EKS and AKS for Linux and Windows
/nephe-test-e2e-aws-upgrade, /nephe-test-e2e-azure-upgrade For upgrade
/nephe-test-e2e-all. test everything
f4bc23e
to
ddbcc8e
Compare
node: 'nephe' | ||
description: 'This is the {test_name}.' | ||
branches: | ||
- ${{sha1}} |
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.
Is this still a pull request job? Is there any new added period job?
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.
yes. 2 news jobs are being added with PR for upgrade.
There should be a corresponding period job for it.
fingerprint: false | ||
only-if-success: false | ||
- '{test_name}-for-pull-request': | ||
test_name: nephe-test-e2e-azure-upgrade |
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.
azure or aks? better to have the same name.
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.
The upgrade jobs are on AWS and Azure (i.e kind cluster).
da55134
to
93bf65a
Compare
/nephe-test-e2e-aws |
/nephe-test-e2e-kind |
/nephe-test-e2e-agented |
/nephe-test-e2e-agentless-upgrade |
/nephe-test-e2e-aws-upgrade |
/nephe-test-e2e-azure-upgrade |
/nephe-test-e2e-aks |
/nephe-test-e2e-eks |
/nephe-test-e2e-kind |
1 similar comment
/nephe-test-e2e-kind |
c18212b
to
ab63155
Compare
/nephe-test-e2e-aks |
/nephe-test-e2e-eks |
7338b22
to
1216168
Compare
/nephe-test-e2e-agentless |
/nephe-test-e2e-aws |
/nephe-test-e2e-eks-linux-agent |
/nephe-test-e2e-eks |
/nephe-test-e2e-agented |
/nephe-test-e2e-agentless-upgrade |
2711c2c
to
499896c
Compare
- The weeklyjobs are re-named as testname-cron - Also update the trigger phrase for existing jobs. - Fix alignment. Signed-off-by: Anand Kumar <[email protected]>
/nephe-test-e2e-agentless |
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.
/LGTM