Skip to content

Conversation

ghukill
Copy link
Contributor

@ghukill ghukill commented Sep 4, 2025

Purpose and background context

The primary purpose of this PR is to update the TDA version to 3.2, picking up new read methods.

Additionally, this updates README and the Docker Compose YAML to pin to Opensearch version 2.x to align with our current version in AWS.

How can a reviewer manually see the effects of these changes?

Successful StepFunction run in Dev shows pipeline lambdas using updated TDA version: https://222053980223-fpdparvm.us-east-1.console.aws.amazon.com/states/home?region=us-east-1#/v2/executions/details/arn:aws:states:us-east-1:222053980223:execution:timdex-ingest-dev:ea132241-1c30-4992-9b59-c5ba15a40ab9.

Includes new or updated dependencies?

YES

Changes expectations for external applications?

NO

What are the relevant tickets?

  • None

Why these changes are being introduced:

We formerly documented and used the tag ':latest' for standing up a local
Opensearch instance, but this has the potential to pull a newer version than
we currently have in AWS.

How this addresses that need:
* Pins references in README and Docker Compose YAML to version 2
* Will require updates when we eventually migrate to version 3

Side effects of this change:
* Build Opensearch 2.x locally for testing and development

Relevant ticket(s):
* None
@ghukill ghukill requested a review from a team September 5, 2025 13:02
@coveralls
Copy link

coveralls commented Sep 5, 2025

Pull Request Test Coverage Report for Build 17494822697

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 97.424%

Totals Coverage Status
Change from base Build 16998583164: 0.0%
Covered Lines: 416
Relevant Lines: 427

💛 - Coveralls

@ghukill ghukill merged commit 7d06a88 into main Sep 5, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants