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

fix(build): inject Build Info into systemd package #1928

Closed
wants to merge 7 commits into from

Conversation

Atharva-Kanherkar
Copy link
Contributor

Purpose of PR?:
This PR ensures that Build Info is included in the KubeArmor systemd package, which was previously missing.

Fixes #1921

Does this PR introduce a breaking change?
No

Additional information for reviewer? :
No

Checklist:

@rksharma95
Copy link
Collaborator

@Atharva-Kanherkar thanks for the PR, can you please sign-off, and squash the commits.

@Atharva-Kanherkar
Copy link
Contributor Author

@Atharva-Kanherkar thanks for the PR, can you please sign-off, and squash the commits.

sure!

Signed-off-by: Atharva-Kanerkar <atharvakanherkar25@gmail.com>
Signed-off-by: Atharva-Kanerkar <atharvakanherkar25@gmail.com>
Signed-off-by: Atharva-Kanerkar <atharvakanherkar25@gmail.com>
Signed-off-by: Atharva-Kanerkar <atharvakanherkar25@gmail.com>
@Atharva-Kanherkar
Copy link
Contributor Author

Hello, @DelusionalOptimist Kindly please review the PR :)

@tesla59
Copy link
Contributor

tesla59 commented Dec 28, 2024

Hey @Atharva-Kanherkar
Could u please squash your commits (ref)

@@ -10,6 +10,10 @@ builds:
- arm64
env:
- CGO_ENABLED=0
ldflags:
- "-X main.BuildDate={{.Date}} -X main.GitCommit={{.Commit}} -X main.GitBranch={{.Branch}} -X main.GitSummary={{.Summary}}"
Copy link
Contributor

Choose a reason for hiding this comment

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

u can add the flags in different lines instead of writing all flags in single line

@Atharva-Kanherkar
Copy link
Contributor Author

Hey @Atharva-Kanherkar Could u please squash your commits (ref)

I have squashed them!

@Aryan-sharma11
Copy link
Member

Aryan-sharma11 commented Jan 7, 2025

@Atharva-Kanherkar Thanks for the PR. Kindly address the review comments by @tesla59 and also squash your commits.

@Atharva-Kanherkar
Copy link
Contributor Author

@Atharva-Kanherkar Thanks for the PR. Kindly address the review comments by @tesla59 and also squash your commits.

Okay! I am sorry but there was some problem which I could not solve so I will just open a new PR. Sorry for the inconvenience.

rksharma95 added a commit that referenced this pull request Jan 9, 2025
fix(build): inject Build Info into systemd package #1928
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.

Missing build info in KubeArmor systemd package
4 participants