Skip to content

fix: ensure vsphere password is quoted#342

Merged
ahmad-ibra merged 1 commit intomainfrom
fix/vsphere-values-password
Jul 23, 2024
Merged

fix: ensure vsphere password is quoted#342
ahmad-ibra merged 1 commit intomainfrom
fix/vsphere-values-password

Conversation

@ahmad-ibra
Copy link
Member

Description

ensure the vsphere password is quoted to avoid any yaml to json conversion errors like the following:

Error: YAML parse error on validator/templates/plugin-secret-vsphere.yaml: error converting YAML to JSON: yaml: line 7: did not find expected comment or line break

@ahmad-ibra ahmad-ibra requested a review from a team as a code owner July 23, 2024 23:14
@ahmad-ibra ahmad-ibra requested a review from arturshadnik July 23, 2024 23:14
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. bug Something isn't working labels Jul 23, 2024
@ahmad-ibra ahmad-ibra requested a review from TylerGillson July 23, 2024 23:14
@codecov
Copy link

codecov bot commented Jul 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

@@            Coverage Diff             @@
##             main     #342      +/-   ##
==========================================
+ Coverage   48.58%   48.90%   +0.31%     
==========================================
  Files          22       22              
  Lines        1274     1274              
==========================================
+ Hits          619      623       +4     
+ Misses        587      584       -3     
+ Partials       68       67       -1     

see 1 file with indirect coverage changes


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 5f57fb8...a70e799. Read the comment docs.

@ahmad-ibra ahmad-ibra force-pushed the fix/vsphere-values-password branch 3 times, most recently from 8e9e6cd to 5b0ed88 Compare July 23, 2024 23:25
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 23, 2024
@ahmad-ibra ahmad-ibra merged commit 3f297de into main Jul 23, 2024
@ahmad-ibra ahmad-ibra deleted the fix/vsphere-values-password branch July 23, 2024 23:51
ahmad-ibra pushed a commit that referenced this pull request Jul 24, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.0.49](v0.0.48...v0.0.49)
(2024-07-23)


### Bug Fixes

* ensure vsphere password is quoted
([#342](#342))
([3f297de](3f297de))

---
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:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants