Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

godoc for etcd 3.5 is missing/broken #19645

Open
4 tasks done
purpleidea opened this issue Mar 23, 2025 · 6 comments · May be fixed by #19649
Open
4 tasks done

godoc for etcd 3.5 is missing/broken #19645

purpleidea opened this issue Mar 23, 2025 · 6 comments · May be fixed by #19649
Labels
area/documentation priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. stage/triaged type/bug

Comments

@purpleidea
Copy link
Contributor

Bug report criteria

What happened?

If you browse: https://pkg.go.dev/github.com/etcd-io/etcd you see version "v3.3.27+incompatible". No idea why this shows such an old version, but this is quite confusing when trying to read the docs.

I've tried to ask the website to pull a new version but it seems to not work. I've also tried to fetch docs on the better godocs.io site: https://godocs.io/github.com/etcd-io/etcd/clientv3 but also only old versions exist.

If you could fix this, that would be great. Not sure who's at fault, probably google somehow!

Thanks!

What did you expect to happen?

Docs should see v3.5.20

How can we reproduce it (as minimally and precisely as possible)?

See above

Anything else we need to know?

No response

Etcd version (please run commands below)

N/A

Etcd configuration (command line flags or environment variables)

paste your configuration here

Etcd debug information (please run commands below, feel free to obfuscate the IP address or FQDN in the output)

$ etcdctl member list -w table
# paste output here

$ etcdctl --endpoints=<member list> endpoint status -w table
# paste output here

Relevant log output

@ghouscht
Copy link
Member

ghouscht commented Mar 23, 2025

The link you’re looking for is: https://pkg.go.dev/go.etcd.io/etcd/v3

The link you posted is pointing to the „v0“ (in go module terms) version of the code, this is also the reason why it has the +incompatible suffix.

@purpleidea
Copy link
Contributor Author

This doesn't seem to show the API docs. I'm following the go documentation badge in the etcd README to get the "bad" link. If you send me the correct one I'll update it with a patch.

@ivanvc
Copy link
Member

ivanvc commented Mar 24, 2025

Thanks, @ghouscht, for pointing out the correct location in the documentation.

@purpleidea, thanks for raising this issue. It's right; the link is broken because of how 3.5 split the project into submodules. I believe the API documentation you're looking for is at https://pkg.go.dev/go.etcd.io/etcd/api/v3. We could update the badge URL to match what Thomas suggested. Maybe we should have a section in the README pointing to the godoc for each of the modules:

@purpleidea
Copy link
Contributor Author

These are the links I was looking for, thanks! In particular: https://pkg.go.dev/go.etcd.io/etcd/client/v3#pkg-index and also https://pkg.go.dev/go.etcd.io/etcd/server/v3#section-sourcefiles for example.

Want me to send a patch? Not sure what the consensus about what the patch should look like is, but at the moment it's definitely something that needs fixing.

Cheers!

@ivanvc
Copy link
Member

ivanvc commented Mar 24, 2025

@purpleidea, if you're able, please send the patch. It would be greatly appreciated. I suggest replacing the Go doc badge with https://pkg.go.dev/go.etcd.io/etcd/v3 and adding a section in the README pointing to the submodule documentation pages. Thanks!

@ivanvc ivanvc added area/documentation stage/triaged priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. labels Mar 24, 2025
@purpleidea purpleidea linked a pull request Mar 24, 2025 that will close this issue
@purpleidea
Copy link
Contributor Author

Here you go: #19649

purpleidea added a commit to purpleidea/etcd that referenced this issue Mar 25, 2025
The current links point to the old v3.3 docs which aren't relevant
anymore. This also adds simple links to the current docs.

This fixes etcd-io#19645

Signed-off-by: James Shubin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. stage/triaged type/bug
Development

Successfully merging a pull request may close this issue.

3 participants