Skip to content

fix: support validator rules check -f config.yaml without all plugins defined#191

Merged
ahmad-ibra merged 3 commits intomainfrom
fix/direct-nil-ptr
Aug 22, 2024
Merged

fix: support validator rules check -f config.yaml without all plugins defined#191
ahmad-ibra merged 3 commits intomainfrom
fix/direct-nil-ptr

Conversation

@ahmad-ibra
Copy link
Member

Issue

Resolves #188

Description

Ensures we dont panic when executing validator rules check -f config.yaml without all plugins defined

@ahmad-ibra ahmad-ibra requested a review from a team as a code owner August 22, 2024 20:43
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. bug Something isn't working labels Aug 22, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 22, 2024
@codecov
Copy link

codecov bot commented Aug 22, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 6 lines in your changes missing coverage. Please review.

Files Patch % Lines
pkg/components/validator.go 0.00% 0 Missing and 6 partials ⚠️
@@            Coverage Diff             @@
##             main     #191      +/-   ##
==========================================
+ Coverage   53.51%   53.52%   +0.01%     
==========================================
  Files          44       44              
  Lines        6296     6294       -2     
==========================================
  Hits         3369     3369              
+ Misses       2077     2076       -1     
+ Partials      850      849       -1     
Files Coverage Δ
pkg/cmd/validator/validator.go 58.51% <100.00%> (ø)
pkg/components/validator.go 58.50% <0.00%> (+0.33%) ⬆️

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 3d20725...d400b6f. Read the comment docs.

@ahmad-ibra ahmad-ibra merged commit 6829834 into main Aug 22, 2024
@ahmad-ibra ahmad-ibra deleted the fix/direct-nil-ptr branch August 22, 2024 21:29
mattwelke pushed a commit that referenced this pull request Aug 23, 2024
…ns defined (#191)

Resolves #188

Ensures we dont panic when executing `validator rules check -f
config.yaml` without all plugins defined
TylerGillson added a commit that referenced this pull request Sep 3, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.1.3](v0.1.2...v0.1.3)
(2024-09-03)


### Features

* inline auth for MAAS
([#198](#198))
([ed78617](ed78617))
* support Azure plugin community gallery image rule
([#181](#181))
([3d20725](3d20725))


### Bug Fixes

* ensure ErrValidationFailed is returned for maas direct validation
failures
([#179](#179))
([0e0e7b9](0e0e7b9))
* handle validation errors, result count mismatch
([#204](#204))
([a2ea08b](a2ea08b))
* only require docker, kind when provisioning kind cluster
([#180](#180))
([1bbdb0e](1bbdb0e))
* remove duplicate maas base values from template
([#182](#182))
([cfa39a8](cfa39a8))
* support `validator rules check -f config.yaml` without all plugins
defined
([#191](#191))
([6829834](6829834))


### Other

* cleanup comment
([#176](#176))
([f371927](f371927))


### Dependency Updates

* **deps:** update anchore/sbom-action action to v0.17.2
([#184](#184))
([00d3a5c](00d3a5c))
* **deps:** update github.com/validator-labs/validator-plugin-maas
digest to e903cc7
([#175](#175))
([09c3ad1](09c3ad1))
* **deps:** update golang.org/x/exp digest to 9b4947d
([#190](#190))
([8c427e9](8c427e9))
* **deps:** update module github.com/canonical/gomaasclient to v0.7.0
([#197](#197))
([f43a5ba](f43a5ba))
* **deps:** update module github.com/validator-labs/validator to v0.1.8
([#120](#120))
([fe587c6](fe587c6))
* **deps:** update module github.com/vmware/govmomi to v0.42.0
([#172](#172))
([f0488c9](f0488c9))


### Refactoring

* move vsphere account under auth to match new plugin api
([#189](#189))
([90352e1](90352e1))

---
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:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛 validator rules check -f config.yaml chokes if provided a config file without all plugins defined

2 participants