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

Add feature flag for ignoring proxy within oneAgent and activeGate #737

Merged
merged 5 commits into from
Apr 26, 2022

Conversation

luhi-DT
Copy link
Collaborator

@luhi-DT luhi-DT commented Apr 21, 2022

Description

Currently we have the problem, when a proxy is set in the custom resource, it will get applied to all the components (operator, oneAgent and activeGate). This is so far intended, but if there is an activeGate with the purpose of routing agents traffic, then the oneAgent will also try to connect to the activeGate using the proxy, which causes problems

How can this be tested?

Setup the operator and set a proxy in the customResource. This proxy will get applied to dtclient, oneAgents and activeGate (if defined in the same CR). If you add one or both featureFlags

  • "operator.dynatrace.com/feature-activegate-ignore-proxy=true"
  • "operator.dynatrace.com/feature-oneagent-ignore-proxy=true"

proxy should not be set for activeGate and/or oneAgent

Checklist

  • Unit tests have been updated/added
  • PR is labeled accordingly

@luhi-DT luhi-DT added the bug Something isn't working label Apr 21, 2022
@luhi-DT luhi-DT requested a review from a team as a code owner April 21, 2022 12:27
meik99
meik99 previously requested changes Apr 21, 2022
src/controllers/dynakube/dynakube_controller.go Outdated Show resolved Hide resolved
src/controllers/dynakube/oneagent/daemonset/arguments.go Outdated Show resolved Hide resolved
@luhi-DT luhi-DT requested a review from meik99 April 25, 2022 06:09
@luhi-DT luhi-DT dismissed meik99’s stale review April 26, 2022 09:43

already implemented

@luhi-DT luhi-DT merged commit 67e0f31 into master Apr 26, 2022
@luhi-DT luhi-DT deleted the bugfix/add-feature-flag-for-proxy branch April 26, 2022 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants