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.

Sign up for GitHub

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

Jump to bottom

configure hostNetwork #75

Merged
merged 1 commit into from
Mar 13, 2023
Merged

configure hostNetwork #75

merged 1 commit into from
Mar 13, 2023

Conversation

khartahk
Copy link
Contributor

This gives us the ability to set hostNetwork to true. For example, required when running in EKS with cilium as CNI and tagger requires access to k8s api which is running in a separate subnet.

@khartahk khartahk force-pushed the set-hostNetwork branch 3 times, most recently from 226328f to 8c8103a Compare March 13, 2023 11:53
@khartahk
Copy link
Contributor Author

khartahk commented Mar 13, 2023 •
edited
Loading

I've changed gosec version to v2.15.0 (latest at the moment) instead of latest as the error says it does not exists. I've also updated golang.org/x/net dependecies as it threw a CVE-2022-41723 on ContainerScan

@mtougeron
Copy link
Owner

I'm sorry, looks like we were both working on changes at the same time and both updated the go.mod and gosec.yml. If you could rebase I'll merge this change and cut a new release after I update the chart with the new appVersion.

@khartahk
Copy link
Contributor Author

I've rebased and pushed the changes.

@mtougeron mtougeron merged commit b43b518 into mtougeron:main Mar 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mtougeron mtougeron mtougeron approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@khartahk @mtougeron