Skip to content
This repository has been archived by the owner on Jun 25, 2024. It is now read-only.

feat(INT-5917): add cloud function storage bucket relationship #554

Merged
merged 1 commit into from
Nov 9, 2022

Conversation

gastonyelmini
Copy link
Contributor

Context

Adds Google Cloud Functions Source Repository Relation.

Captura de Pantalla 2022-11-04 a la(s) 10 51 14

const cloudFunction =
getRawData<cloudfunctions_v1.Schema$CloudFunction>(cloudFunctionEntity);

const storageBucketKey = cloudFunction?.sourceArchiveUrl?.split('/')[2];
Copy link
Contributor Author

Choose a reason for hiding this comment

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

soruceArchivrUrl would be something like gs://[bucket-id]/[file-name].

@i5o
Copy link
Contributor

i5o commented Nov 4, 2022

Looking at the build result it looks like you forgot to add this step to the index test; just a heads up

src/steps/functions/index.ts Outdated Show resolved Hide resolved
@gastonyelmini gastonyelmini force-pushed the INT-5917-cloud-function-relationship branch from 0caa9dc to 301013c Compare November 4, 2022 16:45
@gastonyelmini gastonyelmini marked this pull request as ready for review November 4, 2022 16:45
@gastonyelmini gastonyelmini requested a review from a team as a code owner November 4, 2022 16:45
@gastonyelmini gastonyelmini requested a review from i5o November 4, 2022 16:46
@gastonyelmini gastonyelmini force-pushed the INT-5917-cloud-function-relationship branch from 301013c to 85ccb2b Compare November 4, 2022 20:31
@gastonyelmini gastonyelmini requested a review from i5o November 4, 2022 20:35
i5o
i5o previously approved these changes Nov 5, 2022
Copy link
Contributor

@i5o i5o left a comment

Choose a reason for hiding this comment

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

Looks like you figured it out already :)
Thank you

@gastonyelmini gastonyelmini force-pushed the INT-5917-cloud-function-relationship branch 3 times, most recently from a0879e6 to 87b7d88 Compare November 8, 2022 19:28
ndowmon
ndowmon previously approved these changes Nov 8, 2022
src/steps/functions/constants.ts Outdated Show resolved Hide resolved
src/steps/functions/constants.ts Outdated Show resolved Hide resolved
@gastonyelmini gastonyelmini merged commit 43c93cc into main Nov 9, 2022
@gastonyelmini gastonyelmini deleted the INT-5917-cloud-function-relationship branch November 9, 2022 02:26
@gastonyelmini gastonyelmini added the minor Increment the minor version when merged label Nov 9, 2022
@gastonyelmini gastonyelmini added the release Create a release when this pr is merged label Nov 9, 2022
@gastonyelmini gastonyelmini restored the INT-5917-cloud-function-relationship branch November 9, 2022 12:59
@gastonyelmini gastonyelmini deleted the INT-5917-cloud-function-relationship branch November 9, 2022 13:07
@j1-internal-automation
Copy link
Collaborator

🚀 PR was released in v2.20.0 🚀

@j1-internal-automation j1-internal-automation added the released This issue/pull request has been released. label Nov 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
minor Increment the minor version when merged release Create a release when this pr is merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants