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

Push down more state into /etc/podinfo #206

Merged
merged 11 commits into from
May 9, 2022
Merged

Conversation

spilchen
Copy link
Collaborator

@spilchen spilchen commented May 6, 2022

This adds more information to /etc/podinfo. The motivation is that we want the vertica server to include this stuff in a new k8s specific DC table.

A summary of the new information in /etc/podinfo is as follows:

  • k8s-build-date: Build date of the k8s server
  • k8s-git-commit: The git commit of the k8s server
  • k8s-version: The version of the k8s server
  • annotations: A list of all of the annotations set for the pod
  • namespace: The namespace the pod is created in
  • operator-deployment-method: What method did we use to deploy the operator
  • operator-version: What version of the operator are we using

As well, we are exposing the k8s node name and IP as environment variables:

  • HOST_IP
  • HOST_NODENAME

Two of the states (operator-deployment-method and operator-version) are fixed for the life of the operator. For this reason they come from a configMap that is populated either in the helm chart or the OLM bundle.

Also, changed PodFacts so that we have access to a Log.

@spilchen spilchen requested a review from roypaulin May 6, 2022 15:14
@spilchen spilchen self-assigned this May 6, 2022
Copy link
Collaborator

@roypaulin roypaulin left a comment

Choose a reason for hiding this comment

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

Looks good. Just some typo in some comments.

@spilchen spilchen requested a review from roypaulin May 6, 2022 19:56
@spilchen spilchen requested a review from roypaulin May 9, 2022 11:17
@spilchen spilchen merged commit d706926 into vertica:main May 9, 2022
@spilchen spilchen deleted the new-podinfo branch May 9, 2022 14:46
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