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

Kubeflow 1.3 release doc updates #2546

Closed
Bobgy opened this issue Mar 17, 2021 · 49 comments
Closed

Kubeflow 1.3 release doc updates #2546

Bobgy opened this issue Mar 17, 2021 · 49 comments

Comments

@Bobgy
Copy link
Contributor

Bobgy commented Mar 17, 2021

Creating an issue to track updates preparing for Kubeflow 1.3 doc release.

@kubeflow-bot kubeflow-bot added this to To Do in Needs Triage Mar 17, 2021
@Bobgy
Copy link
Contributor Author

Bobgy commented Mar 17, 2021

I'm following https://github.com/kubeflow/kubeflow/blob/master/docs_dev/releasing.md#lifecycle.

I just forked current master branch as v1.2-branch, and pinned www.kubeflow.org to v1.2-branch.
It's OK to move forward with large changes preparing for the next release now.

@Bobgy
Copy link
Contributor Author

Bobgy commented Mar 17, 2021

/cc @joeliedtke @RFMVasconcelos

@rui-vas
Copy link
Contributor

rui-vas commented Mar 17, 2021

Thank you for taking on this work @Bobgy !

@rui-vas
Copy link
Contributor

rui-vas commented Mar 17, 2021

I have just created #2547 #2548 #2549 #2550 #2551 to move all platforms under distributions

cc @Bobgy @joeliedtke @animeshsingh @8bitmp3 @yanniszark

@rui-vas rui-vas moved this from Started to Blocking issue for v1.3 in Kubeflow docs restructure Mar 23, 2021
@thesuperzapper
Copy link
Member

thesuperzapper commented Mar 24, 2021

@RFMVasconcelos before we close this, I think we should make https://www.kubeflow.org/ default to master branch, as there is nothing 1.3 specific in master, (and it is much better structured).

