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

feat: add wiz manifests #8679

Open
wants to merge 39 commits into
base: dev
Choose a base branch
from
Open

feat: add wiz manifests #8679

wants to merge 39 commits into from
+719 −0

Conversation

vinaythupili
Copy link
Collaborator

-- add wiz manifests

@vinaythupili
Copy link
Collaborator Author

👍

zaklawrencea
zaklawrencea reviewed Dec 19, 2024
View reviewed changes
cluster/manifests/wiz/sensor/clusterrole.yaml Outdated Show resolved Hide resolved
cluster/manifests/wiz/sensor/clusterrole.yaml Outdated Show resolved Hide resolved
cluster/manifests/wiz/sensor/clusterrole.yaml Outdated Show resolved Hide resolved
cluster/manifests/wiz/sensor/dameonset.yaml Outdated Show resolved Hide resolved
cluster/manifests/wiz/sensor/dameonset.yaml Outdated Show resolved Hide resolved
cluster/manifests/wiz/sensor/dameonset.yaml Outdated Show resolved Hide resolved
@zaklawrencea zaklawrencea added major Major feature changes or updates, e.g. feature rollout to a new country, new API calls. do-not-merge labels Dec 19, 2024
@linki
Copy link
Member

linki commented Dec 19, 2024

We should also populate the deletions.yaml. Otherwise, the ConfigItem cannot be used to turn it off again.

@vinaythupili
Copy link
Collaborator Author

We should also populate the deletions.yaml. Otherwise, the ConfigItem cannot be used to turn it off again.

updated 👍

@katyanna
Fix typos
eb25c7f 
Signed-off-by: Katyanna Moura <[email protected]>
@demonCoder95 @zaklawrencea
Update cluster/manifests/deletions.yaml
3630e94 
Trigger deletions when daemonset is disabled.

Co-authored-by: Zak Lawrence A <[email protected]>
mikkeloscar
mikkeloscar reviewed Jan 8, 2025
View reviewed changes
cluster/manifests/wiz/sensor-daemonset.yaml Outdated
restartPolicy: Always
containers:
- name: wiz-sensor
image: container-registry-test.zalando.net/secops-systems/wiz-sensor:v1-pr-1-4
Copy link
Contributor

Choose a reason for hiding this comment

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

The images should be updated to use a production image (There are more places were a test image is used)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zaklawrencea zaklawrencea zaklawrencea left review comments

@nctram nctram nctram left review comments

@katyanna katyanna katyanna left review comments

@mikkeloscar mikkeloscar mikkeloscar left review comments

@linki linki linki left review comments

@demonCoder95 demonCoder95 Awaiting requested review from demonCoder95

Assignees
No one assigned
Labels
do-not-merge major Major feature changes or updates, e.g. feature rollout to a new country, new API calls.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@vinaythupili @linki @mikkeloscar @katyanna @nctram @zaklawrencea @demonCoder95