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

Add event watching to deploy #405

Merged
merged 2 commits into from
Jul 26, 2023
Merged

Add event watching to deploy #405

merged 2 commits into from
Jul 26, 2023

Conversation

alexmasi
Copy link
Contributor

tested locally:

$ kne deploy ...
....
18:54:08 NS: lemming-operator Event name: lemming-controller-manager-6fc9d47f7d-tfbff.17757fb47c4a770a Type: Normal Message: Container image "gcr.io/kubebuilder/kube-rbac-proxy:v0.12.0" already present on machine
18:54:08 NS: lemming-operator Event name: lemming-controller-manager-6fc9d47f7d-tfbff.17757fb47d878ad2 Type: Normal Message: Created container kube-rbac-proxy
18:54:08 NS: lemming-operator Event name: lemming-controller-manager-6fc9d47f7d-tfbff.17757fb49ab9b5a7 Type: Normal Message: Started container kube-rbac-proxy
....

@coveralls
Copy link

coveralls commented Jul 26, 2023

Pull Request Test Coverage Report for Build 5674290242

  • 0 of 8 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.09%) to 65.537%

Changes Missing Coverage Covered Lines Changed/Added Lines %
deploy/deploy.go 0 8 0.0%
Totals Coverage Status
Change from base Build 5659185573: -0.09%
Covered Lines: 3906
Relevant Lines: 5960

💛 - Coveralls

NehaManjunath
NehaManjunath previously approved these changes Jul 26, 2023
deploy/deploy.go Show resolved Hide resolved
@alexmasi alexmasi merged commit 635256a into main Jul 26, 2023
6 checks passed
@alexmasi alexmasi deleted the dep-watch branch July 26, 2023 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants