Skip to content

Migrate GCP resource detector (opentelemetry-resourcedetector-gcp) to contrib#4749

Merged
xrmx merged 12 commits into
open-telemetry:mainfrom
DylanRussell:donate_resource_detector
Jul 15, 2026
Merged

Migrate GCP resource detector (opentelemetry-resourcedetector-gcp) to contrib#4749
xrmx merged 12 commits into
open-telemetry:mainfrom
DylanRussell:donate_resource_detector

Conversation

@DylanRussell

Copy link
Copy Markdown
Contributor

Description

Move the GCP resource detector to this repo.

We'll have to transfer ownership of this package name in PyPi..

I started the version in this repo at __version__ = "1.13.0.dev" because the latest released version on PyPi is 1.12.0a0

Type of change

  • [x ] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Unit tests

Does This PR Require a Core Repo Change?

  • [ x] No.

Checklist:

See contributing.md for styleguide, changelog guidelines, and more.

  • [x ] Followed the style guidelines of this project
  • [ x] Changelogs have been updated
  • [ x] Unit tests have been added
  • [x ] Documentation has been updated

@DylanRussell
DylanRussell requested a review from a team as a code owner June 26, 2026 20:59
@DylanRussell DylanRussell moved this to Ready for review in Python PR digest Jul 8, 2026

@xrmx xrmx left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this a move of the current one or does it have some changes? I am grateful for the rename of the entry point :)

Comment thread resource/opentelemetry-resourcedetector-gcp/README.rst Outdated
@xrmx xrmx added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jul 13, 2026
@DylanRussell

Copy link
Copy Markdown
Contributor Author

Just a move of the existing one.. I had to make some minor changes to fix the pre commit checks (changed the license, run precommit)

Comment thread resource/opentelemetry-resourcedetector-gcp/README.rst Outdated

@xrmx xrmx left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Small nits in the README but LGTM

@emdneto emdneto left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you add the package opentelemetry-resourcedetector-gcp in here? https://github.com/open-telemetry/opentelemetry-python-contrib/blob/main/.github/workflows/package-prepare-release.yml#L10 and also add in the others package-*.yml workflows?

Comment thread resource/opentelemetry-resourcedetector-gcp/requirements.oldest.txt
@DylanRussell

Copy link
Copy Markdown
Contributor Author

I think the workflow stuff is best done in a separate PR after we move the package name on PyPi over

@xrmx
xrmx added this pull request to the merge queue Jul 15, 2026
Merged via the queue into open-telemetry:main with commit aaf3caf Jul 15, 2026
774 checks passed
@github-project-automation github-project-automation Bot moved this from Ready for review to Done in Python PR digest Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants