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

feat: support direct oci validation of private registries #173

Merged
merged 10 commits into from
Aug 19, 2024

Conversation

ahmad-ibra
Copy link
Collaborator

Issue

N/A

Description

Prior to this change, validatorctl was only able to configure auth for oci registry validation via secrets. Now, we can configure auth via either basicAuth or ecrAuth.

@ahmad-ibra ahmad-ibra force-pushed the feat/add-oci-direct-auth branch 3 times, most recently from b6fa3aa to 32ac8f2 Compare August 16, 2024 22:53
@ahmad-ibra ahmad-ibra marked this pull request as ready for review August 16, 2024 23:18
@ahmad-ibra ahmad-ibra requested a review from a team as a code owner August 16, 2024 23:18
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. new-feature Net-new feature labels Aug 16, 2024
Copy link

codecov bot commented Aug 17, 2024

Codecov Report

Attention: Patch coverage is 25.00000% with 51 lines in your changes missing coverage. Please review.

Files Patch % Lines
pkg/services/validator/oci.go 20.45% 30 Missing and 5 partials ⚠️
pkg/components/validator.go 16.66% 8 Missing and 2 partials ⚠️
pkg/services/validator/common.go 45.45% 3 Missing and 3 partials ⚠️
@@            Coverage Diff             @@
##             main     #173      +/-   ##
==========================================
- Coverage   53.77%   53.60%   -0.18%     
==========================================
  Files          43       44       +1     
  Lines        6086     6218     +132     
==========================================
+ Hits         3273     3333      +60     
- Misses       1994     2056      +62     
- Partials      819      829      +10     
Files Coverage Δ
pkg/config/constants.go 100.00% <ø> (ø)
pkg/services/validator/aws.go 50.86% <100.00%> (+0.42%) ⬆️
pkg/services/validator/common.go 47.91% <45.45%> (-0.21%) ⬇️
pkg/components/validator.go 58.52% <16.66%> (-1.42%) ⬇️
pkg/services/validator/oci.go 46.42% <20.45%> (-4.37%) ⬇️

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 c3a714c...3044d62. Read the comment docs.

pkg/services/validator/oci.go Outdated Show resolved Hide resolved
pkg/services/validator/oci.go Show resolved Hide resolved
pkg/services/validator/oci.go Show resolved Hide resolved
pkg/components/validator.go Show resolved Hide resolved
@ahmad-ibra ahmad-ibra force-pushed the feat/add-oci-direct-auth branch 2 times, most recently from 9e28ff0 to d077f5d Compare August 19, 2024 18:02
TylerGillson
TylerGillson previously approved these changes Aug 19, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 19, 2024
@ahmad-ibra ahmad-ibra merged commit 9cfeab9 into main Aug 19, 2024
7 of 8 checks passed
@ahmad-ibra ahmad-ibra deleted the feat/add-oci-direct-auth branch August 19, 2024 18:45
ahmad-ibra pushed a commit that referenced this pull request Aug 19, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.1.2](v0.1.1...v0.1.2)
(2024-08-19)


### Features

* add maas plugin
([#160](#160))
([ab9f21a](ab9f21a))
* allow selecting aws creds from filesystem
([#171](#171))
([c3a714c](c3a714c))
* allow specifying Azure cloud to connect to
([#170](#170))
([6a4a704](6a4a704))
* read vCenter privileges from local file or editor
([#152](#152))
([94ddd90](94ddd90))
* set exit code 2 on validation failure; restore debug log file
([#150](#150))
([2a3fe4d](2a3fe4d))
* support configuring oci validationType on a rule
([#161](#161))
([8dfc501](8dfc501))
* support direct oci validation of private registries
([#173](#173))
([9cfeab9](9cfeab9))


### Bug Fixes

* correct TUI flow for `validator install -o --apply`
([#169](#169))
([0912f6e](0912f6e))
* export creds for aws and azure direct check
([#167](#167))
([5d569de](5d569de))


### Dependency Updates

* **deps:** update anchore/sbom-action action to v0.17.1
([#163](#163))
([416d23c](416d23c))
* **deps:** update github.com/validator-labs/validator-plugin-azure
digest to b4687e5
([#149](#149))
([e7ab9a6](e7ab9a6))
* **deps:** update github.com/validator-labs/validator-plugin-vsphere
digest to a93cb70
([#147](#147))
([79304b9](79304b9))
* **deps:** update module github.com/vmware/govmomi to v0.40.0
([#162](#162))
([acf4a25](acf4a25))


### Refactoring

* lazy configuration of oci auth and signature verification secrets
([#168](#168))
([cc2c056](cc2c056))
* remove explicit TypeMetas; use vapi constants
([#154](#154))
([28b321c](28b321c))

---
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
lgtm This PR has been approved by a maintainer new-feature Net-new feature size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants