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

CI: Changed lint Helm CI trigger #2804

Merged

Conversation

PhilipSchmid
Copy link
Collaborator

Description

The 'lint Helm' GitHub workflow should also be triggered when its definition changes. Required for a follow up improvement of it.

Changelog

CI: Changed lint Helm CI trigger

The 'lint Helm' GitHub workflow should also be triggered when
its definition changes.

Signed-off-by: Philip Schmid <[email protected]>
@PhilipSchmid PhilipSchmid requested review from willfindlay and a team as code owners August 16, 2024 07:18
Copy link

netlify bot commented Aug 16, 2024

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 99d2057
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/66befd5be2d4720008da04c6
😎 Deploy Preview https://deploy-preview-2804--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mtardy mtardy added the release-note/ci This PR makes changes to the CI. label Aug 16, 2024
Copy link
Member

@mtardy mtardy left a comment

Choose a reason for hiding this comment

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

agree! thanks! :)

@mtardy
Copy link
Member

mtardy commented Aug 16, 2024

the error is a known flake we can ignore (for now) #2576

@lambdanis lambdanis merged commit a3da4f6 into cilium:main Aug 16, 2024
42 of 44 checks passed
@PhilipSchmid PhilipSchmid deleted the pr/philip/changed_lint_helm_ci_trigger branch August 16, 2024 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants