quilkin icon indicating copy to clipboard operation
quilkin copied to clipboard

Documentation for xDS Delta API

Open markmandel opened this issue 2 years ago • 4 comments

With the implementation of #792 we should update our xDS documentation to reflect this change.

Just digging into the implementation, these are the pages that need updating:

  • https://googleforgames.github.io/quilkin/main/book/services/xds.html
  • https://googleforgames.github.io/quilkin/main/book/services/xds/metrics.html

I don't think there is anything else?

markmandel avatar Dec 14 '23 21:12 markmandel

I think it depends on #884 because the changes in that issue would change the documentation, since we would change the types, and thus the documentation, and I'm reluctant to write docs that'd only be valid for one commit.

XAMPPRocky avatar Dec 15 '23 08:12 XAMPPRocky

I think it depends on #884 because the changes in that issue would change the documentation, since we would change the types, and thus the documentation, and I'm reluctant to write docs that'd only be valid for one commit.

I guess the only mitigating factor would be that if we want to cut a release, we would have an undocumented API surface at release time -- but we can make that decision when we are ready to make a new release 👍🏻

markmandel avatar Jan 03 '24 04:01 markmandel

I guess the only mitigating factor would be that if we want to cut a release

I think regardless of this specific feature, I think we want to move away from instant feature stability from version releases. As once we've started shipping "stable" versions, we're still going to want to merge features incrementally.

XAMPPRocky avatar Jan 08 '24 08:01 XAMPPRocky

I think regardless of this specific feature, I think we want to move away from instant feature stability from version releases. As once we've started shipping "stable" versions, we're still going to want to merge features incrementally.

100% Agreed (we use feature gates in Agones for exactly the same reason) - but even some kind of non-stable documentation with a warning of stability guarantee, would be useful if we do cut a release, so that end users can at least play with it and give feedback, etc.

For example, this is what we do on Agones: https://agones.dev/site/docs/guides/player-tracking/

Not wedded to the format, but it's definitely useful for exactly the reasons you describe 👍🏻

markmandel avatar Jan 09 '24 02:01 markmandel