Also we should resolve the issue about not having v1.2 in the dropdown (#2564)

@rui-vas
Copy link
Contributor

rui-vas commented Mar 24, 2021

@thesuperzapper this is part of the docs release process. I will follow these steps to have Kubeflow.org point to master and there will be v1.2 in the selector.

This is a state in transition. When we point it to master (by EOW latest) this will be resolved.

@rui-vas
Copy link
Contributor

rui-vas commented Mar 24, 2021

In fact, trying to do this now I realize this is done on Netlify which I have not played with yet, I am not sure this is googlers only. I think @Bobgy did this for us in the previous release.

I think the best thing to do is for me to get the master branch ready to be pointed to, and @Bobgy, to point kubeflow.org to master once he feels comfortable with the status.

@rui-vas
Copy link
Contributor

rui-vas commented Mar 24, 2021

Sounds good @Bobgy ?

@thesuperzapper
Copy link
Member

I think we can try and resolve #2536 before doing the master switch.

If your willing to make a PR @RFMVasconcelos?

@Bobgy
Copy link
Contributor Author

Bobgy commented Mar 27, 2021

Sure, for any netlify related steps. You can ping @joeliedtke or me about them.

@rui-vas
Copy link
Contributor

rui-vas commented Mar 29, 2021

Given the ETA for release process » Step 6: Tuesday, EOD 2021-04-06 [Pacific Time]

Should we aim at pointing the docs to master at that same time? And push to have as many improvements as possible by then?

@joeliedtke would you be able to make the necessary netlify changes by then?

thoughts? @Bobgy @thesuperzapper @yanniszark @8bitmp3 @animeshsingh @castrojo @jbottum

@Bobgy
Copy link
Contributor Author

Bobgy commented Mar 29, 2021

IIUC, I think we need to wait for at least most distributions are ready with their docs, so it will very likely be a later time than that date

@rui-vas
Copy link
Contributor

rui-vas commented Mar 29, 2021

That is a good point @Bobgy!

It does raise some concerns - what if distribution X does not have the resources to update docs within the next months? In the past we added the "Outdated" banners to guide users to what docs are outdated, while not freezing improvements.

I think that the current v1.2-branch is now at a worse state than master, so pointing docs to master would IMO add value to users and should be done sooner rather than later.

@thesuperzapper
Copy link
Member

@Bobgy We should aim to point the docs at master on or before "Step 6" EOD 2021-04-06 [Pacific Time]

Note that "Step 6" is the offical release for Kubeflow 1.3 (From the OSS community perspective), as after that date a distribution could release Kubeflow 1.3 to their users. (There is no reason to penalise the faster distributions if the slow ones aren't ready)

Also, there is a case to point at master even earlier as @RFMVasconcelos said, due to master being MUCH cleaner than v1.2-branch.

@Bobgy
Copy link
Contributor Author

Bobgy commented Mar 31, 2021

I don't understand the benefit, at that date, no distribution is ready to release and there can be issues found that require a rc1. Why would that be a date to switch to master branch?

I love the improvements in master branch, but as they are mostly information restructuring, I don't feel the urgency to release them. Can we wait until most of the next release is ready?

I think it's reasonable to set a cut date for doc when at least 2 distributions are ready. What do you think?

@thesuperzapper
Copy link
Member

@Bobgy I think it's just semantics, as some of the smaller distributions will be ready in the first few days.

This is the same as how Kuberntes does not delay their release if GKE is not ready (just as an example, not picking on GKE).

Furthermore we would have to open the debate of what a "distribution" even means, as we have no validation process right now, which is not really a discussion we need to start just before 1.3 is released.

@Bobgy
Copy link
Contributor Author

Bobgy commented Mar 31, 2021

Can I confirm if we align on this?
Doc switch to master should happen after manifests cut 1.3.0 tag.

@rui-vas
Copy link
Contributor

rui-vas commented Mar 31, 2021

@Bobgy I agree that docs should only be released once people can use them in the equivalent release.

I think this problem arrives from the fact that we're synchronizing a docs release with a docs restructuring, and so by holding off both, we keep users in a non-optimal information structure when we have a better alternative.

I don't see urgency in pointing to master from the docs release, but I do see it from the docs restructure standpoint.

I am unsure on what percentage of the docs is actually tied to a specific version of KF, I would say that the rate of change in docs has been fairly slow and that we likely will need only a few version-specific changes.

WDYT?

@thesuperzapper
Copy link
Member

@RFMVasconcelos @Bobgy I think its probably OK to move to point at master now, mostly as there is very little 1.3 specific docs, but the structure is MUCH better.

@rui-vas
Copy link
Contributor

rui-vas commented Apr 6, 2021

+1 ! :)

@Bobgy
Copy link
Contributor Author

Bobgy commented Apr 6, 2021

What do you think about merging master to v1.2 branch? If the restructuring is what you care about

@thesuperzapper
Copy link
Member

@Bobgy this is a good idea, we should probably just take master's commits and put them into v.1.2

@rui-vas
Copy link
Contributor

rui-vas commented Apr 12, 2021

+1. That is a good idea!

@rui-vas
Copy link
Contributor

rui-vas commented Apr 12, 2021

Are there any v1.3 specific commits we should cherry-pick out of this move?

If not, I can simply make a PR to merge all the commits currently in master to v1.2-branch.

@rui-vas
Copy link
Contributor

rui-vas commented Apr 12, 2021

@Bobgy @thesuperzapper WDYT?

@rui-vas
Copy link
Contributor

rui-vas commented Apr 19, 2021

@Bobgy, at what point do you think we should move the docs to point to master again?

@rui-vas
Copy link
Contributor

rui-vas commented Apr 19, 2021

There are a bunch of ongoing fixes being done in master - e.g. #2593 - which would be great to make effective

@rui-vas
Copy link
Contributor

rui-vas commented Apr 19, 2021

And unfortunately this is taking a bit longer as well #2612

@Bobgy
Copy link
Contributor Author

Bobgy commented Apr 20, 2021

@RFMVasconcelos I personally think as long as master branch is not confusing in obvious ways, it's okay to switch.

In previous releases, we wait until most distributions are ready, but I don't think that is necessary, as long as each distribution can release their docs on their own, it's fine.

@rui-vas
Copy link
Contributor

rui-vas commented Apr 21, 2021

@Bobgy I agree!

#2612 has been merged, so we already have a good structure of the docs. 🎉 But in any case, small fixes and updates are being done in master now that are relevant for v1.3 users - e.g. #2595. I propose we move to master branch as most distributions appear now ready for this release :)

@rui-vas
Copy link
Contributor

rui-vas commented Apr 21, 2021

I just realized that GCP and Azure appear not ready according to the list, we may want to wait for it to be the case as these are obviously big distributions :)

@rui-vas
Copy link
Contributor

rui-vas commented Apr 21, 2021

One great change we are working on is #2611 which would be amazing to have already as part of user journey today, as we release v1.3 with the new blog.

@Bobgy do you think this is possible?

@rui-vas
Copy link
Contributor

rui-vas commented Apr 22, 2021

@Bobgy @joeliedtke @animeshsingh @thesuperzapper @castrojo @jbottum @cvenets @yanniszark @theadactyl

