Skip to content

Conversation

@koba1t
Copy link
Member

@koba1t koba1t commented Aug 16, 2025

from go v1.22.7 to v1.24.6

@k8s-ci-robot
Copy link
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Aug 16, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: koba1t

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

The pull request process is described here

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

@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 16, 2025
@k8s-ci-robot
Copy link
Contributor

This PR has multiple commits, and the default merge method is: merge.
You can request commits to be squashed using the label: tide/merge-method-squash

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@koba1t koba1t added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Aug 16, 2025
@koba1t koba1t force-pushed the upgrade/go.1.24.6 branch from 3ed4cac to 5d95d2d Compare August 16, 2025 23:30
@k8s-ci-robot k8s-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 17, 2025
@koba1t koba1t force-pushed the upgrade/go.1.24.6 branch from c568382 to 0d334ef Compare August 17, 2025 11:34
@koba1t koba1t force-pushed the upgrade/go.1.24.6 branch from 0d334ef to 64f385d Compare August 17, 2025 11:41
@koba1t koba1t force-pushed the upgrade/go.1.24.6 branch from 64f385d to 7591346 Compare August 17, 2025 11:42
@koba1t koba1t marked this pull request as ready for review August 17, 2025 11:50
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 17, 2025
@koba1t
Copy link
Member Author

koba1t commented Aug 17, 2025

/retest

@koba1t koba1t force-pushed the upgrade/go.1.24.6 branch from c01a93a to a9bb490 Compare August 17, 2025 12:14
@koba1t
Copy link
Member Author

koba1t commented Aug 17, 2025

/test kustomize-presubmit-master

@koba1t
Copy link
Member Author

koba1t commented Aug 17, 2025

/hold wait a review from some reviewer/approver

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 17, 2025
@@ -1,6 +1,4 @@
// Code generated by pluginator on AnnotationsTransformer; DO NOT EDIT.
// pluginator {(devel) unknown }
Copy link
Member Author

@koba1t koba1t Aug 17, 2025

Choose a reason for hiding this comment

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

I removed pluginator's version line.
Since we're developing in the same repo, it should always be devel.

Copy link
Member

@stormqueen1990 stormqueen1990 left a comment

Choose a reason for hiding this comment

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

/lgtm
/unhold

Args: cobra.ExactArgs(1),
RunE: func(cmd *cobra.Command, args []string) error {
if err := os.WriteFile(filepath.Join(args[0], "Dockerfile"), []byte(`FROM public.ecr.aws/docker/library/golang:1.22.7-bullseye as builder
if err := os.WriteFile(filepath.Join(args[0], "Dockerfile"), []byte(`FROM public.ecr.aws/docker/library/golang:1.24.6-bullseye as builder
Copy link
Member

Choose a reason for hiding this comment

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

Perhaps we could make this a constant somewhere?

Copy link
Member Author

Choose a reason for hiding this comment

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

There are multiple locations in go program where container image versions are specified, but since the packages are separated, making these constant values might be challenging.
https://github.com/search?q=repo%3Akubernetes-sigs%2Fkustomize++public.ecr.aws%2Fdocker%2Flibrary%2Fgolang%3A1.24.6-bullseye+language%3AGo&type=code&l=Go

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 17, 2025
@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 17, 2025
@k8s-ci-robot k8s-ci-robot merged commit 87d0629 into kubernetes-sigs:master Aug 17, 2025
11 checks passed
@koba1t koba1t deleted the upgrade/go.1.24.6 branch August 17, 2025 21:36
Copy link

@therealdwright therealdwright left a comment

Choose a reason for hiding this comment

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

Could you please cut a release of kustomize to get this out?

tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Nov 14, 2025
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [kubernetes-sigs/kustomize](https://github.com/kubernetes-sigs/kustomize) | minor | `v5.7.1` -> `v5.8.0` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>kubernetes-sigs/kustomize (kubernetes-sigs/kustomize)</summary>

### [`v5.8.0`](https://github.com/kubernetes-sigs/kustomize/releases/tag/kustomize/v5.8.0)

[Compare Source](kubernetes-sigs/kustomize@kustomize/v5.7.1...kustomize/v5.8.0)

### Highlights

##### implements to replacements value in the structured data

Now, We can edit yaml/json in yaml manifests with replacements transformer.
See [#&#8203;5679](kubernetes-sigs/kustomize#5679)

##### For example

```yaml

## source
apiVersion: v1
kind: ConfigMap
metadata:
  name: source-configmap
data:
  HOSTNAME: www.example.com
---
apiVersion: v1
kind: ConfigMap
metadata:
  name: target-configmap
data:
  config.json: |-
    {"config": {
      "id": "42",
      "hostname": "REPLACE_TARGET_HOSTNAME"
    }}
```

```yaml

## replacement
replacements:
- source:
    kind: ConfigMap
    name: source-configmap
    fieldPath: data.HOSTNAME
  targets:
  - select:
      kind: ConfigMap
      name: target-configmap
    fieldPaths:
    - data.config\.json.config.hostname
```

##### fix: Propagate Namespace correctly to Helm

The long-standing bug where kustomize's namespace transformer did not pass namespaces to helmCharts has been fixed.
See [#&#8203;5940](kubernetes-sigs/kustomize#5940)

##### For example

```yaml

## define namespace
namespace: any-namespace

helmCharts:
- name: minecraft
  repo: https://kubernetes-charts.storage.googleapis.com
  version: v1.2.0
  # namespace: any-namespace   ## propagates without additional namespace specific
  valuesFile: values.yaml
```

#### Feature

[#&#8203;5679](kubernetes-sigs/kustomize#5679): implements to replacements value in the structured data
[#&#8203;5863](kubernetes-sigs/kustomize#5863): Add regex support for Replacement selectors
[#&#8203;5930](kubernetes-sigs/kustomize#5930): feat: add PatchArgs API type to populate patch options

#### fix

[#&#8203;5940](kubernetes-sigs/kustomize#5940): fix: Propagate Namespace correctly to Helm
[#&#8203;5971](kubernetes-sigs/kustomize#5971): fix: performance recession when propagating namespace to helm
[#&#8203;5942](kubernetes-sigs/kustomize#5942): fix fnplugin storagemounts validation
[#&#8203;5958](kubernetes-sigs/kustomize#5958): fix: make AbsorbAll conflict error more verbose
[#&#8203;5961](kubernetes-sigs/kustomize#5961): refactor: nested format string
[#&#8203;5967](kubernetes-sigs/kustomize#5967): Fix infinite loop in HTTP client by validating URLs before requests
[#&#8203;5985](kubernetes-sigs/kustomize#5985): fix(kyaml/yaml): minor nil safety fix for RNode.Content etc
[#&#8203;5991](kubernetes-sigs/kustomize#5991): Fix duplicate key error when adding multiple labels with --without-selector

#### Dependencies

[#&#8203;5962](kubernetes-sigs/kustomize#5962): chore: update dependencies from security alert
[#&#8203;5959](kubernetes-sigs/kustomize#5959): update go 1.24.6

#### chore

[#&#8203;6007](kubernetes-sigs/kustomize#6007): Update kyaml to v0.21.0
[#&#8203;6008](kubernetes-sigs/kustomize#6008): Update cmd/config to v0.21.0
[#&#8203;6009](kubernetes-sigs/kustomize#6009): Update api to v0.21.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0MS4xNzMuMSIsInVwZGF0ZWRJblZlciI6IjQxLjE3My4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiXX0=-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants