Skip to content

chore(chart-deps): update knative-operator to version v1.22.3 - #3431

Closed
svcAPLBot wants to merge 29 commits into
mainfrom
ci-update-knative-operator-to-v1.22.3
Closed

chore(chart-deps): update knative-operator to version v1.22.3#3431
svcAPLBot wants to merge 29 commits into
mainfrom
ci-update-knative-operator-to-v1.22.3

Conversation

@svcAPLBot

Copy link
Copy Markdown
Contributor

This PR updates the dependency knative-operator to version v1.22.3.

@svcAPLBot svcAPLBot added the chart-deps Auto generated helm chart dependencies label Jul 8, 2026
Copilot AI lite review requested due to automatic review settings July 29, 2026 13:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Knative Operator chart dependency and related metadata to target Knative Operator v1.22.3, aligning the chart image tags and app catalog version with the intended upstream release.

Changes:

  • Bump Knative Operator and webhook image tags to v1.22.3.
  • Update the vendored knative-operator chart version metadata to 1.22.3.
  • Update the app catalog (apps.yaml) to reflect 1.22.3.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
charts/knative-operator/values.yaml Updates operator + webhook image tags to v1.22.3.
charts/knative-operator/Chart.yaml Bumps the local chart version to 1.22.3.
chart/chart-index/Chart.yaml Updates the knative-operator dependency version (but currently uses a likely invalid v-prefixed chart version).
apps.yaml Updates Knative appVersion metadata to 1.22.3.

Copilot AI review requested due to automatic review settings July 29, 2026 14:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings July 30, 2026 13:41
Copilot AI review requested due to automatic review settings August 3, 2026 14:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

Suppressed comments (1)

apps.yaml:111

  • apps.yaml lists this entry as "Knative Operator" but the repo points to knative/serving, which is inconsistent with the chart/dependency being updated (knative/operator). This can mislead users looking for the upstream source for the operator.
    repo: https://github.com/knative/serving

Copilot AI review requested due to automatic review settings August 4, 2026 07:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings August 4, 2026 07:14

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings August 4, 2026 07:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings August 4, 2026 09:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

@merll
merll marked this pull request as ready for review August 4, 2026 11:54
@svcAPLBot

Copy link
Copy Markdown
Contributor Author

Comparison of Helm chart templating output:

# knative-operator/templates/crds/knativeeventings.yaml

# knative-operator/templates/crds/knativeservings.yaml

# knative-operator/templates/operator.yaml

@@ spec.template.spec.containers.operator-webhook.image @@
# apps/v1/Deployment/knative-operator/operator-webhook
! ± value change
- gcr.io/knative-releases/knative.dev/operator/cmd/webhook:v1.22.2
+ gcr.io/knative-releases/knative.dev/operator/cmd/webhook:v1.22.3

@@ spec.template.spec.containers.knative-operator.image @@
# apps/v1/Deployment/knative-operator/knative-operator
! ± value change
- gcr.io/knative-releases/knative.dev/operator/cmd/operator:v1.22.2
+ gcr.io/knative-releases/knative.dev/operator/cmd/operator:v1.22.3

# knative-operator/templates/rbac/eventing-operator-role.yaml

# knative-operator/templates/rbac/serving-operator-role.yaml

# otomi-api/templates/core-config.yaml

@@ data.core.yaml @@
! ± value change in multiline text (one insert, one deletion)
  adminApps:
  - deps:
    - prometheus
    name: alertmanager
  
  [307 lines unchanged)]
  
    knative:
      about: Knative Serving builds on Kubernetes to support deploying and serving of
        applications and functions as serverless containers. Serving is easy to get
        started with and scales to support advanced scenarios.
-     appVersion: 1.22.2
+     appVersion: 1.22.3
      chartName: knative-operator
      integration: Knative serving can be activated to deliver Container-as-a-Service
        (CaaS) functionality with a scale-to-zero option. It can be compared to Functions-as-a-service
        (FaaS) but is container oriented, and takes only one manifest to configure an
  
  [278 lines unchanged)]
  
    path: /explore?orgId=1&left=%7B"datasource":"loki","queries":%5B%7B"refId":"A","expr":"","queryType":"range","datasource":%7B"type":"loki","uid":"loki"%7D%7D%5D,"range":%7B"from":"now-1h","to":"now"%7D%7D
    useHost: grafana
  - name: tekton
    ownHost: true

# otomi-api/templates/deployment.yaml

# rabbitmq-cluster-operator/templates/messaging-topology-operator/validating-webhook-configuration.yaml

# values-repo.yaml

Copilot AI review requested due to automatic review settings August 5, 2026 10:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

@merll merll closed this Aug 7, 2026
@merll
merll deleted the ci-update-knative-operator-to-v1.22.3 branch August 7, 2026 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chart-deps Auto generated helm chart dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants