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

add origin_referrer_url and origin_url to the process attribute #1517

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

AsuNa-jp
Copy link

@AsuNa-jp AsuNa-jp commented Oct 25, 2024

Changes

This PR adds the following attributes. (Updated Dec/25/2024)

  • process.executable.origin_referrer_url
  • process.executable.origin_url

Details

Generally, a process is generated from an executable file. Therefore, the process model naturally includes attributes containing information about the originating executable file. In this PR, we are adding the following two attributes to further extend the originating executable file details.

  • process.executable.origin_referrer_url
  • process.executable.origin_url

The name of these attributes and where to add it were decided after consulting with @joe-desimone and @trisch-me.

The meaning of each field is explained in detail in the PR below.

Merge requirement checklist

Copy link

github-actions bot commented Dec 4, 2024

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Dec 4, 2024
@AsuNa-jp AsuNa-jp changed the title add origin_referrer_url, origin_url and zone_identifier to the process attribute add origin_referrer_url, origin_url to the process attribute Dec 5, 2024
@AsuNa-jp AsuNa-jp requested a review from a team as a code owner December 5, 2024 05:28
@AsuNa-jp AsuNa-jp changed the title add origin_referrer_url, origin_url to the process attribute add origin_referrer_url and origin_url to the process attribute Dec 5, 2024
@braydonk
Copy link
Contributor

braydonk commented Dec 5, 2024

Since this is a property of the process's executable, and not of a process itself, I think it makes more sense for these attributes to be process.executable.origin_referrer_url and process.executable.origin_url.

@github-actions github-actions bot removed the Stale label Dec 6, 2024
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Dec 22, 2024
@trisch-me
Copy link
Contributor

@AsuNa-jp could you update this PR and address comments? thanks

@github-actions github-actions bot removed the Stale label Dec 24, 2024
@AsuNa-jp
Copy link
Author

Hi @braydonk @trisch-me
Sorry for the late reply and thank you for the feedback!
As per your suggestion, I have added origin_referrer_url and origin_url under the executable as shown below. If you have any other feedback, please feel free to share it.

  • process.origin_referrer_url to process.executable.origin_referrer_url
  • process.origin_url to process.executable.origin_url

6e52941

Copy link
Contributor

@braydonk braydonk left a comment

Choose a reason for hiding this comment

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

LGTM

brief: >
The URL of the webpage that linked to the process's executable file.
note: >
This information comes from metadata or alternate data streams linked to the process's executable file.
Copy link
Contributor

Choose a reason for hiding this comment

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

Non-blocking, can be merged without resolving this unless there's a good answer.


@open-telemetry/specs-semconv-maintainers is there a better way to specify a note like this in the YAML? This results in two separate identical notes being generated in the markdown. A way to make one note about this that could be linked by both attributes would be nice, but as far as I know there isn't a way to do that right?

Copy link
Contributor

Choose a reason for hiding this comment

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

there is no way to do this currently afaik, because we are not linking notes, it happens automatically

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Needs More Approval
Development

Successfully merging this pull request may close these issues.

3 participants