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.

Sign up for GitHub

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

Jump to bottom

[Stack Integration] Fix error text in encryption keys test for alert connectors #110268

Merged
merged 1 commit into from
Aug 26, 2021
Merged

[Stack Integration] Fix error text in encryption keys test for alert connectors #110268

merged 1 commit into from
Aug 26, 2021

Conversation

marius-dr
Copy link
Member

This changes the text we have in the test to the new, more verbose one.
Encryption Rotation encryption key rotation with email connectors without the key used to create it should show a failure callout

@marius-dr marius-dr added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes auto-backport Deprecated - use backport:version if exact versions are needed v7.15.0 v7.16.0 labels Aug 26, 2021
@marius-dr marius-dr requested a review from wayneseymour August 26, 2021 15:37
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@marius-dr marius-dr merged commit 18fe8c4 into master Aug 26, 2021
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Aug 26, 2021
@marius-dr @kibanamachine
Fix for changed failed alert text (elastic#110268)
0f2da99
@kibanamachine kibanamachine mentioned this pull request Aug 26, 2021
Merged
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Aug 26, 2021
@marius-dr @kibanamachine
Fix for changed failed alert text (elastic#110268)
b52fcfa
@kibanamachine kibanamachine mentioned this pull request Aug 26, 2021
Merged
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
✅ 7.15
✅ 7.x

The backport PRs will be merged automatically after passing CI.

@marius-dr marius-dr deleted the marius-dr-patch-encryption-text branch August 26, 2021 21:16
kibanamachine added a commit that referenced this pull request Aug 26, 2021
@kibanamachine @marius-dr
Fix for changed failed alert text (#110268) (#110344)
fa310c2 
Co-authored-by: Marius Dragomir <marius-dr@users.noreply.github.com>
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

@kibanamachine kibanamachine added backport missing Added to PRs automatically when the are determined to be missing a backport. and removed backport missing Added to PRs automatically when the are determined to be missing a backport. labels Aug 27, 2021
kibanamachine added a commit that referenced this pull request Aug 30, 2021
@kibanamachine @marius-dr
Fix for changed failed alert text (#110268) (#110343)
8dc8626 
Co-authored-by: Marius Dragomir <marius-dr@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wayneseymour wayneseymour wayneseymour approved these changes

Assignees
No one assigned
Labels
auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes test-stack-integration v7.15.0 v7.16.0 v8.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@marius-dr @kibanamachine @wayneseymour