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.

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

PierianDx Production Fixes #659

Merged
merged 11 commits into from
Nov 6, 2024
Merged

Conversation

alexiswl
Copy link
Member

@alexiswl alexiswl commented Nov 6, 2024

  • Prod PierianDx bucket requires KMS encryption
  • Can't use LATEST version for invocation of 'collectPierianDxAccessToken'
    • Use '*' on function name followed by NagSuppression
  • Fix SSM parameter determining the path of the PierianDx bucket
  • Differences between requesting_physicians_x_name and requetsing_physician_x_name was causing default name to be used (Sean)
  • Wrong JSON path used in SFN to collect the panel version
  • Report ID stored in database as an integer not as a string
  • Fixed GET for linked libraries which had an incorrect JSON path when pushing a workflow run state change event
  • Fixed analysis path generation for data outputs

P.S have already pushed this to OrcaBus Prod manually

@alexiswl alexiswl requested a review from victorskl November 6, 2024 06:46
@alexiswl alexiswl self-assigned this Nov 6, 2024
@victorskl victorskl added the fix label Nov 6, 2024
@victorskl victorskl linked an issue Nov 6, 2024 that may be closed by this pull request
Copy link
Member

@victorskl victorskl left a comment

Choose a reason for hiding this comment

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

LGTM

@alexiswl alexiswl merged commit 626de1a into main Nov 6, 2024
6 checks passed
@alexiswl alexiswl deleted the bump-pieriandx-generate-outputs-timer branch November 6, 2024 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add PierianDx Pipeline Manager as a service
2 participants