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

[elasticsearch] Adding testing for security context #203

Merged
merged 2 commits into from
Jul 10, 2019
Merged

Conversation

Crazybus
Copy link
Contributor

@Crazybus Crazybus commented Jul 5, 2019

Template tests for the changes from #171

  • 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

Template tests for the changes from #171
The `for i in {1..5}` was working for my shell locally but in CI it was
only retrying once.
Copy link
Contributor

@tylerjl tylerjl left a comment

Choose a reason for hiding this comment

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

LGTM (and nice portable shell changes to Makefile) 👍

@Crazybus Crazybus merged commit baf85cc into master Jul 10, 2019
@Crazybus Crazybus deleted the securitest branch July 10, 2019 14:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants