konflux-ci / architecture

Technical and Architecture documents
https://konflux-ci.dev/architecture/
18 stars 70 forks source link

Enable Native OpenTelemetry Tracing ADR #177

Closed mftb closed 7 months ago

mftb commented 8 months ago

Proposing an ADR for enabling native tracing capabilities on Konflux

mftb commented 8 months ago

@gbenhaim @amisstea PTAL

mftb commented 7 months ago

@ralphbean PTAL

ralphbean commented 7 months ago

@gabemontero @enarha I'd appreciate your review here.

gabemontero commented 7 months ago

@gabemontero @enarha I'd appreciate your review here.

@enarha is on PTO this week @ralphbean but I will try and look at this soon, today possibly

mftb commented 7 months ago

just the minor add of pipelines as code for future items wrt the tekton umbrella of stuff

otherwise I'm fine with the Tekton related references

I am curious why the other Konflux components like build sevice , application service, release service, integration service, enterprise contract were not explicitly called out like Tekton was. If it is implied here, apologies, I'm missing it.

Explicitly mentioned a few Konflux services.