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

Kubernetes_secrets provider improvements #24912

Merged
merged 1 commit into from
Apr 7, 2021

Conversation

ChrsMark
Copy link
Member

@ChrsMark ChrsMark commented Apr 2, 2021

Minor leftovers from #24789.

  1. Return from Fetch() if k8s client is not initialised. This can happen if the provider failed to get started with Run() cause api can be unreachable. In such cases Fetch() will then panic. We need to skip Fetch() in such cases.
  2. Add the commented out section in Agent's k8s manifests about Secrets api.

Signed-off-by: chrismark <chrismarkou92@gmail.com>
@ChrsMark ChrsMark added review needs_backport PR is waiting to be backported to other branches. v7.13.0 labels Apr 2, 2021
@ChrsMark ChrsMark requested a review from blakerouse April 2, 2021 08:28
@ChrsMark ChrsMark self-assigned this Apr 2, 2021
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Apr 2, 2021
@ChrsMark ChrsMark added Team:Elastic-Agent Label for the Agent team Team:Integrations Label for the Integrations team labels Apr 2, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations (Team:Integrations)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/agent (Team:Agent)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Apr 2, 2021
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #24912 opened

  • Start Time: 2021-04-02T08:29:52.373+0000

  • Duration: 57 min 8 sec

  • Commit: b5282b6

Test stats 🧪

Test Results
Failed 0
Passed 6612
Skipped 16
Total 6628

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 6612
Skipped 16
Total 6628

@ChrsMark
Copy link
Member Author

ChrsMark commented Apr 6, 2021

@blakerouse could you have a look into this one please?

Copy link
Contributor

@blakerouse blakerouse left a comment

Choose a reason for hiding this comment

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

Looks good.

@ChrsMark ChrsMark merged commit 1cf2366 into elastic:master Apr 7, 2021
ChrsMark added a commit to ChrsMark/beats that referenced this pull request Apr 7, 2021
Signed-off-by: chrismark <chrismarkou92@gmail.com>
(cherry picked from commit 1cf2366)
@ChrsMark ChrsMark removed the needs_backport PR is waiting to be backported to other branches. label Apr 7, 2021
v1v added a commit to v1v/beats that referenced this pull request Apr 7, 2021
…eline

* upstream/master:
  CI: curl seems to be available but where cannot find it (elastic#24965)
  [mergify] backport automation to 7.x, 7.12 and 7.11 (elastic#24608)
  Only show deprecation warnings for CN-based verification once (elastic#24948)
  CI: use ubuntu-20 (elastic#24963)
  Debug empty root field instead of error (elastic#24966)
  Kubernetes_secrets provider improvements (elastic#24912)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review Team:Elastic-Agent Label for the Agent team Team:Integrations Label for the Integrations team v7.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants