Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

Test added for CFn-Opensearch:Domain#6943

Merged
pinzon merged 4 commits intomasterfrom
fix-cfn-opensearch-domain
Oct 4, 2022
Merged

Test added for CFn-Opensearch:Domain#6943
pinzon merged 4 commits intomasterfrom
fix-cfn-opensearch-domain

Conversation

@pinzon
Copy link
Copy Markdown
Member

@pinzon pinzon commented Sep 27, 2022

Test to validate Opensearch domain deployment through Cfn. Related to #6830

@pinzon pinzon temporarily deployed to localstack-ext-tests September 27, 2022 16:02 Inactive
@coveralls
Copy link
Copy Markdown

coveralls commented Sep 27, 2022

Coverage Status

Coverage decreased (-0.003%) to 79.107% when pulling 914d621 on fix-cfn-opensearch-domain into 52b1831 on master.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 27, 2022

LocalStack integration with Pro

       3 files  ±  0         3 suites  ±0   1h 13m 14s ⏱️ - 2m 50s
1 350 tests +11  1 219 ✔️ +3  131 💤 +8  0 ±0 
1 862 runs  +  9  1 587 ✔️ +3  275 💤 +6  0 ±0 

Results for commit 914d621. ± Comparison against base commit 52b1831.

♻️ This comment has been updated with latest results.

@pinzon pinzon force-pushed the fix-cfn-opensearch-domain branch from d1e1d13 to 44fc39f Compare September 28, 2022 13:38
@pinzon pinzon temporarily deployed to localstack-ext-tests September 28, 2022 13:39 Inactive
@pinzon pinzon marked this pull request as ready for review September 28, 2022 15:57
@pinzon pinzon force-pushed the fix-cfn-opensearch-domain branch from 44fc39f to 50c8bab Compare October 3, 2022 13:26
@pinzon pinzon temporarily deployed to localstack-ext-tests October 3, 2022 13:26 Inactive
@pinzon pinzon force-pushed the fix-cfn-opensearch-domain branch from 50c8bab to 85331c1 Compare October 4, 2022 13:49
@pinzon pinzon temporarily deployed to localstack-ext-tests October 4, 2022 13:49 Inactive
Copy link
Copy Markdown
Member

@dominikschubert dominikschubert left a comment

Choose a reason for hiding this comment

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

Minor nit regarding the pytest marker, otherwise LGTM 👍

Co-authored-by: Dominik Schubert <dominik.schubert91@gmail.com>
@pinzon pinzon temporarily deployed to localstack-ext-tests October 4, 2022 18:21 Inactive
@pinzon pinzon temporarily deployed to localstack-ext-tests October 4, 2022 19:10 Inactive
@pinzon pinzon merged commit 6af5dcf into master Oct 4, 2022
@pinzon pinzon deleted the fix-cfn-opensearch-domain branch October 4, 2022 20:44
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.

3 participants