Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Jun 23, 2025

https://pkg.go.dev/github.com/ceph/ceph-csi does not list the current
v3.14.0 release. It seems to come from the fact that a v3 release of a
Go module expects a v3/go.mod file in the repository. Instead of moving
all files from the root of the repository to a new v3 directory, make
a symlink so that https://github.com/ceph/ceph-csi/v3/go.mod resolves
correctly (eventhough the go.mod does not reference /v3).

Steps to add a package (also has a suggestion for a version):

The Go proxy protocol is used to track available versions, and this would be a request for the v3.14.0 version:

  • https://proxy.golang.org/github.com/ceph/ceph-csi/v3/@v/v3.14.0.info

That results in an error like

not found: github.com/ceph/ceph-csi/[email protected]: invalid version: go.mod has non-.../v3 module path "github.com/ceph/ceph-csi" (and .../v3/go.mod does not exist) at revision v3.14.0

Adding the v3 symlink to the root of the directory might be sufficient to get the documentation generated.


This is an automatic backport of pull request #5364 done by Mergify.

@mergify mergify bot added ok-to-test Label to trigger E2E tests ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures component/docs Issues and PRs related to documentation labels Jun 23, 2025
@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.32

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.32

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.30

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.31

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.30

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.32

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.31

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.30

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.31

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Jun 23, 2025
@Madhu-1 Madhu-1 requested a review from nixpanic June 23, 2025 07:23
@Madhu-1
Copy link
Collaborator

Madhu-1 commented Jun 23, 2025

@Mergifyio rebase

https://pkg.go.dev/github.com/ceph/ceph-csi does not list the current
v3.14.0 release. It seems to come from the fact that a v3 release of a
Go module expects a v3/go.mod file in the repository. Instead of moving
all files from the root of the repository to a new `v3` directory, make
a symlink so that https://github.com/ceph/ceph-csi/v3/go.mod resolves
correctly (eventhough the go.mod does not reference /v3).

Signed-off-by: Niels de Vos <[email protected]>
(cherry picked from commit 0778a37)
@mergify
Copy link
Contributor Author

mergify bot commented Jun 23, 2025

rebase

✅ Branch has been successfully rebased

@Madhu-1 Madhu-1 force-pushed the mergify/bp/release-v3.14/pr-5364 branch from 76eb15e to 7a500da Compare June 23, 2025 07:58
@Madhu-1 Madhu-1 added the ready-to-merge This PR is ready to be merged and it doesn't need second review (backports only) label Jun 23, 2025
@mergify mergify bot merged commit 063c186 into release-v3.14 Jun 23, 2025
18 checks passed
@mergify mergify bot deleted the mergify/bp/release-v3.14/pr-5364 branch June 23, 2025 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures component/docs Issues and PRs related to documentation ready-to-merge This PR is ready to be merged and it doesn't need second review (backports only)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants