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.

Sign up for GitHub

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

Jump to bottom

Proposal for SIG OpenTelemetry on Mainframe #1687

Merged
merged 14 commits into from
Jan 25, 2024
Merged

Proposal for SIG OpenTelemetry on Mainframe #1687

merged 14 commits into from
Jan 25, 2024

Conversation

rrschulze
Copy link
Contributor

@rrschulze rrschulze commented Sep 21, 2023 •
edited by arminru
Loading

This is the proposal for a work group and project focusing on enhancing OpenTelemetry to support the Mainframe. For details, please see the proposal in projects/mainframe.md.

Resolves #1686.

@rrschulze rrschulze mentioned this pull request Sep 21, 2023
Closed
@tedsuo
Copy link
Contributor

tedsuo commented Sep 21, 2023

Hi @rrschulze. Can you include the standard staffing section from the template? https://github.com/open-telemetry/community/blob/main/project-template.md#staffing--help-wanted

@rrschulze
Refined SIG OpenTelemetry on Mainframe proposal in mainframe.md
dee3247 
- Included number of contributors we foresee
- Clarified scope of tracks beyond the implementation of the code instrumentation SDK for Cobol
- Detailed approach enhancements of the semantic conventions.
@jmertic jmertic mentioned this pull request Oct 12, 2023
Closed
@rrschulze
Copy link
Contributor Author

I confirm to contribute to this SIG.

@jackjia-ibm
Copy link

I confirm to contribute to this SIG. I’m Jack T. Jia and currently working for IBM.

@youngaaronm
Copy link

I confirm to contribute to this SIG. - Aaron Young, Broadcom

@v1gnesh
Copy link

v1gnesh commented Oct 24, 2023 •
edited
Loading

Hello @rrschulze, could you please invite me to the mentioned slack channel.
I may not be able to attend the meetings due to time zone differences.
However, can follow along from notes & recordings.

PS: I'm a mainframer representing only myself.

@rrschulze
Copy link
Contributor Author

@v1gnesh it is the Slack channel #otel-mainframes in the CNCF Slack workspace. If you're not yet a member of the Slack workspace, you'll need to join it first and then you can become a member of the channel #otel-mainframes yourself.

@v1gnesh
Copy link

v1gnesh commented Oct 24, 2023

It seems to be... broken? openmainframeproject/tac#535 (comment)

@jimporell
Copy link

working through CLA discussion within Rocket, but plan to participate

@msomasu
Copy link

msomasu commented Oct 27, 2023 •
edited
Loading

I confirm to contribute to this SIG. Muruganandam (Anand) Somasundaram, IBM.

@v1gnesh
Copy link

v1gnesh commented Nov 8, 2023

@rrschulze Am I still supposed to see the cert warning in the slack workspace?
Also, can I sign up with email rather than GH/Google, etc. ?

@trask
Copy link
Member

trask commented Nov 8, 2023

@rrschulze Am I still supposed to see the cert warning in the slack workspace? Also, can I sign up with email rather than GH/Google, etc. ?

what cert warning are you seeing?

you should be able to sign up with any email you wish

@davdai01
Copy link

davdai01 commented Nov 9, 2023

I confirm to contribute to this SIG. - David Dai, Rocket Software

@v1gnesh
Copy link

v1gnesh commented Nov 9, 2023

@trask
image

@jmertic
Copy link

jmertic commented Nov 17, 2023

@trask image

@dyladan - what is the right escalation path for issues accessing CNCF Slack? https://servicedesk.cncf.io/?

@dyladan
Copy link
Member

dyladan commented Nov 17, 2023

@trask image

@dyladan - what is the right escalation path for issues accessing CNCF Slack? https://servicedesk.cncf.io/?

That would be my best guess

@trask
Copy link
Member

trask commented Nov 17, 2023

@v1gnesh https://slack.cncf.io is just a redirect to https://communityinviter.com/apps/cloud-native/cncf, can you try that directly instead?

