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 openetelemetry_info to user_params #1129

Merged
merged 4 commits into from
Sep 14, 2023
Merged

Conversation

hjmodi
Copy link
Contributor

@hjmodi hjmodi commented Sep 11, 2023

Maintainers will complete the following section

  • Commit messages are descriptive enough
  • Code coverage from testing does not decrease and new code is covered
  • [n/a] JSON/YAML configuration changes are updated in the relevant schema
  • [n/a] Changes to metadata also update the documentation for the metadata
  • [n/a] Pull request has a link to an osbs-docs PR for user documentation updates

the outdated versions are getting random
error messages so updating to recent fedora
versions fixes that

Signed-off-by: Harsh Modi <[email protected]>
@mkosiarc
Copy link
Contributor

@rcerven @hjmodi Do we need to update the docs?

@mkosiarc
Copy link
Contributor

@hjmodi I have the same question here as in atomic-reactor PR regarding the commit messages

@rcerven
Copy link
Member

rcerven commented Sep 13, 2023

@rcerven @hjmodi Do we need to update the docs?

we can add that otel_url https://github.com/containerbuildsystem/osbs-client/blob/master/docs/configuration_file.md and I'd say that is all we need to document

This option will point to the OpenTelemetry
Collector root URL.

This is where all opentelemetry traces will be sent.

* MMENG-3834

Signed-off-by: Harsh Modi <[email protected]>
This will be a dictionary containing, `otel_url`
and `traceparent`. There are used in
tekton tasks to initialize opentelementry and
send traces to opentelemetry collector

* MMENG-3834

Signed-off-by: Harsh Modi <[email protected]>
expecially tasks with http requests.

This will allow us to get traces for all important
HTTP calls. We are also collecting all args provided
by the user for the build

* MMENG-3834

Signed-off-by: Harsh Modi <[email protected]>
@hjmodi
Copy link
Contributor Author

hjmodi commented Sep 13, 2023

@mkosiarc @rcerven updated commit messages and added documentation for config change.

@hjmodi hjmodi merged commit 89cd65a into containerbuildsystem:master Sep 14, 2023
14 checks passed
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.

3 participants