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

Host Service in Container Apps through IAC #1301

Closed
Tracked by #1041
arealmaas opened this issue Oct 16, 2024 · 0 comments
Closed
Tracked by #1041

Host Service in Container Apps through IAC #1301

arealmaas opened this issue Oct 16, 2024 · 0 comments
Assignees
@arealmaas

Comments

@arealmaas
Copy link
Collaborator

No description provided.

@arealmaas arealmaas mentioned this issue Oct 16, 2024
Closed
@arealmaas arealmaas self-assigned this Oct 16, 2024
@arealmaas arealmaas moved this from New issues to Doing in Dialogporten / Arbeidsflate Oct 16, 2024
@arealmaas arealmaas mentioned this issue Oct 16, 2024
Merged
4 tasks
arealmaas added a commit that referenced this issue Oct 16, 2024
@arealmaas
feat(service): deploy application in container apps (#1303)
a309044 
<!--- Provide a general summary of your changes in the Title above -->

## Description

<!--- Describe your changes in detail -->

- Added Service in all environments in container app. Using webapi for
now until Service is ready
- Added user assigned identity instead of using the created managed
identity created by the container app to avoid potential race
conditions. This should be required eventually for all container apps.
Making it optional for now.

## Related Issue(s)

- #1301

## 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)

## Documentation

- [ ] Documentation is updated (either in `docs`-directory, Altinnpedia
or a separate linked PR in
[altinn-studio-docs.](https://github.com/Altinn/altinn-studio-docs), if
applicable)


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **New Features**
- Introduced a new infrastructure configuration for deploying a
container application in Azure.
- Added support for multiple environments (production, staging, test)
with dynamic parameter management.
- Enhanced identity management for the container app, allowing for
user-assigned identities.

- **Bug Fixes**
- Improved health probe configurations for better application
monitoring.

- **Documentation**
- Updated workflow to include deployment capabilities for the new
service component.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
@arealmaas arealmaas mentioned this issue Oct 16, 2024
Merged
4 tasks
arealmaas added a commit that referenced this issue Oct 16, 2024
@arealmaas
chore(service): use correct docker image (#1308)
462f908 
<!--- Provide a general summary of your changes in the Title above -->

## Description

<!--- Describe your changes in detail -->

Using the healthz endpoint before the proper service is deployed

## Related Issue(s)

- #1301 

## 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)

## Documentation

- [ ] Documentation is updated (either in `docs`-directory, Altinnpedia
or a separate linked PR in
[altinn-studio-docs.](https://github.com/Altinn/altinn-studio-docs), if
applicable)
@arealmaas arealmaas closed this as completed by moving to Done in Dialogporten / Arbeidsflate Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arealmaas arealmaas

Labels
None yet
Projects
Dialogporten / Arbeidsflate
Archived in project
Milestone
No milestone
Development

No branches or pull requests
1 participant
@arealmaas