@tedsuo tedsuo changed the title Proposal for establishing a SIG OpenTelemetry on Mainframe Proposal for SIG OpenTelemetry on Mainframe Dec 14, 2023
@rrschulze
Copy link
Contributor Author

Hi @tedsuo, @mtwo can you please advise what needs to happen next for this proposal?

@mtwo
Copy link
Member

mtwo commented Jan 4, 2024 •
edited
Loading

Reconciling meeting updates and discussions with this issue. We discussed this in the October GC / TC meeting.

The TC is supportive of the mainframe effort given that it doesn’t stretch existing OTel technical leadership, who are already slammed with semantic convention stabilization. This means:

  • No major semantic changes are required in the short-term, the focus should be on fitting mainframe telemetry metadata into the existing OTel semantics
  • There are net-new people to OTel leading this (confirmed via the comments above)

I will be the GC sponsor for this project, and I think that @alolita and @dyladan plan to participate to some extent. As for next steps, we should get weekly meetings planned and put onto the calendar, and then we should get to work.

@rrschulze
Copy link
Contributor Author

Thanks @mtwo. @alolita or @dyladan, may I ask you to do a review of the PR as well?

@rrschulze
Copy link
Contributor Author

We need at least one more review and approval. Who can help with it?

@jmertic
Copy link

jmertic commented Jan 10, 2024

Let me know when the meeting schedule is set; I want to cross-post to the Open Mainframe Project Community calendar

@trask
Copy link
Member

trask commented Jan 10, 2024

can you fix the spell checking CI by adding the unknown words to https://github.com/open-telemetry/community/blob/main/.cspell.yaml?

@trask trask mentioned this pull request Jan 10, 2024
Closed
@rrschulze
Copy link
Contributor Author

@jmertic we will start the weekly meetings for the SIG OpenTelemetry on Mainframe next Tuesday, January 16, at 10:00 am PT. Please see the OpenTelemetry calendar.

@trask
Copy link
Member

trask commented Jan 24, 2024

@dyladan @alolita can you review / approve since you are listed as sponsors? thx

@alolita
Copy link
Member

alolita commented Jan 25, 2024 •
edited
Loading

Hi Folks! Sorry about the delayed response. Thrilled to see this SIG for OpenTelemetry on mainframe platforms defining sem conventions take off. I plan to participate in the discussions and reviews. Also I support @mtwo in sponsoring this SIG.

alolita
alolita approved these changes Jan 25, 2024
View reviewed changes
Copy link
Member

@alolita alolita left a comment

Choose a reason for hiding this comment

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

Lgtm. Thanks for the suggestions @trask @tigrannajaryan

@mtwo mtwo merged commit c79e65c into open-telemetry:main Jan 25, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@trask trask trask left review comments

@tigrannajaryan tigrannajaryan tigrannajaryan left review comments

@mtwo mtwo mtwo approved these changes

@alolita alolita alolita approved these changes

@bhs bhs Awaiting requested review from bhs

@bogdandrutu bogdandrutu Awaiting requested review from bogdandrutu

@dyladan dyladan Awaiting requested review from dyladan dyladan is a code owner

@jpkrohling jpkrohling Awaiting requested review from jpkrohling jpkrohling is a code owner

@tedsuo tedsuo Awaiting requested review from tedsuo tedsuo is a code owner

@yurishkuro yurishkuro Awaiting requested review from yurishkuro

@austinlparker austinlparker Awaiting requested review from austinlparker austinlparker is a code owner

@danielgblanco danielgblanco Awaiting requested review from danielgblanco danielgblanco is a code owner

@svrnm svrnm Awaiting requested review from svrnm svrnm is a code owner

Assignees
No one assigned
Labels
area/project-proposal Submitting a filled out project template
Projects
🔭 Main Backlog
Status: Current Projects
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Proposal for establishing the SIG OpenTelemetry on Mainframe