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

Implement Cloud Run and Cloud Functions faas resource detection #346

Merged
merged 1 commit into from
Jul 31, 2024

Conversation

aabmass
Copy link
Collaborator

@aabmass aabmass commented Jul 23, 2024

Copied from JS/Go implementations. Subsequent PRs for e2e tests:

@aabmass aabmass added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Jul 29, 2024
Copy link

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Jul 30, 2024
@aabmass aabmass force-pushed the resource-faas-2 branch 4 times, most recently from 2222d84 to 8f8a84f Compare July 31, 2024 02:18
@aabmass aabmass changed the title Implement faas resource detection Implement Cloud Run and Cloud Functions faas resource detection Jul 31, 2024
@aabmass aabmass marked this pull request as ready for review July 31, 2024 02:39
@aabmass aabmass requested a review from a team as a code owner July 31, 2024 02:39
@aabmass
Copy link
Collaborator Author

aabmass commented Jul 31, 2024

Please ignore "ops-python-e2e-cloud-functions-gen2 (opentelemetry-ops-e2e)" failure, it is implemented in the following PRs

@aabmass aabmass merged commit 33c333f into GoogleCloudPlatform:main Jul 31, 2024
19 of 20 checks passed
@aabmass aabmass deleted the resource-faas-2 branch July 31, 2024 15:27
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.

3 participants