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

Separate Site and Controller repositories #436

Closed
15 tasks done
squeedee opened this issue Dec 7, 2021 · 7 comments
Closed
15 tasks done

Separate Site and Controller repositories #436

squeedee opened this issue Dec 7, 2021 · 7 comments
Assignees
Labels
blocked chore documentation Improvements or additions to documentation

Comments

@squeedee
Copy link
Member

squeedee commented Dec 7, 2021

Description of problem

  1. Site content changes at a different rate than the code
  2. We often need to update versioned docs separate from the tag for that version (site is always built from a single branch, namely main)
  3. Waiting for code checks on PRs when the only thing we've changed is documentation, is a significant pain
  4. If we do end up creating a multi-product site (Eg, introduce runnable as it's own project) then we'll be out ahead of the infra changes needed (namely, seperating the Site repo :D)

Plan:

  • create cartographer-site repo DONE

  • Place all migration work on a branch initial-migration

    • ensure all branches for site work is merged
    • Move experimental editor over and patch the release code
    • Deal with the CRD generation code so it bases itself off the latest release in the cartographer repository
    • call for a moratorium on site PRs (explain why) in Slack, and put a [Blocked on Migration] tag on all related doc stories.
    • Ensure Linter runs in github actions
    • Ensure dependabot is configured
    • Configure require PR approval
  • Make PR for initial-migration and get it merged

  • prepare cartographer PR:

    • Clean up any call-through makefile targets in the cartographer makefile (they will no longer be able to reach the site) (dont merge!)
    • remove site and experimental live-editor from cartographer repo (Don't merge!)
  • Have netlify use the new repository

  • Merge cartographer PR

  • unblock stories and let Slack know that the migration is complete

@squeedee squeedee added documentation Improvements or additions to documentation chore labels Dec 7, 2021
@squeedee squeedee added this to the 1.0+ milestone Dec 7, 2021
@scothis
Copy link
Contributor

scothis commented Dec 7, 2021

An intermediate option between same branch and different repos is to move website publishing to the gh-pages branch. It does mean that PRs need to be more considerate about which branch is the base.

@squeedee
Copy link
Member Author

squeedee commented Dec 7, 2021

An intermediate option between same branch and different repos is to move website publishing to the gh-pages branch. It does mean that PRs need to be more considerate about which branch is the base.

Do you propose this because of the pain of obtaining another repository? Otherwise it's pretty slight for me - in fact I'd prefer the separation of concerns and the effort involved over the weirdness of a branch.

Would love for these steps to also help ease the plan, up front, for adopting other subsystems.

@scothis
Copy link
Contributor

scothis commented Dec 7, 2021

Do you propose this because of the pain of obtaining another repository?

Nope, getting a new repo should be easy. Multi-repo vs mono-repo is always a balancing act

@squeedee
Copy link
Member Author

squeedee commented Dec 8, 2021 via email

@scothis
Copy link
Contributor

scothis commented Dec 8, 2021

There are no conditionals required for actions. Since the workflows are defined in source, you can have different workflows in different branches for pushes/prs.

@squeedee
Copy link
Member Author

squeedee commented Dec 8, 2021

Which implies an intentionally different structure of the content of the branch (I'd rip out all the code) at which point, what benefit the branch over a new repo with clear concerns?

@martyspiewak
Copy link
Contributor

I moved the last task for removing patch versions from the docs dropdown to a new issue in the cartographer-site repo: vmware-tanzu/cartographer-site#20

Closing this as It looks like the rest of the tasks have been completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked chore documentation Improvements or additions to documentation
Projects
Status: Closed
Development

No branches or pull requests

4 participants