Skip to content

Commit 24ae34e

Browse files
upgrade to latest dependencies
bumping knative.dev/client-pkg 8478498...afd7c8d: > afd7c8d Update community files (# 201) bumping knative.dev/hack e92a16a...c142b48: > c142b48 Refactor release script to gh CLI (# 408) > f8be0cc Update community files (# 407) Signed-off-by: Knative Automation <[email protected]>
1 parent bffeb7a commit 24ae34e

File tree

4 files changed

+30
-32
lines changed

4 files changed

+30
-32
lines changed

go.mod

+2-2
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@ require (
77
github.com/docker/go-connections v0.5.0
88
github.com/spf13/cobra v1.7.0
99
gotest.tools/v3 v3.3.0
10-
knative.dev/client-pkg v0.0.0-20241223132438-847849858508
11-
knative.dev/hack v0.0.0-20241227080210-e92a16ae0893
10+
knative.dev/client-pkg v0.0.0-20250109131824-afd7c8d9de7b
11+
knative.dev/hack v0.0.0-20250116150306-c142b4835bc5
1212

1313
)
1414

go.sum

+4-4
Original file line numberDiff line numberDiff line change
@@ -163,10 +163,10 @@ k8s.io/klog/v2 v2.130.1 h1:n9Xl7H1Xvksem4KFG4PYbdQCQxqc/tTUyrgXaOhHSzk=
163163
k8s.io/klog/v2 v2.130.1/go.mod h1:3Jpz1GvMt720eyJH1ckRHK1EDfpxISzJ7I9OYgaDtPE=
164164
k8s.io/utils v0.0.0-20240711033017-18e509b52bc8 h1:pUdcCO1Lk/tbT5ztQWOBi5HBgbBP1J8+AsQnQCKsi8A=
165165
k8s.io/utils v0.0.0-20240711033017-18e509b52bc8/go.mod h1:OLgZIPagt7ERELqWJFomSt595RzquPNLL48iOWgYOg0=
166-
knative.dev/client-pkg v0.0.0-20241223132438-847849858508 h1:/Ht6zpdIDciiwFr48iRDmY2Ifwo7xmJQnTX2DI6ER+k=
167-
knative.dev/client-pkg v0.0.0-20241223132438-847849858508/go.mod h1:1Wp8+rz7JT3c17r0+2IdxWjkNvEZ/4/AHywaV12nq3I=
168-
knative.dev/hack v0.0.0-20241227080210-e92a16ae0893 h1:zy7LwNJ2S7obPMHVAtxQgZPXxBTZzoxHbtb6uhxOl7Q=
169-
knative.dev/hack v0.0.0-20241227080210-e92a16ae0893/go.mod h1:R0ritgYtjLDO9527h5vb5X6gfvt5LCrJ55BNbVDsWiY=
166+
knative.dev/client-pkg v0.0.0-20250109131824-afd7c8d9de7b h1:LqysBUe0hLjZhVNFRlSaOjQ8SRJs2K+eMt6YlTvIJJw=
167+
knative.dev/client-pkg v0.0.0-20250109131824-afd7c8d9de7b/go.mod h1:1Wp8+rz7JT3c17r0+2IdxWjkNvEZ/4/AHywaV12nq3I=
168+
knative.dev/hack v0.0.0-20250116150306-c142b4835bc5 h1:CwLVoSblWmfaIkgRFX2LcEA0uYZo+nOhdPatsz+QL9w=
169+
knative.dev/hack v0.0.0-20250116150306-c142b4835bc5/go.mod h1:R0ritgYtjLDO9527h5vb5X6gfvt5LCrJ55BNbVDsWiY=
170170
knative.dev/pkg v0.0.0-20241021183759-9b9d535af5ad h1:Nrjtr2H168rJeamH4QdyLMV1lEKHejNhaj1ymgQMfLk=
171171
knative.dev/pkg v0.0.0-20241021183759-9b9d535af5ad/go.mod h1:StJI72GWcm/iErmk4RqFJiOo8RLbVqPbHxUqeVwAzeo=
172172
sigs.k8s.io/json v0.0.0-20221116044647-bc3834ca7abd h1:EDPBXCAspyGV4jQlpZSudPeMmr1bNJefnuqLsRAsHZo=

vendor/knative.dev/hack/release.sh

+22-24
Original file line numberDiff line numberDiff line change
@@ -90,11 +90,10 @@ export GOFLAGS="-ldflags=-s -ldflags=-w"
9090
export GITHUB_TOKEN=""
9191
readonly IMAGES_REFS_FILE="${IMAGES_REFS_FILE:-$(mktemp -d)/images_refs.txt}"
9292

93-
# Convenience function to run the hub tool.
94-
# Parameters: $1..$n - arguments to hub.
95-
function hub_tool() {
96-
# Pinned to SHA because of https://github.com/github/hub/issues/2517
97-
go_run github.com/github/hub/v2@363513a "$@"
93+
# Convenience function to run the GitHub CLI tool `gh`.
94+
# Parameters: $1..$n - arguments to gh.
95+
function gh_tool() {
96+
go_run github.com/cli/cli/v2/cmd/[email protected] "$@"
9897
}
9998

10099
# Shortcut to "git push" that handles authentication.
@@ -193,7 +192,7 @@ function prepare_dot_release() {
193192
# Support tags in two formats
194193
# - knative-v1.0.0
195194
# - v1.0.0
196-
releases="$(hub_tool release | cut -d '-' -f2)"
195+
releases="$(gh_tool release list --json tagName --jq '.[].tagName' | cut -d '-' -f2)"
197196
echo "Current releases are: ${releases}"
198197
[[ $? -eq 0 ]] || abort "cannot list releases"
199198
# If --release-branch passed, restrict to that release
@@ -218,7 +217,7 @@ function prepare_dot_release() {
218217
# Ensure there are new commits in the branch, otherwise we don't create a new release
219218
setup_branch
220219
# Use the original tag (ie. potentially with a knative- prefix) when determining the last version commit sha
221-
local github_tag="$(hub_tool release | grep "${last_version}")"
220+
local github_tag="$(gh_tool release list --json tagName --jq '.[].tagName' | grep "${last_version}")"
222221
local last_release_commit="$(git rev-list -n 1 "${github_tag}")"
223222
local last_release_commit_filtered="$(git rev-list --invert-grep --grep "\[skip-dot-release\]" -n 1 "${github_tag}")"
224223
local release_branch_commit="$(git rev-list -n 1 upstream/"${RELEASE_BRANCH}")"
@@ -239,7 +238,7 @@ function prepare_dot_release() {
239238
# If --release-notes not used, copy from the latest release
240239
if [[ -z "${RELEASE_NOTES}" ]]; then
241240
RELEASE_NOTES="$(mktemp)"
242-
hub_tool release show -f "%b" "${github_tag}" > "${RELEASE_NOTES}"
241+
gh_tool release view "${github_tag}" --json "body" --jq '.body' > "${RELEASE_NOTES}"
243242
echo "Release notes from ${last_version} copied to ${RELEASE_NOTES}"
244243
fi
245244
}
@@ -640,18 +639,12 @@ function set_latest_to_highest_semver() {
640639

641640
local last_version release_id # don't combine with assignment else $? will be 0
642641

643-
last_version="$(hub_tool -p release | cut -d'-' -f2 | grep '^v[0-9]\+\.[0-9]\+\.[0-9]\+$'| sort -r -V | head -1)"
642+
last_version="$(gh_tool release list --json tagName --jq '.[].tagName' | cut -d'-' -f2 | grep '^v[0-9]\+\.[0-9]\+\.[0-9]\+$'| sort -r -V | head -1)"
644643
if ! [[ $? -eq 0 ]]; then
645644
abort "cannot list releases"
646645
fi
647-
648-
release_id="$(hub_tool api "/repos/${ORG_NAME}/${REPO_NAME}/releases/tags/knative-${last_version}" | jq .id)"
649-
if [[ $? -ne 0 ]]; then
650-
abort "cannot get relase id from github"
651-
fi
652-
653-
hub_tool api --method PATCH "/repos/${ORG_NAME}/${REPO_NAME}/releases/$release_id" \
654-
-F make_latest=true > /dev/null || abort "error setting $last_version to 'latest'"
646+
647+
gh_tool release edit "knative-${last_version}" --latest > /dev/null || abort "error setting $last_version to 'latest'"
655648
echo "Github release ${last_version} set as 'latest'"
656649
}
657650

@@ -742,12 +735,14 @@ function publish_to_github() {
742735
local description="$(mktemp)"
743736
local attachments_dir="$(mktemp -d)"
744737
local commitish=""
738+
local target_branch=""
745739
local github_tag="knative-${TAG}"
746740

747741
# Copy files to a separate dir
742+
# shellcheck disable=SC2068
748743
for artifact in $@; do
749744
cp ${artifact} "${attachments_dir}"/
750-
attachments+=("--attach=${artifact}#$(basename ${artifact})")
745+
attachments+=("${artifact}#$(basename ${artifact})")
751746
done
752747
echo -e "${title}\n" > "${description}"
753748
if [[ -n "${RELEASE_NOTES}" ]]; then
@@ -774,13 +769,16 @@ function publish_to_github() {
774769
git tag -a "${github_tag}" -m "${title}"
775770
git_push tag "${github_tag}"
776771

777-
[[ -n "${RELEASE_BRANCH}" ]] && commitish="--commitish=${RELEASE_BRANCH}"
772+
[[ -n "${RELEASE_BRANCH}" ]] && target_branch="--target=${RELEASE_BRANCH}"
778773
for i in {2..0}; do
779-
hub_tool release create \
780-
${attachments[@]} \
781-
--file="${description}" \
782-
"${commitish}" \
783-
"${github_tag}" && return 0
774+
# shellcheck disable=SC2068
775+
gh_tool release create \
776+
"${github_tag}" \
777+
--title "${title}" \
778+
--notes-file "${description}" \
779+
"${target_branch}" \
780+
${attachments[@]} && return 0
781+
784782
if [[ "${i}" -gt 0 ]]; then
785783
echo "Error publishing the release, retrying in 15s..."
786784
sleep 15

vendor/modules.txt

+2-2
Original file line numberDiff line numberDiff line change
@@ -128,9 +128,9 @@ gotest.tools/v3/internal/assert
128128
gotest.tools/v3/internal/difflib
129129
gotest.tools/v3/internal/format
130130
gotest.tools/v3/internal/source
131-
# knative.dev/client-pkg v0.0.0-20241223132438-847849858508
131+
# knative.dev/client-pkg v0.0.0-20250109131824-afd7c8d9de7b
132132
## explicit; go 1.22.0
133133
knative.dev/client-pkg/pkg/plugin
134-
# knative.dev/hack v0.0.0-20241227080210-e92a16ae0893
134+
# knative.dev/hack v0.0.0-20250116150306-c142b4835bc5
135135
## explicit; go 1.21
136136
knative.dev/hack

0 commit comments

Comments
 (0)