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

OpenSearch: Move Tagging into Provider Methods#13624

Merged
aidehn merged 4 commits intomainfrom
aidehn/feat/opensearch-rgta-integration
Feb 11, 2026
Merged

OpenSearch: Move Tagging into Provider Methods#13624
aidehn merged 4 commits intomainfrom
aidehn/feat/opensearch-rgta-integration

Conversation

@aidehn
Copy link
Contributor

@aidehn aidehn commented Jan 16, 2026 •
edited
Loading

Changes

  • Moves the existing tagging functionality into provider methods so that it can be overwritten in Pro.
  • This doesn't change any existing functionality.

@github-actions
Copy link

github-actions bot commented Jan 16, 2026 •
edited
Loading

Test Results - Preflight, Unit

23 140 tests  ±0   21 276 ✅ ±0   6m 8s ⏱️ -1s
     1 suites ±0    1 864 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 872788e. ± Comparison against base commit 5562582.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jan 16, 2026 •
edited
Loading

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   2m 59s ⏱️ -2s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 872788e. ± Comparison against base commit 5562582.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jan 16, 2026 •
edited
Loading

Test Results (amd64) - Integration, Bootstrap

 5 files  ±    0   5 suites  ±0   18m 57s ⏱️ - 2h 19m 46s
73 tests  - 5 551  58 ✅  - 5 045  15 💤  - 506  0 ❌ ±0 
79 runs   - 5 551  58 ✅  - 5 045  21 💤  - 506  0 ❌ ±0 

Results for commit 872788e. ± Comparison against base commit 5562582.

This pull request removes 5551 tests. 
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_lambda_dynamodb
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_opensearch_crud
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_search_books
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_setup
tests.aws.scenario.kinesis_firehose.test_kinesis_firehose.TestKinesisFirehoseScenario ‑ test_kinesis_firehose_s3
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_destination_sns
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_infra
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_prefill_dynamodb_table
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input0-SUCCEEDED]
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input1-SUCCEEDED]
…

♻️ This comment has been updated with latest results.

@aidehn aidehn added this to the 4.14 milestone Jan 23, 2026
@aidehn aidehn force-pushed the aidehn/feat/opensearch-rgta-integration branch from 840b9e5 to 5a069cb Compare January 27, 2026 16:14
@github-actions
Copy link

github-actions bot commented Jan 27, 2026 •
edited
Loading

LocalStack Community integration with Pro

 2 files  ±    0   2 suites  ±0   10m 4s ⏱️ - 1h 49m 29s
49 tests  - 5 154  34 ✅  - 4 818  15 💤  - 336  0 ❌ ±0 
51 runs   - 5 154  34 ✅  - 4 818  17 💤  - 336  0 ❌ ±0 

Results for commit 872788e. ± Comparison against base commit 5562582.

This pull request removes 5154 tests. 
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_lambda_dynamodb
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_opensearch_crud
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_search_books
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_setup
tests.aws.scenario.kinesis_firehose.test_kinesis_firehose.TestKinesisFirehoseScenario ‑ test_kinesis_firehose_s3
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_destination_sns
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_infra
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_prefill_dynamodb_table
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input0-SUCCEEDED]
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input1-SUCCEEDED]
…

♻️ This comment has been updated with latest results.

@aidehn aidehn force-pushed the aidehn/feat/opensearch-rgta-integration branch from 5a069cb to 2fbdf55 Compare February 3, 2026 05:54
@aidehn aidehn added semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Feb 4, 2026
@aidehn aidehn force-pushed the aidehn/feat/opensearch-rgta-integration branch 3 times, most recently from ed2911e to 62c0c03 Compare February 10, 2026 04:33
@aidehn aidehn marked this pull request as ready for review February 10, 2026 04:36
silv-io
silv-io approved these changes Feb 10, 2026
View reviewed changes
Copy link
Member

@silv-io silv-io left a comment

Choose a reason for hiding this comment

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

LGTM!

@aidehn aidehn force-pushed the aidehn/feat/opensearch-rgta-integration branch from 7ead223 to 872788e Compare February 11, 2026 05:18
@aidehn aidehn merged commit 8cda50b into main Feb 11, 2026
61 of 62 checks passed
@aidehn aidehn deleted the aidehn/feat/opensearch-rgta-integration branch February 11, 2026 07:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@alexrashed alexrashed Awaiting requested review from alexrashed alexrashed is a code owner

1 more reviewer

@silv-io silv-io silv-io approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases

Projects

None yet

Milestone

4.14

Development

Successfully merging this pull request may close these issues.

2 participants

@aidehn @silv-io
X Tutup