Skip to content

Commit 2ae7fef

Browse files
committed
fix: Fix incomplete air-gapped support
1 parent fc47e0d commit 2ae7fef

File tree

5 files changed

+15
-17
lines changed

5 files changed

+15
-17
lines changed

.github/workflows/release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,6 @@ jobs:
1818
registry-server: ghcr.io
1919
registry-username: ${{ github.actor }}
2020
image: ${{ github.repository }}
21-
version: 1.15.0
21+
version: 1.15.0+kadras.1
2222
secrets:
2323
pull-request-token: ${{ secrets.GH_ORG_PAT }}

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ schema:
2828

2929
# Use kbld to resolve the OCI images referenced within the manifests
3030
kbld:
31-
rm -f package/.imgpkg/images.yml && mkdir -p package/.imgpkg && kbld --file package/config --imgpkg-lock-output package/.imgpkg/images.yml 1>> /dev/null
31+
rm -f package/.imgpkg/images.yml && mkdir -p package/.imgpkg && ytt --file package/config | kbld -f- --imgpkg-lock-output package/.imgpkg/images.yml 1>> /dev/null
3232

3333
# Check the ytt-annotated Kubernetes configuration and its validation
3434
test-config:

package/config/kbld-config.yml

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,15 @@ kind: Config
55
# Search rules specify custom strategies for kbld to identify references for container images.
66
# See more about the search rules: https://carvel.dev/kbld/docs/latest/config/#search-rules.
77
searchRules:
8-
9-
# Resolves the image references from the specified OCI repository,
10-
# even when included in non-standard locations such as values in
11-
# environment variables.
12-
- valueMatcher:
13-
imageRepo: quay.io/jetstack/cert-manager-acmesolver
8+
9+
#! Some of the OCI images used by this package are defined as arguments to containers.
10+
#! We change the configuration to pass the image reference via configuration file
11+
#! so that can be discovered and its digest resolved by kbld. It's a pre-requisite for supporting
12+
#! package relocation to another registry and air-gapped scenarios.
13+
- keyMatcher:
14+
name: config.yaml
15+
updateStrategy:
16+
yaml:
17+
searchRules:
18+
- keyMatcher:
19+
path: [ acmeHTTP01Config, solverImage ]

package/config/overlays/config-controller.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,10 +18,6 @@ logging:
1818
verbosity: #@ data.values.controller.loglevel
1919
format: text
2020

21-
#! Some of the OCI images used by this package are defined as arguments to containers.
22-
#! We change the configuration to pass the image reference via configuration file
23-
#! so that can be discovered and its digest resolved by kbld. It's a pre-requisite for supporting
24-
#! package relocation to another registry and air-gapped scenarios.
2521
acmeHTTP01Config:
2622
solverImage: quay.io/jetstack/cert-manager-acmesolver:v1.15.0
2723
#@ end

package/package-resources.yml

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,7 @@ spec:
1212
template:
1313
spec:
1414
deploy:
15-
- kapp:
16-
rawOptions:
17-
- --wait-timeout=5m
18-
- --kube-api-qps=25
19-
- --kube-api-burst=50
15+
- kapp: {}
2016
fetch:
2117
- git: {}
2218
template:

0 commit comments

Comments
 (0)