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

move IaC to broker repo #403

Merged
merged 28 commits into from
Apr 29, 2024
Merged

move IaC to broker repo #403

merged 28 commits into from
Apr 29, 2024

Conversation

Andreass2
Copy link
Collaborator

Description

Move IaC to the broker repo

Related Issue(s)

Verification

  • Your code builds clean without any errors or warnings
  • Manual testing done (required)
  • Relevant automated test added (if you find this hard, leave it and we'll help out)
  • All tests run green

Documentation

  • User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

Hammerbeck and others added 25 commits April 24, 2024 09:19
move IaC to broker repo
e09f8ee
remove example files
3596483
@Ceredron
Fix bug
6afa000
@Ceredron
Container app job name can be max 32 characters long
4c8192c
@Ceredron
Also can get too long
4a551ce
@Ceredron
Remove dead code
d59892e
@Ceredron
Consistent file name
2b24980
@Ceredron
Fix migration bicep
dd6eebb
@Ceredron
Fixes after testing
0f5b7ba
@Ceredron
Shorter name
2fb2290
@Ceredron
Need to expose name from bicep for use in pipeline
88af7d2
@Ceredron
Use environment for azure cli
2040ef4
@Ceredron
Use app version env variable to find container job instead of name
82edbb5
@Ceredron
Fix
fa6c353
@Ceredron
Select first in array
02e064a
@Ceredron
Fix
816b076
@Ceredron
There seems to be a delay after Postgres password has been updated du…
e09507f 
…ring a bicep deployment before said password can be used. Therefore uses existing broker password if possible.
@Ceredron
Fix
abd0b88
@Ceredron
Failed to use keyvault in same deployment that creates it
0d3e8a6
@Ceredron
Generate password without characters that become troublesome in an ur…
e74ca65 
…l. Easier to change generation than do html encode/decode in bicep
@Ceredron
We do not use source keys because we use Github secrets as our config…
5740b94 
…uration source
@Ceredron
Added logic for initial deploy
ee85ee4
@Ceredron
This works
143082a
@Ceredron
Required parameter is required
53149b3
@Ceredron
Fixed pwd generator to not use chars not supported by postgres (and a…
e6cae16 
…lso compatible with jdbc url)
@Andreass2 Andreass2 merged commit e9a0cf0 into main Apr 29, 2024
@Andreass2 Andreass2 deleted the feat/infrastructure branch April 29, 2024 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ceredron Ceredron Ceredron approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Andreass2 @Ceredron