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

feat(agent): add support for Windows Agent #1730

Draft
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

iurly
Copy link
Collaborator

@iurly iurly commented May 6, 2024

What this PR does / why we need it:

This is draft/WIP PR to add support for the Windows Agent
Please note:

  • The structure of the new values is open for discussion
  • the access key is currently added to the configmap (dragent.yaml) to allow testing, pending changes to the agent's code (see SSPROD-39123). THIS CHANGE SHOULD NOT BE MERGED
  • as of today, the image published on quay does not work out of the box, so a specially crafted image should be specified through custom values.yaml

Checklist

  • Title of the PR starts with type and scope, (e.g. feat(agent,node-analyzer,sysdig-deploy):)
  • Chart Version bumped for the respective charts
  • Variables are documented in the README.md (or README.tpl in some charts)
  • Check GithubAction checks (like lint) to avoid merge-check stoppers
  • All test files are added in the tests folder of their respective chart and have a "_test" suffix

@iurly iurly requested a review from a team as a code owner May 6, 2024 10:29
@iurly iurly marked this pull request as draft May 6, 2024 10:29
{{- else }}
secretName: {{ include "agent.accessKeySecret" . }}
{{- end }}
nodeSelector:
Copy link
Contributor

Choose a reason for hiding this comment

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

@iurly would it make sense to add

"kubernetes.io/os": linux

in:

  • charts/agent/templates/daemonset.yaml
  • charts/agent/templates/deployment.yaml
    ?

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.

None yet

2 participants