Bump versions of Grafana and InfluxDB Helm charts #37
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Should we also bump the K8s version used in the tests to get coverage for this issue? |
I'm working on that now. That will require a few more changes. I'll update this review when I have this figured out. |
- Bump versions of Grafana and InfluxDB Helm charts. The latest version of Grafana suppresses creation of the PodSecurityPolicy resource if it is not available. - Replace requirements.lock with Chart.lock. Chart.lock is what gets generated when dependencies are defined using Chart.yaml. There is no requirements.yaml, so requirements.lock should not exist. - Update versions of dependencies installed in install_deps.sh. - Update install_deps.sh to use latest minikube and to not use --vm-driver, which apparently does not (or no longer) accepts --cpus and --memory. - Use large VM instance, which has 4 cores. - Update Golang libraries. All dependencies are inherited through the nuodb-helm-charts test library, so go.mod was updated to bump the nuodb-helm-charts version and all other libraries there were deleted, and then 'go mod tidy' was invoked to update go.mod and go.sum.
|
@adriansuarez Thanks for figuring out the K8s version bump! Looks good to me. |
adriansuarez
force-pushed
the
asuarez/fix-psp-deprecation
branch
from
ed6b2e6 to
23ed10a
Compare
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This should address the removal of PodSecurityPolicy from Kubernetes 1.25, since the Grafana Helm charts now conditionalize the creation of the PodSecurityPolicy on the existence of the API resource.