We just merged the last PRs that IMO were critical for a successful v1.3 release, we now just need to point the website to master » #2633 for these changes to be effective 🎉 @Bobgy @joeliedtke do you think you can review the master branch, verify if all is in order to proceed, and if so action #2633?

Many thanks to all for this push!

@thesuperzapper
Copy link
Member

@Bobgy I think we are all good to make the switch to master tonight/ASAP, as we are about to release the 1.3 blog.

@Bobgy
Copy link
Contributor Author

Bobgy commented Apr 22, 2021

Discussed over the release meeting, I'm going to change website to master branch right now.
I'm going to sleep, so bear in mind someone still needs to change the version menu to 1.3 following https://github.com/kubeflow/kubeflow/blob/master/docs_dev/releasing.md#lifecycle.

/cc @yanniszark @RFMVasconcelos @thesuperzapper

UPDATE: done the switch, the next push in master will switch the website to master branch.
v1.2-branch is still not available in kubeflow.org. I'll try to set it up over the next few days.

@thesuperzapper
Copy link
Member

thesuperzapper commented Apr 27, 2021

To close out this issue, we just need to fix issue: #2657

I believe we can track other doc issues separately, for example:

  1. "remove distribution docs from kubeflow.org" (issue: remove distribution docs from kubeflow.org #2629)
  2. "update component docs for Kubeflow 1.3" (issue: update component docs for Kubeflow 1.3 #2658)
  3. "refactor About / Getting Started + create Concepts section"(issue: Refactor About / Getting Started + create Concepts section #2659)
  4. "clean up Troubleshooting section" (issue: Clean up Troubleshooting section #2616)
  5. "add "Resources" and organize content under it" (issue: Add "Resources" and organize content under it #2491)

@rui-vas
Copy link
Contributor

rui-vas commented Apr 27, 2021

I've made a couple of PRs to solve #2657

@rui-vas
Copy link
Contributor

rui-vas commented Apr 27, 2021

@thesuperzapper @Bobgy could you please review and LGTM #2666 and #2667 ? :)

Thank you @thesuperzapper for listing what's left! Let's try to use the Kubeflow Docs Restructure project still for kanban.

@thesuperzapper
Copy link
Member

@RFMVasconcelos can you add me as an editor on that project? I cant add cards right now

https://github.com/kubeflow/website/projects/2

@rui-vas
Copy link
Contributor

rui-vas commented May 4, 2021

For sure! I thought this was open to the community, let me check how to open permissions

@rui-vas
Copy link
Contributor

rui-vas commented May 4, 2021

@thesuperzapper This is very strange, there is no settings where I can tweak projects permissions. Can others move cards in the project?

@Bobgy are you able to verify that you can?

@thesuperzapper
Copy link
Member

@RFMVasconcelos I personally use projects which are on the kubeflow organisation, but if you are the owner of the docs refactor project, you should be able to:

  1. Open the project
  2. Click the Hamburger menu on the top right
  3. Click the 3 dot menu
  4. Click settings
  5. Add Collaborators

@rui-vas
Copy link
Contributor

rui-vas commented May 5, 2021

@thesuperzapper thank you for the guidance!

I don't see settings, maybe I am not an owner? I did create the project and am able to use it...
Screenshot from 2021-05-05 13-52-38

@Bobgy do you see settings?

@thesuperzapper
Copy link
Member

@RFMVasconcelos I created a project on the kubeflow org for the docs and made you admin on it, everyone else is only a reader unless we explicitly add them).

It's probably easier to just move over to that one, where we have actual admin access, feel free to change the structure, etc.

@thesuperzapper thesuperzapper added this to In progress in Website & Docs May 6, 2021
@rui-vas
Copy link
Contributor

rui-vas commented May 6, 2021

@thesuperzapper thanks for this!
LGTM

We can classify the current Docs Restructure project as finished, and track continuous improvements there. Lets start porting items to that board, and next week I will take some time to organize them if you don't get to it before me :)

@rui-vas
Copy link
Contributor

rui-vas commented May 27, 2021

I believe we can now close this tracking issue!

Thank you all for the amazing work :)

@rui-vas rui-vas closed this as completed May 27, 2021
Needs Triage automation moved this from To Do to Closed May 27, 2021
Website & Docs automation moved this from In progress to Done May 27, 2021
@rui-vas rui-vas moved this from Blocking issue for v1.3 to Done in Kubeflow docs restructure May 27, 2021
@kubeflow-bot kubeflow-bot removed this from Closed in Needs Triage May 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

3 participants