Skip to content

internal/plugins/ansible: manager scaffolding updates for --metrics-addr and envvars#3570

Merged
joelanford merged 2 commits intooperator-framework:masterfrom
joelanford:ansible-metrics-addr
Jul 27, 2020
Merged

internal/plugins/ansible: manager scaffolding updates for --metrics-addr and envvars#3570
joelanford merged 2 commits intooperator-framework:masterfrom
joelanford:ansible-metrics-addr

Conversation

@joelanford
Copy link
Copy Markdown
Member

Description of the change:

  • Remove scaffolding of POD_NAME and WATCH_NAMESPACE
  • Use default --metrics-addr flag when auth proxy is not in use

Motivation for the change:
Closes #3567 and #3569

Checklist

If the pull request includes user-facing changes, extra documentation is required:

Copy link
Copy Markdown
Contributor

@camilamacedo86 camilamacedo86 left a comment

Choose a reason for hiding this comment

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

/lgtm

After pass in the CI

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 27, 2020
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Jul 27, 2020
Copy link
Copy Markdown
Member

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 27, 2020
Copy link
Copy Markdown
Contributor

@hasbro17 hasbro17 left a comment

Choose a reason for hiding this comment

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

LGTM

@joelanford joelanford merged commit ad2759e into operator-framework:master Jul 27, 2020
@joelanford joelanford deleted the ansible-metrics-addr branch July 27, 2020 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ansible plugin is scaffolding manager with WATCH_NAMESPACE and POD_NAME env vars

5 participants