Skip to content

Conversation

@katarzyna-koltun-mx
Copy link
Collaborator

No description provided.

@katarzyna-koltun-mx katarzyna-koltun-mx self-assigned this Oct 22, 2024
Copy link

@muhong0 muhong0 left a comment

Choose a reason for hiding this comment

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

@katarzyna Kołtun I read the update, Svix parameter “Image" is not optional,

  • Image - Optional. If you are using a self-signed TLS certificate, set this path to {customer-private-image-registry-url}/svix/svix-server:v1.25.tls.

The change suggestion:

Image - The default path is svix/svix-server:v1.25.0, or if you are using a self-signed TLS certificate and private build image, set this path to {customer-private-image-registry-url}/svix/svix-server:v1.25.tls.

All of other sections look fine, thanks

@katarzyna-koltun-mx
Copy link
Collaborator Author

@muhong0 Done! Shall I publish the PR or wait for the go ahead?

@muhong0
Copy link

muhong0 commented Oct 22, 2024

@katarzyna Kołtun It's OK, please publish it, thanks

@katarzyna-koltun-mx katarzyna-koltun-mx merged commit ac0f2bf into development Oct 23, 2024
2 checks passed
@katarzyna-koltun-mx katarzyna-koltun-mx deleted the kk-pmp-svix-ssl branch October 23, 2024 09:48
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