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

Refactoring Kubernetes recommanded labels #78

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

nlamirault
Copy link
Contributor

@nlamirault nlamirault commented May 2, 2024

All resources have Kubernetes recommended labels

Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
@nlamirault nlamirault changed the title Cleanup Kubernetes recommanded labels Refactoring Kubernetes recommanded labels May 2, 2024
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Copy link
Contributor

@rdettai rdettai left a comment

Choose a reason for hiding this comment

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

The effort to have a more unified and standardized labeling is very welcome, thanks for your contribution!

charts/quickwit/templates/serviceaccount.yaml Outdated Show resolved Hide resolved
{{- include "quickwit.control_plane.selectorLabels" . | nindent 8 }}
{{- include "quickwit.additionalLabels" . | nindent 8 }}
Copy link
Contributor

Choose a reason for hiding this comment

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

quickwit.additionalLabels is already part of quickwit.labels

charts/quickwit/templates/_helpers.tpl Outdated Show resolved Hide resolved
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
* main: (42 commits)
  Bump appVersion to 0.8.2
  Feat issue quickwit-oss#98: merging jobs and bootstrap section
  Fix metastore volume indentation
  Bump to v0.6.4
  Fix helm chart comments
  Reduce indentation by 2
  Bump version to 0.6.2
  Introduce component specific initContainers, volumes and volumeMounts Use nodeSelector, affinity, tolerations from bootstrap config values. This ensures pods generated from k8s Job have these settings propagated. Allow command and args to be overridden via values files. This helps in providing custom config files generated from templates as necessary
  Add possibility to specify services type per service
  fix: readme missing newlines
  fix: readme suggested changes
  fix: add back indexer & search settings example
  fix: readme depreciation notice
  chore: document the breaking change and change the versioning accordingly
  fix: add QW_METASTORE_URI doc
  chore: add configMapRef example in values.yaml
  fix: add suggested change
  Add artifacthub metadata file
  feat: add requested changes
  fix: quickwit secret leak in configmap
  ...

Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
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.

2 participants