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

Promtail pod is not istio injected #604

Open
rjferguson21 opened this issue Jul 24, 2024 · 0 comments
Open

Promtail pod is not istio injected #604

rjferguson21 opened this issue Jul 24, 2024 · 0 comments
Labels
possible-bug Something may not be working

Comments

@rjferguson21
Copy link
Contributor

rjferguson21 commented Jul 24, 2024 •
edited
Loading

As part of integrating Lula into our CI process @mjnagel noticed Promtail was running without istio injection. We're not sure how this could be possible but need to investigate further.

The current high level order of operations around istio injection for promtail are:

  • helm creates promtail namespace with uds-promtail-config chart
  • promtail Package is created (NS gets labeled by operator)
  • promtail chart gets deployed (presumably promtail namespace should be labeled by this point)

Failures in CI:
@rjferguson21 rjferguson21 added the possible-bug Something may not be working label Jul 24, 2024
@rjferguson21 rjferguson21 mentioned this issue Jul 24, 2024
Merged
5 tasks
mjnagel added a commit that referenced this issue Aug 5, 2024
@rjferguson21 @mjnagel
chore: add debug logs for istio injection logic (#602)
9075436 
## Description
Adds debug logs to better understand circumstances where istio injection
is not occurring

## Related Issue
#604

## Type of change

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Other (security config, docs update, etc)

## Checklist before merging

- [ ] Test, docs, adr added or updated as needed
- [ ] [Contributor
Guide](https://github.com/defenseunicorns/uds-template-capability/blob/main/CONTRIBUTING.md)
followed

---------

Co-authored-by: Micah Nagel <micah.nagel@defenseunicorns.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
possible-bug Something may not be working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rjferguson21