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

change/from_vault_to_secretmanager #3654

Open
wants to merge 110 commits into
base: master
Choose a base branch
from

Conversation

rfc3092
Copy link
Contributor

@rfc3092 rfc3092 commented Oct 17, 2024

Erstatter bruk av Vault med GCP Secret Manager.

Gjenværende i /apps med Vault:

  • brreg-stub
  • testnorge-statisk-data-forvalter
  • udi-stub

Felles for disse er at de deployer til dev-fss og har en onprem PSQL. Ingenting trenger lenger Vault for å kjøre lokalt.

…d given proper login with gcloud for current project.
- Normalizes config file name to application-test.yml.
# Conflicts:
#	examples/reactive-rest-example/README.md
#	examples/reactive-rest-example/build.gradle
#	examples/reactive-rest-example/settings.gradle
#	examples/reactive-rest-example/src/main/resources/application-local.yml
#	examples/reactive-rest-example/src/main/resources/application.yml
- Removed unnecessary hibernate.dialect config.
- H2 didn't work with current migrations - switched to local PSQL.
… IBM MQ is used instead.

- Reenabling test.
- Config renamed from .yaml to yml.
- Updated README.md in /apps.
@rfc3092 rfc3092 marked this pull request as ready for review November 26, 2024 16:34
@rfc3092 rfc3092 requested a review from a team as a code owner November 26, 2024 16:34
@rfc3092
Copy link
Contributor Author

rfc3092 commented Nov 26, 2024

@rfc3092
Copy link
Contributor Author

rfc3092 commented Nov 28, 2024

1 similar comment
@rfc3092
Copy link
Contributor Author

rfc3092 commented Nov 29, 2024

# Conflicts:
#	apps/sykemelding-api/src/main/java/no/nav/registre/testnorge/sykemelding/config/LocalVaultConfig.java
#	apps/sykemelding-api/src/main/resources/application.yml
#	apps/synt-sykemelding-api/src/main/java/no/nav/testnav/apps/syntsykemeldingapi/config/LocalVaultConfig.java
#	apps/synt-sykemelding-api/src/main/resources/application-dev.yml
#	apps/synt-sykemelding-api/src/main/resources/application.yml
Copy link
Contributor

@krharum krharum left a comment

Choose a reason for hiding this comment

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

Supert! 💯

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.

2 participants