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 vsphere password is quoted #342

Merged
merged 1 commit into from
Jul 23, 2024
Merged

Conversation

ahmad-ibra
Copy link
Contributor

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
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. bug Something isn't working labels Jul 23, 2024
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
8 checks passed
@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