-
Notifications
You must be signed in to change notification settings - Fork 42
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
Update upload artifacts action #2382
Conversation
This pull request does not have a backport label. Could you fix it @gurevichdmitry? 🙏
|
📊 Allure Report - 💚 No failures were reported.
|
@@ -7,7 +7,7 @@ replicaCount: 1 | |||
image: | |||
name: cloudbeat-test | |||
repository: cloudbeat-test | |||
pullPolicy: Always | |||
pullPolicy: IfNotPresent |
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.
For k8s tests, the images are loaded to the nodes locally, so the pull policy should be IfNotPresent
.
@oren-zohar, since the tests are using |
(cherry picked from commit cb6ece0) # Conflicts: # tests/test_environments/k8s-cloudbeat-tests/values.yaml
* Update upload artifacts action (#2382) (cherry picked from commit cb6ece0) # Conflicts: # tests/test_environments/k8s-cloudbeat-tests/values.yaml * conflict fix --------- Co-authored-by: Dmitry Gurevich <99176494+gurevichdmitry@users.noreply.github.com> Co-authored-by: Kostas Stamatakis <moukoublen@gmail.com>
Summary of your changes
This PR pins the upload artifacts version to v4.3.4 due to this bug and also fixes the k8s CI.
Screenshot/Data
Related Issues
Checklist
Introducing a new rule?