Skip to content

Conversation

@miguelsorianod
Copy link
Contributor

Rename .spec.openTracing.tracingConfigRef to .spec.openTracing.tracingConfigSecretRef to better indicate that the config to be provided is a K8s secret. Used in APIcast production and staging

@miguelsorianod
Copy link
Contributor Author

This will need to be backported to 2.11-candidate and 2.11-stable. QE is aware of this

Copy link
Member

@eguzki eguzki left a comment

Choose a reason for hiding this comment

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

Additionally, here the openTracingSpec link is broken

@miguelsorianod miguelsorianod force-pushed the rename-tracingconfigsecretref branch from 36b9e46 to 6722d33 Compare September 21, 2021 14:58
@miguelsorianod miguelsorianod force-pushed the rename-tracingconfigsecretref branch from 6722d33 to 2c2ced9 Compare September 21, 2021 15:00
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 2c2ced9 and detected 0 issues on this pull request.

View more on Code Climate.

@miguelsorianod miguelsorianod merged commit ab916df into master Sep 22, 2021
@miguelsorianod miguelsorianod deleted the rename-tracingconfigsecretref branch September 22, 2021 08:40
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