Skip to content

Add specific attributes for Apache RocketMQ#1904

Merged
arminru merged 1 commit intoopen-telemetry:mainfrom
aaron-ai:pr
Oct 13, 2021
Merged

Add specific attributes for Apache RocketMQ#1904
arminru merged 1 commit intoopen-telemetry:mainfrom
aaron-ai:pr

Conversation

@aaron-ai
Copy link
Copy Markdown
Member

@aaron-ai aaron-ai commented Sep 2, 2021

Changes

Add specific attributes for Apache RocketMQ

Related issues #

Related oteps #1900

@aaron-ai aaron-ai requested review from a team September 2, 2021 12:43
@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Sep 2, 2021 •
edited
Loading

CLA Signed

The committers are authorized under a signed CLA.

@arminru arminru added area:semantic-conventions Related to semantic conventions spec:trace Related to the specification/trace directory labels Sep 2, 2021
Oberon00
Oberon00 reviewed Sep 2, 2021
View reviewed changes
Comment thread semantic_conventions/trace/messaging.yaml
arminru
arminru reviewed Sep 2, 2021
View reviewed changes
Comment thread semantic_conventions/trace/messaging.yaml Outdated
Comment thread semantic_conventions/trace/messaging.yaml Outdated
Comment thread semantic_conventions/trace/messaging.yaml Outdated
Comment thread semantic_conventions/trace/messaging.yaml Outdated
Comment thread semantic_conventions/trace/messaging.yaml Outdated
Comment thread semantic_conventions/trace/messaging.yaml Outdated
Comment thread semantic_conventions/trace/messaging.yaml Outdated
Comment thread semantic_conventions/trace/messaging.yaml Outdated
Comment thread specification/trace/semantic_conventions/messaging.md
Comment thread specification/trace/semantic_conventions/messaging.md Outdated
Oberon00
Oberon00 reviewed Sep 2, 2021
View reviewed changes
Comment thread semantic_conventions/trace/messaging.yaml
Oberon00
Oberon00 reviewed Sep 2, 2021
View reviewed changes
Comment thread semantic_conventions/trace/messaging.yaml
Oberon00
Oberon00 reviewed Sep 2, 2021
View reviewed changes
Comment thread semantic_conventions/trace/messaging.yaml Outdated
@aaron-ai
Copy link
Copy Markdown
Member Author

aaron-ai commented Sep 2, 2021

All issues mentioned above have been fixed.

@aaron-ai aaron-ai force-pushed the pr branch 3 times, most recently from 46eac99 to 2b2cf6e Compare September 2, 2021 15:55
@aaron-ai aaron-ai requested review from Oberon00 and arminru September 2, 2021 16:15
@aaron-ai
Copy link
Copy Markdown
Member Author

aaron-ai commented Sep 10, 2021

any more feedback? @arminru @Oberon00

arminru
arminru reviewed Sep 10, 2021
View reviewed changes
Comment thread semantic_conventions/trace/messaging.yaml Outdated
Comment thread semantic_conventions/trace/messaging.yaml
Comment thread semantic_conventions/trace/messaging.yaml
@arminru arminru requested review from a team September 10, 2021 14:40
@arminru arminru linked an issue Sep 13, 2021 that may be closed by this pull request
Add specific attributes for Apache RocketMQ #1900
Closed
mateuszrzeszutek
mateuszrzeszutek reviewed Sep 13, 2021
View reviewed changes
Comment thread semantic_conventions/trace/messaging.yaml Outdated
@aaron-ai aaron-ai requested review from arminru and mateuszrzeszutek and removed request for a team September 14, 2021 07:13
@aaron-ai
Copy link
Copy Markdown
Member Author

aaron-ai commented Sep 15, 2021

All issues above have been addressed. @Oberon00 @mateuszrzeszutek @arminru

@aaron-ai
Copy link
Copy Markdown
Member Author

aaron-ai commented Sep 16, 2021

@open-telemetry/specs-approvers @open-telemetry/specs-trace-approvers @open-telemetry/technical-committee all comments are addressed, could another check and approve it?

@arminru arminru requested review from a team September 16, 2021 09:16
@arminru
Copy link
Copy Markdown
Member

arminru commented Sep 20, 2021

@open-telemetry/specs-approvers @open-telemetry/specs-trace-approvers PTAL 🙂

@aaron-ai
Copy link
Copy Markdown
Member Author

aaron-ai commented Sep 23, 2021

@arminru @Oberon00 all conflicts are resolved

@arminru
Copy link
Copy Markdown
Member

arminru commented Sep 28, 2021

@open-telemetry/specs-approvers @open-telemetry/specs-trace-approvers PTAL 🙂

@tigrannajaryan
Copy link
Copy Markdown
Member

tigrannajaryan commented Sep 28, 2021

@arminru I think this is one the PR's that would benefit from the guide that we thought would be helpful to write (the "How to make progress on proposals that do not initially generate enough interest/comments" here).

I don't know anything about Apache RocketMQ, so I cannot contribute very meaningfully to this PR. We need to come up with a good way to make contributions like this easy to make (I don't know yet how).

@tedsuo who are the current members/approvers/maintainers of Instrumentation SIG? I don't see a github team for Instrumentation to mention it. If you know the them can you please invite relevant people to this PR and also it would be great to have a Github team for the future.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 6, 2021

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 Oct 6, 2021
@tigrannajaryan tigrannajaryan mentioned this pull request Oct 7, 2021
Open
@aaron-ai
Copy link
Copy Markdown
Member Author

aaron-ai commented Oct 8, 2021

This pull request is suspend because of the vocation.

@github-actions github-actions Bot removed the Stale label Oct 8, 2021
Oberon00
Oberon00 reviewed Oct 8, 2021
View reviewed changes
Comment thread CHANGELOG.md
@aaron-ai aaron-ai requested review from Oberon00 and removed request for a team October 9, 2021 08:16
@Oberon00
Copy link
Copy Markdown
Member

Oberon00 commented Oct 12, 2021

I un-resolved #1904 (comment), please see my comment there.

@aaron-ai
Copy link
Copy Markdown
Member Author

aaron-ai commented Oct 13, 2021

could you re-ran the CI? @Oberon00 thanks.

@Oberon00 Oberon00 mentioned this pull request Oct 13, 2021
Closed
@arminru arminru enabled auto-merge (squash) October 13, 2021 08:57
@arminru arminru merged commit a1a8676 into open-telemetry:main Oct 13, 2021
@aaron-ai aaron-ai mentioned this pull request Oct 27, 2022
Closed
6 tasks
@jpkrohling jpkrohling mentioned this pull request Feb 26, 2024
Closed
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
@aaron-ai
Add specific attributes for Apache RocketMQ (open-telemetry#1904)
16d5690
schmikei pushed a commit to schmikei/opentelemetry-specification that referenced this pull request Apr 17, 2025
@aaron-ai
Add specific attributes for Apache RocketMQ (open-telemetry#1904)
94b32ee
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jsuereth jsuereth jsuereth approved these changes

@arminru arminru arminru approved these changes

@mateuszrzeszutek mateuszrzeszutek Awaiting requested review from mateuszrzeszutek

@Oberon00 Oberon00 Awaiting requested review from Oberon00

+1 more reviewer

@RongtongJin RongtongJin RongtongJin approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@yurishkuro yurishkuro

Labels

area:semantic-conventions Related to semantic conventions spec:trace Related to the specification/trace directory

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add specific attributes for Apache RocketMQ

8 participants

@aaron-ai @RongtongJin @arminru @tigrannajaryan @Oberon00 @jsuereth @mateuszrzeszutek @yurishkuro