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

Publish NPM package for deployment #270

Open
wants to merge 59 commits into
base: main
Choose a base branch
from
Open

Publish NPM package for deployment #270

wants to merge 59 commits into from

Conversation

brainstorm
Copy link
Member

Fixes #269

brainstorm and others added 30 commits October 11, 2024 16:25
@brainstorm
First pass at stack to construct conversion
00b3b3f
@brainstorm
No stack vars/refs in construct
96b7f25
@brainstorm
Reasonable CDK defaults for region not dependant on Stack
8939914
@brainstorm
Must be semver, otherwise:
b4ef35e 
% npm install $HOME/dev/umccr/htsget-rs/deploy/
npm error Invalid Version: 1.0
@brainstorm
Doesn't synth without this
78e61d2
@mmalenic
deploy: allow experimental flag to be passed on CDK build code
799fd1a
@mmalenic
feat: allow retrieving c4gh keys from secrets manager
0b79f79
@brainstorm
Fix imports/paths
a62cec2
@mmalenic
feat(config): allow retrieving secret from binary or string
5ebf264
@mmalenic
docs(config): add section on using Secrets Manager to fetch keys
9e5393b
@mmalenic
build(deploy): add options to deploy C4GH keys to secrets manager
331f258
@mmalenic
fix(storage): ensure c4gh data is read after determining the header size
a3c4f77
@mmalenic
refactor(search): remove unnecessary Arcs and unwraps in test code
98b7f84
@mmalenic
Merge pull request #271 from umccr/fix/experimental-deploy
d6ec3b8 
fix: c4gh deploy
@github-actions
chore: release
fb92e89
@brainstorm
Merge pull request #272 from umccr/release-plz-2024-10-16T22-45-42Z
5bd8036 
chore: release
@brainstorm
Cdk synths, fixed issuer making as many optional settings as possible…
0526ae0 
… for easy deploy (reasonable defaults)
@brainstorm
Add app to the test stack
31171c1
@brainstorm @mmalenic
Audience is mandatory and we are picking a diffent subdomain.
c12bb7e 
Removing unnecessary BAM symlinks under crypt4gh data.

Co-authored-by: Marko Malenic <[email protected]>
@mmalenic
fix(deploy): update settings to allow fetching both C4GH and regular …
a5c4105 
…files
@mmalenic
Merge pull request #273 from umccr/fix/deploy-c4gh-settings
18f33a8 
fix(deploy): allow fetching C4GH and regular files
@brainstorm
We have a dev.umccr.org hosted zone already, so don't create an extra…
49f94bd 
… one... this setting should probably be called 'usePreExistingHostedZone' I reckon?
@brainstorm
Add changes from #273
aba97f8
@mmalenic
feat(deploy): add ability to specify Secrets Manager policies separat…
34aef64 
…e to copying local keys
@brainstorm
First pass at stack to construct conversion
f581af7
@brainstorm
No stack vars/refs in construct
ee92a74
@brainstorm
Must be semver, otherwise:
f94e200 
% npm install $HOME/dev/umccr/htsget-rs/deploy/
npm error Invalid Version: 1.0
@brainstorm
Doesn't synth without this
96daf81
@brainstorm
Fix imports/paths
ccd3128
@brainstorm
Cdk synths, fixed issuer making as many optional settings as possible…
1df1ce5 
… for easy deploy (reasonable defaults)
brainstorm and others added 29 commits October 22, 2024 12:42
@brainstorm
Add app to the test stack
5bb656e
@brainstorm @mmalenic
Audience is mandatory and we are picking a diffent subdomain.
d76d915 
Removing unnecessary BAM symlinks under crypt4gh data.

Co-authored-by: Marko Malenic <[email protected]>
@brainstorm
We have a dev.umccr.org hosted zone already, so don't create an extra…
11852c7 
… one... this setting should probably be called 'usePreExistingHostedZone' I reckon?
@brainstorm
Add changes from #273
d19af5d
@brainstorm
copyExampleKeys and jwtAudience (which is just cognito's client id)
89ad899
@brainstorm
Merge pull request #275 from umccr/feat/deploy-secrets-arns
3ed81b4 
feat(deploy): secrets manager policies
@brainstorm
Rebase against main, do not copyExampleKeys yet...
8ce75f5
@brainstorm
Merge branch 'npmjs_deploy_publishing' of github.com:umccr/htsget-rs …
6d5a8af 
…into npmjs_deploy_publishing
@brainstorm
Remove duplicate config keys on settings
51e4c29
@brainstorm
First pass at stack to construct conversion
c82e59c
@brainstorm
No stack vars/refs in construct
4c91958
@brainstorm
Must be semver, otherwise:
5b315d9 
% npm install $HOME/dev/umccr/htsget-rs/deploy/
npm error Invalid Version: 1.0
@brainstorm
Doesn't synth without this
7e8f164
@brainstorm
Fix imports/paths
d57d1b0
@brainstorm
Cdk synths, fixed issuer making as many optional settings as possible…
094e47d 
… for easy deploy (reasonable defaults)
@brainstorm
Add app to the test stack
0aa42ab
@brainstorm @mmalenic
Audience is mandatory and we are picking a diffent subdomain.
73a64d5 
Removing unnecessary BAM symlinks under crypt4gh data.

Co-authored-by: Marko Malenic <[email protected]>
@brainstorm
We have a dev.umccr.org hosted zone already, so don't create an extra…
07874af 
… one... this setting should probably be called 'usePreExistingHostedZone' I reckon?
@brainstorm
Add changes from #273
5ff9b5e
@brainstorm
copyExampleKeys and jwtAudience (which is just cognito's client id)
2b0aafd
@brainstorm
Rebase against main, do not copyExampleKeys yet...
e35c5dd
@brainstorm
First pass at stack to construct conversion
e13d05c
@brainstorm
Doesn't synth without this
c4abda4
@brainstorm
Fix imports/paths
78104d0
@brainstorm
Cdk synths, fixed issuer making as many optional settings as possible…
4909b63 
… for easy deploy (reasonable defaults)
@brainstorm
Add app to the test stack
a63c709
@brainstorm
Add changes from #273
0f36759
@brainstorm
Remove duplicate config keys on settings
57b20c0
@brainstorm
Merge branch 'npmjs_deploy_publishing' of github.com:umccr/htsget-rs …
ba061a9 
…into npmjs_deploy_publishing
@brainstorm brainstorm marked this pull request as ready for review October 22, 2024 02:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feat: publish htsget deployment to NPM
2 participants
@brainstorm @mmalenic