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

fix: ensure passwords in helm templates are quoted #96

Merged
merged 5 commits into from
Jul 24, 2024

Conversation

ahmad-ibra
Copy link
Collaborator

@ahmad-ibra ahmad-ibra commented Jul 23, 2024

Description

This is to address issues like the one below when converting yaml to json:

failed to install validator: failed to install validator helm chart: error executing helm upgrade: Release "validator" does not exist. Installing it now.
Error: failed to parse /var/folders/h_/0543c2rd71z0gvw2xbqxcjsr0000gn/T/3171513764: error converting YAML to JSON: yaml: line 69: did not find expected comment or line break

The other changes made in the components package to the ConfigureXXXPlugin functions ensure they each setup their structs correctly to avoid nil pointer dereferences

@ahmad-ibra
Copy link
Collaborator Author

This PR should not be merged until validator-labs/validator#342 is merged and a new release of validator is created. We'll need to bump our dependencies to use the latest version of validator.

Copy link

codecov bot commented Jul 23, 2024

Codecov Report

Attention: Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.

@@            Coverage Diff             @@
##             main      #96      +/-   ##
==========================================
- Coverage   52.32%   52.28%   -0.05%     
==========================================
  Files          45       45              
  Lines        5005     5009       +4     
==========================================
  Hits         2619     2619              
- Misses       1725     1729       +4     
  Partials      661      661              
Files Coverage Δ
pkg/components/validator.go 59.87% <0.00%> (-0.19%) ⬇️
pkg/components/network.go 0.00% <0.00%> (ø)
pkg/components/oci.go 0.00% <0.00%> (ø)
pkg/components/vsphere.go 0.00% <0.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2a01e95...341eaee. Read the comment docs.

@ahmad-ibra ahmad-ibra marked this pull request as ready for review July 24, 2024 00:18
@ahmad-ibra ahmad-ibra requested a review from a team as a code owner July 24, 2024 00:18
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. bug Something isn't working labels Jul 24, 2024
@ahmad-ibra ahmad-ibra changed the title fix: ensure passwords are quoted fix: ensure passwords in helm templates are quoted Jul 24, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 24, 2024
@ahmad-ibra ahmad-ibra merged commit f36383b into main Jul 24, 2024
7 of 8 checks passed
@ahmad-ibra ahmad-ibra deleted the fix/validator-values-password branch July 24, 2024 00:38
ahmad-ibra pushed a commit that referenced this pull request Jul 24, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.0.5](v0.0.4...v0.0.5)
(2024-07-24)


### Features

* add support for private custom image registries
([#83](#83))
([ae91659](ae91659))
* support env vars in OCI secrets
([#88](#88))
([584b3c7](584b3c7))


### Bug Fixes

* ensure passwords in helm templates are quoted
([#96](#96))
([f36383b](f36383b))


### Other

* omit EDITOR logs by default
([#76](#76))
([b3ab7ec](b3ab7ec))


### Dependency Updates

* **deps:** update anchore/sbom-action action to v0.17.0
([#75](#75))
([caf800d](caf800d))
* **deps:** update golang.org/x/exp digest to 8a7402a
([#89](#89))
([3ffda87](3ffda87))
* **deps:** update golang.org/x/exp digest to e3f2596
([#82](#82))
([a89beb6](a89beb6))
* **deps:** update module github.com/validator-labs/validator to v0.0.47
([#92](#92))
([8c359e1](8c359e1))
* **deps:** update module github.com/validator-labs/validator to v0.0.48
([#94](#94))
([743656d](743656d))
* **deps:** update module
github.com/validator-labs/validator-plugin-azure to v0.0.13
([#79](#79))
([0c2dff7](0c2dff7))
* **deps:** update module
github.com/validator-labs/validator-plugin-network to v0.0.18
([#87](#87))
([c69e355](c69e355))
* **deps:** update module
github.com/validator-labs/validator-plugin-network to v0.0.19
([#95](#95))
([e70433a](e70433a))
* **deps:** update module github.com/validator-labs/validator-plugin-oci
to v0.0.11
([#90](#90))
([a633962](a633962))
* **deps:** update module
github.com/validator-labs/validator-plugin-vsphere to v0.0.27
([#80](#80))
([feb6360](feb6360))
* **deps:** update module github.com/vmware/govmomi to v0.39.0
([#93](#93))
([2a01e95](2a01e95))
* **deps:** update softprops/action-gh-release digest to c062e08
([#85](#85))
([b8b5c62](b8b5c62))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants