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

docs: new features in 3.6 #13739

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

docs: new features in 3.6 #13739

wants to merge 5 commits into from

Conversation

Joibel
Copy link
Member

@Joibel Joibel commented Oct 11, 2024

Motivation

Keep users of Argo Workflows well informed of the new features in each release.

Until the next RC is out the feature list is subject to change. This is correct for RC-2 only. This PR will be updated as RC-3 is cut and merged then.

Modifications

Add a features-3.6.md (with the intention that each version will have its own page) with a concise list of all features based around the blog posting. Aiming for less celebratory, more factual.

Update the new features modal for the UI in preparation for 3.6 with a similar summary as per the Kubecon video.

Verification

Validated modal visually and by clicking the links. RTD 3.6 is not yet a thing, but changing to 3.5 for the cron-workflows link works as expected.
image

Validated new features page here in markdown too.

@Joibel Joibel marked this pull request as ready for review October 11, 2024 13:19
@agilgur5 agilgur5 added the area/docs Incorrect, missing, or mistakes in docs label Oct 12, 2024
@agilgur5 agilgur5 self-assigned this Oct 12, 2024
@agilgur5 agilgur5 added this to the v3.6.0 milestone Oct 12, 2024
@Joibel Joibel requested a review from agilgur5 October 15, 2024 08:08
Copy link
Member

@tico24 tico24 left a comment

Choose a reason for hiding this comment

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

Some nits

docs/features-3.6.md Outdated Show resolved Hide resolved
docs/features-3.6.md Outdated Show resolved Hide resolved
Copy link
Member

@tico24 tico24 left a comment

Choose a reason for hiding this comment

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

I failed to push the right button. Some nits.

Signed-off-by: Alan Clucas <alan@clucas.org>
Signed-off-by: Alan Clucas <alan@clucas.org>
@Joibel
Copy link
Member Author

Joibel commented Oct 16, 2024

I have also added the features merged since RC-2

Copy link
Member

@isubasinghe isubasinghe left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Alan Clucas <alan@clucas.org>
Signed-off-by: Alan Clucas <alan@clucas.org>
## Build and Development

* [#13000](https://github.com/argoproj/argo-workflows/pull/13000): There is now a `/retest` command for retesting PRs in Github that occasionally fail in a flakey test
* [#12867](https://github.com/argoproj/argo-workflows/pull/12867): You can supply your own HTTP client when using the go API client, allowing for adding a proxy
Copy link
Member

Choose a reason for hiding this comment

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

The API client is an SDK change, so should be under an SDKs section

Copy link
Member Author

Choose a reason for hiding this comment

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

SDK stands for software development kit, and this is under development. I don't think we need to split.

Copy link
Member

@agilgur5 agilgur5 Oct 22, 2024

Choose a reason for hiding this comment

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

"Build and Development", especially with the first item being a CI thing, makes it sound like items that are only relevant to Contributors, but the SDK is user facing. It's also called an "SDK" in the docs and in our own labels. Similarly, all contributor facing docs are under the "Developer Guide" in the docs.

I definitely don't think we should be creating new names for existing things, that's confusing at best.

SDK stands for software development kit, and this is under development

If you want to debate the difference between denotation and connotation: the connotations are not equivalent in this context, and if you were to use denotation, the entire codebase could be considered "development", so that is not a sufficient boundary

Comment on lines +74 to +76
## Build and Development

* [#13000](https://github.com/argoproj/argo-workflows/pull/13000): There is now a `/retest` command for retesting PRs in Github that occasionally fail in a flakey test
Copy link
Member

@agilgur5 agilgur5 Oct 18, 2024

Choose a reason for hiding this comment

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

dev stuff is not needed and is counter to the stated purpose of having a concise list (and there's plenty we could bikeshed about if we had a dev list)

Copy link
Member Author

Choose a reason for hiding this comment

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

This is a new feature and this list is to enable people to discover new features. It isn't needed but I'd like to celebrate it by leaving it in.

Copy link
Member

@agilgur5 agilgur5 Oct 22, 2024

Choose a reason for hiding this comment

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

It's in the blog post already, and I think that's sufficient. The blog post includes "non-concise" things.
There are also certainly other new "CI features" if we wanted to call them that, which, as I wrote above, could definitely make for more bikeshedding

Copy link
Member

Choose a reason for hiding this comment

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

It also is not specific to 3.6 either since it's CI; and that can also make for more bikeshedding

@@ -237,6 +237,7 @@ nav:
- installation.md
- releases.md
- upgrading.md
- features-3.6.md
Copy link
Member

@agilgur5 agilgur5 Oct 18, 2024

Choose a reason for hiding this comment

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

I don't think this should be a separate page per version as this will just keep increasing and eventually make it hard to see other pages in this section. It's also inconsistent with the upgrading guide and CHANGELOG, both of which are single pages (with CHANGELOG being eventually split into two per #13352). Deprecations from #13735 is also a single page. EDIT: See also #13735 (comment)

We previously discussed this together in the Oct 1st Contributor Meeting or earlier if I recall correctly, both before and after I met with CNCF Tech Writers

Copy link
Member Author

Choose a reason for hiding this comment

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

We can drop new features-3.6.md from 3.7 because they'll be in 3.6's documentation. This is partially to enable automation of features-3.7.md too.

I can't find other projects that have a mega page full of new features for all prior versions, it seems normal to have one page per feature release.

Copy link
Member

Choose a reason for hiding this comment

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

Most projects use the CHANGELOG, as we already do, or use the GH Releases page, both of which are not one page per release but one heading per release.

Internal consistency is more important in any case, and internally we do have headings per release already, as linked above.

We can drop new features-3.6.md from 3.7 because they'll be in 3.6's documentation

In that case, I would still have it as features.md so that the link /features remains the same for each version. Otherwise a user would have to duplicatively type release-3.6/features-3.6 and would be unable to use the version selector to see new features as release-3.7/features-3.6 (and so forth) would 404

docs/upgrading.md Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

relative links to the docs for some of the features here would be really helpful to be able to see them in context when reading this

docs/features-3.6.md Show resolved Hide resolved
docs/features-3.6.md Outdated Show resolved Hide resolved
docs/features-3.6.md Outdated Show resolved Hide resolved
Comment on lines +41 to +42
* [#12441](https://github.com/argoproj/argo-workflows/pull/12441): Plugins can now be stopped, so that a stopped workflow will shutdown its plugin nodes
* The OSS artifact driver:
Copy link
Member

Choose a reason for hiding this comment

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

"Plugins" and "Artifacts" sections might be good to split out so it's not just "Controller". Artifacts and Plugins also run in the Server + Executor and Agent, respectively, so not in the Controller

Copy link
Member Author

Choose a reason for hiding this comment

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

I've renamed it General. Most people don't care which component a feature happens in, it's an irrelevance to them.

Copy link
Member

@agilgur5 agilgur5 Oct 22, 2024

Choose a reason for hiding this comment

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

I don't think that's necessarily accurate -- if I use artifacts, having an "Artifacts" section would make it easy for me to see what's new with that "feature set".
Permissions are also relevant per component. But usage wise, it's less about the "architectural component" and more about the broader "feature set" (or "area" as we use in labels) like "Artifacts", "Plugins", "SSO", "SDKs", "CLI", etc, which is what I suggested above

Signed-off-by: Alan Clucas <alan@clucas.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs Incorrect, missing, or mistakes in docs area/ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants