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

Proposal for SIG CICD observability #1822

Merged
merged 31 commits into from
Jan 26, 2024
Merged

Conversation

adrielp
Copy link
Contributor

@adrielp adrielp commented Nov 29, 2023

No description provided.

Copy link
Member

@svrnm svrnm left a comment

Choose a reason for hiding this comment

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

Some remarks, some questions. To get rid of the spell checking issues add the words to .cspell.yaml

projects/ci-cd.md Outdated Show resolved Hide resolved
projects/ci-cd.md Outdated Show resolved Hide resolved
projects/ci-cd.md Outdated Show resolved Hide resolved
projects/ci-cd.md Outdated Show resolved Hide resolved
projects/ci-cd.md Outdated Show resolved Hide resolved
projects/ci-cd.md Show resolved Hide resolved
projects/ci-cd.md Outdated Show resolved Hide resolved
projects/ci-cd.md Outdated Show resolved Hide resolved
projects/ci-cd.md Outdated Show resolved Hide resolved
projects/ci-cd.md Outdated Show resolved Hide resolved
Copy link

@ericchapman80 ericchapman80 left a comment

Choose a reason for hiding this comment

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

See inline suggestions.

projects/ci-cd.md Show resolved Hide resolved
projects/ci-cd.md Outdated Show resolved Hide resolved
projects/ci-cd.md Outdated Show resolved Hide resolved
@carlosalberto
Copy link
Contributor

Hey all,

We at the TC briefly discussed this yesterday, and we decided to go ahead with this SIG, and I will be your initial contact point.

As the vast majority of the changes will be semantic convention additions (no major Specification changes are expected at this time), we'd like to have the group attend the Semantic Convention SIG calls on Mondays, in order to present and discuss any initial topic. This will be a good opportunity for others to get a general idea of this initiative, and also good for the new group to get used to the OTel community.

A little while after, as the effort picks up, we can start having separate calls 100% dedicated to CI/CD.

Let me know if that works for you (Monday call times, etc).

@ericchapman80
Copy link

Hey all,

We at the TC briefly discussed this yesterday, and we decided to go ahead with this SIG, and I will be your initial contact point.

As the vast majority of the changes will be semantic convention additions (no major Specification changes are expected at this time), we'd like to have the group attend the Semantic Convention SIG calls on Mondays, in order to present and discuss any initial topic. This will be a good opportunity for others to get a general idea of this initiative, and also good for the new group to get used to the OTel community.

A little while after, as the effort picks up, we can start having separate calls 100% dedicated to CI/CD.

Let me know if that works for you (Monday call times, etc).

That's awesome @carlosalberto! Excited to get rolling!

@adrielp
Copy link
Contributor Author

adrielp commented Jan 18, 2024

Hey all,

We at the TC briefly discussed this yesterday, and we decided to go ahead with this SIG, and I will be your initial contact point.

As the vast majority of the changes will be semantic convention additions (no major Specification changes are expected at this time), we'd like to have the group attend the Semantic Convention SIG calls on Mondays, in order to present and discuss any initial topic. This will be a good opportunity for others to get a general idea of this initiative, and also good for the new group to get used to the OTel community.

A little while after, as the effort picks up, we can start having separate calls 100% dedicated to CI/CD.

Let me know if that works for you (Monday call times, etc).

This is awesome @carlosalberto! Thanks for this update!

Overall, I think the Monday will be fine to start. There's a lot of folks that have been added to be part of the SIG so I think we're likely to have the coverage to join the Monday meetings for a bit until the effort picks up enough to get a dedicated call.

We have a slack channel in the CNCF Slack where we have been communicating so in my mind, the next steps for us is to go to that meeting and present some initial topics we formulate there, which I'm sure will heavily be focused on the OTEP.

Is there anything I need to do to this PR as it stands now to get it approved & merged in?

@carlosalberto
Copy link
Contributor

@svrnm Is the lint step good for you?

@adrielp adrielp requested a review from svrnm January 19, 2024 18:33
@svrnm
Copy link
Member

svrnm commented Jan 19, 2024

@svrnm Is the lint step good for you?

yes! Apologies for keeping this blocked with my old review

Copy link

@oleg-nenashev oleg-nenashev left a comment

Choose a reason for hiding this comment

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

The proposal looks good! I definitely confirm my participation while wearing the @jenkinsci hat, and most likely @gradle too

projects/ci-cd.md Show resolved Hide resolved
@carlosalberto
Copy link
Contributor

@jsuereth Looks good?

cc @trask

@jsuereth
Copy link
Contributor

Given this group will share time w/ the Semantic Convention WG meeting, you can add me, as well, for the TC sponsor.

projects/ci-cd.md Outdated Show resolved Hide resolved
@trask trask merged commit d715adf into open-telemetry:main Jan 26, 2024
3 checks passed
@trask
Copy link
Member

trask commented Jan 26, 2024

Welcome CI/CD semconv working group!!!

@adrielp adrielp deleted the cicd-wg branch January 26, 2024 22:41
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

Successfully merging this pull request may close these issues.