Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

[kibana] adding extra volumes and extra volume mounts (#557) #1264

Merged
merged 1 commit into from
Jul 5, 2021

Conversation

DilasserT
Copy link
Contributor

  • Chart version not bumped (the versions are all bumped and released at the same time)
  • README.md updated with any new values or changes
  • Updated template tests in ${CHART}/tests/*.py
  • Updated integration tests in ${CHART}/examples/*/test/goss.yaml

PR Following #1247

Adding options to mount additionnal volumes with extraVolumes and extraVolumeMounts through values.yaml

@jmlrt kibana/tests/kibana_test.py formatted using black

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

1 similar comment
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@cla-checker-service
Copy link

cla-checker-service bot commented Jun 24, 2021

💚 CLA has been signed

@DilasserT DilasserT closed this Jun 24, 2021
@DilasserT DilasserT reopened this Jun 24, 2021
@DilasserT
Copy link
Contributor Author

Not sure why the CLA check does not pass, I resigned it and tried to refresh the PR with no success

@jmlrt
Copy link
Member

jmlrt commented Jun 24, 2021

cla/check

@DilasserT
Copy link
Contributor Author

What can I do to resolve this ?

@DilasserT DilasserT force-pushed the feature/kibana-extra-volume branch from 972a395 to 8cb32a5 Compare June 25, 2021 13:51
@DilasserT
Copy link
Contributor Author

Committed with the wrong user...

@DilasserT
Copy link
Contributor Author

@jmlrt Is this PR alright ?

@jmlrt jmlrt added enhancement New feature or request kibana labels Jul 5, 2021
Copy link
Member

@jmlrt jmlrt left a comment

Choose a reason for hiding this comment

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

LGTM⛴

@jmlrt
Copy link
Member

jmlrt commented Jul 5, 2021

jenkins test this please

@jmlrt jmlrt merged commit b870c3f into elastic:master Jul 5, 2021
jmlrt pushed a commit to jmlrt/helm-charts that referenced this pull request Jul 5, 2021
jmlrt pushed a commit to jmlrt/helm-charts that referenced this pull request Jul 5, 2021
jmlrt pushed a commit to jmlrt/helm-charts that referenced this pull request Jul 5, 2021
jmlrt pushed a commit to jmlrt/helm-charts that referenced this pull request Jul 6, 2021
jmlrt pushed a commit to jmlrt/helm-charts that referenced this pull request Jul 6, 2021
jmlrt pushed a commit to jmlrt/helm-charts that referenced this pull request Jul 6, 2021
jmlrt pushed a commit to nflaig/helm-charts that referenced this pull request Jul 6, 2021
jmlrt added a commit that referenced this pull request Jul 6, 2021
…1278)

Co-authored-by: DilasserT <49373066+DilasserT@users.noreply.github.com>
jmlrt added a commit that referenced this pull request Jul 7, 2021
…1280)

Co-authored-by: DilasserT <49373066+DilasserT@users.noreply.github.com>
jmlrt added a commit that referenced this pull request Jul 7, 2021
…1279)

Co-authored-by: DilasserT <49373066+DilasserT@users.noreply.github.com>
This was referenced Jul 8, 2021
@jmlrt jmlrt mentioned this pull request Mar 8, 2022
@jmlrt jmlrt mentioned this pull request Apr 21, 2022
This was referenced Sep 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants