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: add support for private custom image registries #83

Merged
merged 12 commits into from
Jul 22, 2024

Conversation

ahmad-ibra
Copy link
Collaborator

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

Issue

#81

Description

This PR adds support for cutom image registries. Under the hood, the Registry struct will use Hauler if its IsAirgapped prop is set to true.

Tested by configuring artifacts in a harbor registry as shown below and installing validator with InsecureSkipTLSVerify set to true (and passing in the CA cert) and false

Screenshot 2024-07-22 at 1 40 21 PM

@ahmad-ibra ahmad-ibra force-pushed the feat/private-custom-registries branch 7 times, most recently from f11802e to 6417aad Compare July 19, 2024 00:46
@ahmad-ibra ahmad-ibra force-pushed the feat/private-custom-registries branch from 053b983 to a164707 Compare July 19, 2024 00:53
@ahmad-ibra ahmad-ibra force-pushed the feat/private-custom-registries branch from f6b98b6 to a0dc210 Compare July 22, 2024 18:17
@ahmad-ibra ahmad-ibra force-pushed the feat/private-custom-registries branch from a4b8f99 to 4802795 Compare July 22, 2024 20:45
@ahmad-ibra ahmad-ibra force-pushed the feat/private-custom-registries branch from 4802795 to b2c3696 Compare July 22, 2024 20:45
@ahmad-ibra ahmad-ibra force-pushed the feat/private-custom-registries branch from 7c9241d to d5dadb8 Compare July 22, 2024 21:30
@ahmad-ibra ahmad-ibra marked this pull request as ready for review July 22, 2024 22:30
@ahmad-ibra ahmad-ibra requested a review from a team as a code owner July 22, 2024 22:30
@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Jul 22, 2024
@dosubot dosubot bot added the new-feature Net-new feature label Jul 22, 2024
Copy link

codecov bot commented Jul 22, 2024

Codecov Report

Attention: Patch coverage is 21.80851% with 147 lines in your changes missing coverage. Please review.

@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
- Coverage   52.89%   52.32%   -0.58%     
==========================================
  Files          45       45              
  Lines        4881     5005     +124     
==========================================
+ Hits         2582     2619      +37     
- Misses       1640     1725      +85     
- Partials      659      661       +2     
Files Coverage Δ
pkg/components/validator.go 60.06% <100.00%> (ø)
pkg/utils/embed/embed.go 54.05% <100.00%> (+2.77%) ⬆️
...integration/_validator/testcases/test_validator.go 94.90% <100.00%> (+0.01%) ⬆️
pkg/utils/kind/kind.go 53.00% <90.47%> (+24.73%) ⬆️
pkg/services/validator/common.go 47.18% <25.00%> (+0.65%) ⬆️
pkg/components/environment.go 44.44% <44.44%> (+44.44%) ⬆️
pkg/services/validator/validator_service.go 36.28% <23.52%> (-1.83%) ⬇️
pkg/services/env_service.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 feb6360...716260a. Read the comment docs.

Copy link
Member

@TylerGillson TylerGillson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO the code in ensureDockerOciCaConfig needs to be moved into a separate "setup" command, but we can address that in a follow-up. It's just super invasive stuff and the expectation of having sudo is horrible.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 22, 2024
@ahmad-ibra ahmad-ibra merged commit ae91659 into main Jul 22, 2024
7 of 8 checks passed
@ahmad-ibra ahmad-ibra deleted the feat/private-custom-registries branch July 22, 2024 23:01
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
lgtm This PR has been approved by a maintainer new-feature Net-new feature size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants