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

fix cloud function modul #105

Closed
wants to merge 10 commits into from
Closed

fix cloud function modul #105

wants to merge 10 commits into from

Conversation

JoachimHaa
Copy link
Contributor

@JoachimHaa JoachimHaa commented Sep 25, 2024

Har byttet fra data archive_files til resource archive_files for å fikse at cloud functions filer kun blir zippet i plan steget og ikke apply steget. Resource achive_files er tagget som deprecated men som diskutert i denne issue tråden skal den mest sannsynlig ikke fjernes med det første.

Hvordan vi skal løses dette på en permanent basis er litt usikkert.

md5 hash av filer lastet opp til GCS bucket endrer md5 hash mellom plan og apply planen i run terraform git action til SKIP
@JoachimHaa JoachimHaa requested a review from a team as a code owner September 25, 2024 11:42
selv om resource blokken er tagget som archived, virker det som at det er den som har riktig funksjonalitet for å zippe filen under apply steget
La til terraform_created_zip i output path for zip så det blir klarer at dette er noe som opprettes av terraform

La til navn på function på output path i tilfelle flere modul kall skjer i samme terraform script
@JoachimHaa JoachimHaa changed the title md5 hash fix fix cloud function modul Sep 30, 2024
@JoachimHaa
Copy link
Contributor Author

Lukker denne pull requesten da vi ikke lenger kommer til å prioritere cloud functions for å laste opp data, i tillegg til at team Hydris ikke lengre bruker cloud functions som var grunnen for at denne fiksen ble testet.

@JoachimHaa JoachimHaa closed this Dec 3, 2024
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