-
Notifications
You must be signed in to change notification settings - Fork 197
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
Update enduser domain and add enduser.pseudo.id
#1456
Merged
lmolkova
merged 56 commits into
open-telemetry:main
from
heyams:heya/add-enduser-namespace
Feb 3, 2025
Merged
Changes from all commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
3e1655f
Add authentication to enduser namespace
heyams 6f858a2
Fix yamllint
heyams 07cb1eb
Fix weaver
heyams ee0970f
Update markdowns
heyams 38d8687
Generate gh issue
heyams b1cdc13
Fix policies-check and compatibility
heyams 068a822
Update
heyams e05d7a3
Merge branch 'main' into heya/add-enduser-namespace
heyams c17bec6
Update changelog
heyams 0fa3ffc
Merge branch 'heya/add-enduser-namespace' of https://github.com/heyam…
heyams d0e26d5
Update
heyams 614a52e
Fix yamllint
heyams a499237
Fix
heyams 5bfbfe2
Fix trailing spaces
heyams 5f883d0
Use 'make generate-gh-issue-tempaltes
heyams 13919ed
Update
heyams 272ded3
Merge remote-tracking branch 'upstream/main' into heya/add-enduser-na…
heyams 5c0d6d3
Update
heyams ff85999
Fix identity not found
heyams d626e6c
Update
heyams cb32498
Fix yamllint
heyams 1c53711
Fix backward compat
heyams 51d0c99
Rename enduser.id to enduser.pseudo.id
heyams 5d10c08
Merge remote-tracking branch 'upstream/main' into heya/add-enduser-na…
heyams 949bae8
Update md
heyams 9778890
Update
heyams 23bb6fe
Update attributes.md
heyams f0623df
Address comments
heyams 435fc1b
Make update
heyams ca87ccc
Fix yamllint
heyams 2b09f28
Merge remote-tracking branch 'upstream/main' into heya/add-enduser-na…
heyams 2ccbfe3
Update
heyams 10b2e0c
Fix yamllint
heyams 8c89be8
Address comments
heyams a96ebb3
Update
heyams e076c4e
Update
heyams 33decae
Merge remote-tracking branch 'upstream/main' into heya/add-enduser-na…
heyams a5a9229
Comment
heyams 109cf5f
Comment
heyams e2acca0
Revert a newline as it was introduced during merging conflicts
heyams 27753b3
Address comments
heyams 01b9ac8
Update make
heyams 3266909
Fix warning notes
heyams ac1de4d
Fix areas-dropdown
heyams 018e8bd
Fix yamllint
heyams 5448e29
Merge branch 'main' into heya/add-enduser-namespace
heyams f6de459
Fix warning rendering
heyams 2eb1c20
Merge branch 'heya/add-enduser-namespace' of https://github.com/heyam…
heyams 51b018f
Update
heyams 9f16bab
Remove
heyams 3a6f26b
Revert deprecated message for enduser.role
heyams f06b008
Remove 'general identity attributes'
heyams 894b9b2
Fix markdownlint
heyams 71b1f68
Delete enduser/common.yaml
heyams de64730
Comments
heyams 19ba4d2
Merge branch 'main' into heya/add-enduser-namespace
lmolkova File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
# Use this changelog template to create an entry for release notes. | ||
# | ||
# If your change doesn't affect end users you should instead start | ||
# your pull request title with [chore] or use the "Skip Changelog" label. | ||
|
||
# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix' | ||
change_type: enhancement | ||
|
||
# The name of the area of concern in the attributes-registry, (e.g. http, cloud, db) | ||
component: enduser | ||
|
||
# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`). | ||
note: Undeprecate 'enduser.id' and introduce new attribute `enduser.pseudo.id` | ||
|
||
# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists. | ||
# The values here must be integers. | ||
issues: [1104] | ||
|
||
# (Optional) One or more lines of additional information to render under the primary note. | ||
# These lines will be padded with 2 spaces and then inserted directly into the document. | ||
# Use pipe (|) for multiline entries. | ||
subtext: | | ||
The new attribute `enduser.pseudo.id` is intended to provide a unique identifier of a pseudonymous enduser. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
groups: | ||
- id: registry.enduser | ||
type: attribute_group | ||
display_name: End User Attributes | ||
brief: > | ||
Describes the end user. | ||
attributes: | ||
- id: enduser.id | ||
heyams marked this conversation as resolved.
Show resolved
Hide resolved
|
||
type: string | ||
brief: "Unique identifier of an end user in the system. It maybe a username, email address, or other identifier." | ||
note: | | ||
Unique identifier of an end user in the system. | ||
|
||
> [!Warning] | ||
> This field contains sensitive (PII) information. | ||
examples: [ 'username' ] | ||
heyams marked this conversation as resolved.
Show resolved
Hide resolved
|
||
stability: development | ||
- id: enduser.pseudo.id | ||
type: string | ||
stability: development | ||
brief: > | ||
Pseudonymous identifier of an end user. This identifier should be a random value that is not directly linked or associated with the end user's actual identity. | ||
note: | | ||
Pseudonymous identifier of an end user. | ||
|
||
> [!Warning] | ||
> This field contains sensitive (linkable PII) information. | ||
examples: ['QdH5CAWJgqVT4rOr0qtumf'] | ||
heyams marked this conversation as resolved.
Show resolved
Hide resolved
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With this link-target gone, what should be used instead?