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

CP-20221: Labels and Annotations #127

Merged
merged 8 commits into from
Dec 23, 2024
Merged

CP-20221: Labels and Annotations #127

merged 8 commits into from
Dec 23, 2024

Conversation

josephbarnett
Copy link
Contributor

@josephbarnett josephbarnett commented Dec 23, 2024

Description

  • Release notes for 1.0.0 Release
  • Troubleshooting guide for Certificates Issues
  • Bump version of insights-controller image

Testing

N/A

@josephbarnett josephbarnett requested a review from a team as a code owner December 23, 2024 15:04
@josephbarnett josephbarnett changed the base branch from develop to release/1.0.0 December 23, 2024 15:05
@josephbarnett josephbarnett changed the title 1.0.0 ga CP-20221: Labels and Annotations Dec 23, 2024
Copy link

@adamrice32 adamrice32 left a comment

Choose a reason for hiding this comment

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

Question: cert-manager is defaulted to enabled, and yet you state that most environments will already have this deployed. is it just me or is this contradictory?

@josephbarnett
Copy link
Contributor Author

Question: cert-manager is defaulted to enabled, and yet you state that most environments will already have this deployed. is it just me or is this contradictory?

Great question! I think in most "production" environments we've seen they already have a deployment. I think for easy of install we had this enabled, but maybe we should disable it? @dmepham would like your input here.

Copy link
Contributor

@beckilee beckilee left a comment

Choose a reason for hiding this comment

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

Just a few editorial suggestions, but otherwise looks good!

charts/cloudzero-agent/docs/cert-trouble-shooting.md Outdated Show resolved Hide resolved
charts/cloudzero-agent/docs/cert-trouble-shooting.md Outdated Show resolved Hide resolved
- [`cert-manager` Documentation](https://cert-manager.io/docs/)
- [CloudZero Agent Helm Chart README](./README.md)
- [ValidatingWebhookConfiguration Reference](https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.23/#validatingwebhookconfiguration-v1-admissionregistration-k8s-io)
- [Helm Troubleshooting Guide](https://helm.sh/docs/chart_best_practices/troubleshooting/)
Copy link
Contributor

Choose a reason for hiding this comment

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

This URL is broken; should we link to https://helm.sh/docs/faq/troubleshooting/ instead?

charts/cloudzero-agent/docs/cert-trouble-shooting.md Outdated Show resolved Hide resolved
charts/cloudzero-agent/docs/releases/1.0.0.md Outdated Show resolved Hide resolved
@josephbarnett josephbarnett merged commit 12dada1 into release/1.0.0 Dec 23, 2024
2 checks passed
@josephbarnett josephbarnett deleted the 1.0.0-ga branch December 23, 2024 16:19
@josephbarnett josephbarnett mentioned this pull request Jan 9, 2025
3 tasks
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.

4 participants