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

New Labels Request - sampler: aws:xray, propagator: aws:xray, and id-generator: aws:xray #1991

Open
srprash opened this issue Aug 2, 2024 · 2 comments
Assignees

Comments

@srprash
Copy link
Contributor

srprash commented Aug 2, 2024

Hello, I'm reaching out on behalf of the AWS X-Ray team. We would like to request the addition of new labels to this repository: sampler: aws:xray, propagator: aws:xray, and id-generator: aws:xray. These labels would be used for PRs and issues pertaining to the X-Ray Sampler, X-Ray Propagator, and X-Ray ID Generator. Ideally, there would be automation that adds these labels to PRs touching X-Ray component code - we would be happy to help with this if some guidance was provided.

Our goal is to have labelling across open-telemetry repositories so that we can find PRs and issues related to the X-Ray components and give faster support to the OTEL community.

Thank you.

@cijothomas
Copy link
Member

I tried #1994 and it automatically created sampler label

Can you check and let us know what is missing?

@CodeBlanch
Copy link
Member

@srprash

We have some automation in place. When issues are created we'll add labels based on the component the user selects. For PRs we'll add labels based on files being changed.

For AWS at the moment we have...

  • comp:instrumentation.aws
  • comp:extensions.aws
  • comp:sampler.aws
  • comp:resources.aws

When issues are created, based on the component, it looks like the AWS sampler will be tagged as comp:sampler.aws. The propagator and id-generator will both be comp:extensions.aws because they are in the same project.

For issue creation, if you want more granularity, you could create a dedicated template for AWS-related issues. Then you could modify the add-labels script to look for AWS fields from your form and apply tags for what it finds.

For PR tagging. First, do you really need that? I would expect users to open more issues than PRs. Might not be worth the effort. But if you really want it, you can modify that same script where the PR logic is to look at specific AWS paths and apply whatever tags you want.

Happy to create tags for you if you decide to move forward with either plan.

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

No branches or pull requests

3 participants