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

Helm charts: Command is hardcoded in _container_export_stdout.tpl #118

Closed
michaelhyatt opened this issue Jun 2, 2022 · 0 comments · Fixed by #121
Closed

Helm charts: Command is hardcoded in _container_export_stdout.tpl #118

michaelhyatt opened this issue Jun 2, 2022 · 0 comments · Fixed by #121
Labels
good first issue Good for newcomers

Comments

@michaelhyatt
Copy link
Contributor

It is currently set to be hubble-export-stdout and it is a good idea to make it configurable, similar to the rest of the export template, so it is possible to plug in another exporter container for tetragon pod.

@willfindlay willfindlay added the good first issue Good for newcomers label Jun 2, 2022
mrtc0 added a commit to mrtc0/tetragon that referenced this issue Jun 3, 2022
Resolved cilium#118
Supports plugging in another exporter container.

Signed-off-by: Kohei Morita <[email protected]>
jrfastab pushed a commit that referenced this issue Jun 7, 2022
Resolved #118
Supports plugging in another exporter container.

Signed-off-by: Kohei Morita <[email protected]>
Taction added a commit to Taction/tetragon that referenced this issue Jun 9, 2022
…debug

* commit '469bddbb28b43380456d2f4254066b91560e50b7': (24 commits)
  Don't truncate Ktime to milliseconds
  build(deps): bump github.com/stretchr/testify from 1.7.1 to 1.7.2
  checkpatch: ignore OPEN_ENDED_LINE and PREFER_KERNEL_TYPES
  chore: remove accidentally checked in file
  build(deps): bump github.com/google/go-cmp from 0.5.6 to 0.5.8
  chore: remove protoc-gen-go-tetragon binary
  dockerignore: fixes + add doc comments
  operator: fix tool name in log output
  build(deps): bump github/codeql-action from 2.1.11 to 2.1.12
  ci/generated-files: add explanatory message
  ci: add a job to lint helm chart
  ci/checkpatch: rename static-checks job
  fix(helm): Support command override in exporter container cilium#118
  namespaces: fix GetCurrentNamespace() on 5.4
  tests/exec: add TestNamespaces
  Remove errors related to unresolved mount points
  Fix tests with path resolution and mount points
  Increase the number of iterations in path resolution (32 -> 128)
  Delete deprecated code for path resolution (get_full_path)
  Use __d_path_local for getting process cwd
  ...

Signed-off-by: zhangchao <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants