Skip to content

[release-1.20] Remember --registry-insecure setting on recurring runs + backport fixes#3588

Merged
knative-prow[bot] merged 10 commits intoknative:release-1.20from
creydr:cp-remember-registry-insecure-flag-1.20
Apr 14, 2026
Merged

[release-1.20] Remember --registry-insecure setting on recurring runs + backport fixes#3588
knative-prow[bot] merged 10 commits intoknative:release-1.20from
creydr:cp-remember-registry-insecure-flag-1.20

Conversation

@creydr
Copy link
Copy Markdown
Member

@creydr creydr commented Apr 7, 2026

Manual backport of #3574

also adding

  • python template cloudevents package pin to <2 (introduces breaking changes)
  • update-deps.sh backport for conditional run of knative-components bump

creydr and others added 2 commits April 7, 2026 16:58
…ns (knative#3574)

* Pass --registry-insecure flag to Tekton pipeline tlsVerify parameter (knative#3484)

When --registry-insecure is used, set tlsVerify=false in the Tekton
pipeline to allow buildah to push to registries with self-signed certs.

* Remember `--registry-insecure` setting on recurring runs (knative#3490)

* Revert "Pass --registry-insecure flag to Tekton pipeline tlsVerify parameter (knative#3484)"

This reverts commit da21fa2.

* Remember `--registry-insecure` setting on recurring runs

* Run `make schema-generate`

* Simplify parsing

* Extend description of flag, that values are persited

* Add warning, when registry changed but insecure-registry is still set to true

* Fix typo

* Rerun update-codegen

* Cleanup test names

* Add unit test for WarnRegistryInsecureChange

* Run schema-generate

* Update pkg/functions/client.go

Co-authored-by: David Fridrich <49119790+gauron99@users.noreply.github.com>

* Update pkg/functions/client.go

Co-authored-by: David Fridrich <49119790+gauron99@users.noreply.github.com>

* Update pkg/config/config.go

Co-authored-by: David Fridrich <49119790+gauron99@users.noreply.github.com>

* Update pkg/config/config.go

Co-authored-by: David Fridrich <49119790+gauron99@users.noreply.github.com>

* Fix format issues (goimport and gofmt)

* Refactor registry-insecure unit test to make it available for the Deploy tests too

* Refactor GlobalConfig.WarnRegistryInsecureChange() method to function in cmd package

---------

Co-authored-by: David Fridrich <49119790+gauron99@users.noreply.github.com>

* Ran `./hack/update-codegen.sh`

---------

Co-authored-by: David Fridrich <49119790+gauron99@users.noreply.github.com>
@knative-prow knative-prow bot requested review from dsimansk and jrangelramos April 7, 2026 15:03
@knative-prow knative-prow bot added the size/XL 🤖 PR changes 500-999 lines, ignoring generated files. label Apr 7, 2026
@gauron99
Copy link
Copy Markdown
Contributor

gauron99 commented Apr 7, 2026

Sorry, this is missing (most likely) a cloudevents python lib issue. There was a new cloudevents version release which has some breaking changes. You need to go into the python cloudevents template and pin the version to <2 (major) because we did not have it pinned at all.

@gauron99
Copy link
Copy Markdown
Contributor

gauron99 commented Apr 7, 2026

for reference #3524

@creydr
Copy link
Copy Markdown
Member Author

creydr commented Apr 7, 2026

Sorry, this is missing (most likely) a cloudevents python lib issue. There was a new cloudevents version release which has some breaking changes. You need to go into the python cloudevents template and pin the version to <2 (major) because we did not have it pinned at all.

Ah ok. Thanks for the hint. Updated it 🤞

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

❌ Patch coverage is 95.65217% with 1 line in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (release-1.20@7052943). Learn more about missing BASE report.

Files with missing lines Patch % Lines
pkg/pipelines/tekton/templates.go 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##             release-1.20    #3588   +/-   ##
===============================================
  Coverage                ?   60.20%           
===============================================
  Files                   ?      134           
  Lines                   ?    13533           
  Branches                ?        0           
===============================================
  Hits                    ?     8147           
  Misses                  ?     4415           
  Partials                ?      971           
Flag Coverage Δ
e2e-tests 41.61% <60.86%> (?)
integration-tests 54.78% <95.65%> (?)
unit-tests 46.84% <91.30%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@creydr
Copy link
Copy Markdown
Member Author

creydr commented Apr 8, 2026

kubernetes version \"1.32.0\" is not compatible, need at least \"1.33.0-0\" (this can be overridden with the env var \"KUBERNETES_MIN_VERSION\")

This is because Knative components were updated by update-codegen.sh to 1.22 in aaad012

@gauron99 gauron99 changed the title [release-1.20] Remember --registry-insecure setting on recurring runs [release-1.20] Remember --registry-insecure setting on recurring runs + backport fixes Apr 8, 2026
@creydr
Copy link
Copy Markdown
Member Author

creydr commented Apr 9, 2026

Closing as discussed

@creydr creydr closed this Apr 9, 2026
@creydr creydr reopened this Apr 13, 2026
@knative-prow knative-prow bot added size/XXL 🤖 PR changes 1000+ lines, ignoring generated files. and removed size/XL 🤖 PR changes 500-999 lines, ignoring generated files. labels Apr 13, 2026
@creydr creydr force-pushed the cp-remember-registry-insecure-flag-1.20 branch 2 times, most recently from 38b2dee to b610705 Compare April 13, 2026 17:52
@knative-prow knative-prow bot added size/XL 🤖 PR changes 500-999 lines, ignoring generated files. and removed size/XXL 🤖 PR changes 1000+ lines, ignoring generated files. labels Apr 13, 2026
@creydr creydr force-pushed the cp-remember-registry-insecure-flag-1.20 branch from b610705 to 5feb609 Compare April 13, 2026 18:02
@knative-prow knative-prow bot added size/XXL 🤖 PR changes 1000+ lines, ignoring generated files. and removed size/XL 🤖 PR changes 500-999 lines, ignoring generated files. labels Apr 13, 2026
@creydr creydr force-pushed the cp-remember-registry-insecure-flag-1.20 branch from 5feb609 to 56d9970 Compare April 13, 2026 18:09
@knative-prow knative-prow bot added size/XL 🤖 PR changes 500-999 lines, ignoring generated files. and removed size/XXL 🤖 PR changes 1000+ lines, ignoring generated files. labels Apr 13, 2026
@creydr creydr force-pushed the cp-remember-registry-insecure-flag-1.20 branch from 56d9970 to e625281 Compare April 13, 2026 18:17
@creydr creydr force-pushed the cp-remember-registry-insecure-flag-1.20 branch from e625281 to 426dd5d Compare April 13, 2026 18:22
creydr added 3 commits April 13, 2026 21:06
The test was creating a broken Procfile instead of a working main.go,
causing the pipeline build to fail and the test to timeout.
@creydr
Copy link
Copy Markdown
Member Author

creydr commented Apr 13, 2026

/cc @matejvasek @gauron99

@knative-prow knative-prow bot requested review from gauron99 and matejvasek April 13, 2026 20:27
@matejvasek
Copy link
Copy Markdown
Contributor

/approve
/lgtm

@knative-prow knative-prow bot added the lgtm 🤖 PR is ready to be merged. label Apr 13, 2026
@knative-prow
Copy link
Copy Markdown

knative-prow bot commented Apr 13, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: creydr, matejvasek

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow knative-prow bot added the approved 🤖 PR has been approved by an approver from all required OWNERS files. label Apr 13, 2026
@knative-prow knative-prow bot merged commit d4f2a40 into knative:release-1.20 Apr 14, 2026
52 of 57 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved 🤖 PR has been approved by an approver from all required OWNERS files. lgtm 🤖 PR is ready to be merged. size/XL 🤖 PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants