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

Document new staging workflow #83381

Closed
worldofpeace opened this issue Mar 25, 2020 · 8 comments
Closed

Document new staging workflow #83381

worldofpeace opened this issue Mar 25, 2020 · 8 comments
Labels

Comments

@worldofpeace
Copy link
Contributor

Describe the bug
Since RFC 26 was merged we have a new staging workflow that involves a staging-next branch.
It was merged on 15 Nov 2018, but was implemented a bit before that.
Part of the Future Work was to document this workflow so new contributors can understand this. We still need to do this.

cc @vcunat @FRidh

@vcunat
Copy link
Member

vcunat commented Mar 26, 2020

I expect the main thing people need are those three bullets, right? Especially the first two ones, but the workflow change was specifically designed not to really change those two points, so that most people are as little affected as possible.

@vcunat
Copy link
Member

vcunat commented Mar 26, 2020

Wait, 19.09 already has (some) docs for all this: https://nixos.org/nixpkgs/manual/#submitting-changes-staging-next-branch ... so can you (anyone) be more specific about what seems missing?

@worldofpeace
Copy link
Contributor Author

Wait, 19.09 already has (some) docs for all this: https://nixos.org/nixpkgs/manual/#submitting-changes-staging-next-branch ... so can you (anyone) be more specific about what seems missing?

Woah, wonder how I missed that. Added in 62193ec. Though I do think some things are missing because if both of you were to be unavailable the workflow to do the thing is unclear.
Like, from what I see there how are all the merges done?
I've figured out from observation that you

  • merge staging into staging-next (begins staging-next process)
  • pr to master
  • avoid conflicts
    • merge master into staging-next
    • merge staging-next into staging
  • merge staging-next into master (promote staging-next to master)

We could also document the details of the jobsets like it is in the RFC, and how is someone supposed to determine staging-next is of sufficient quality? I do understand this involves looking at hydra.

It's documented how to submit changes, but I think the workflow around it isn't.

I'm raising this because I'm getting the impression that maybe more people would like to help out getting staging-next PRs merged, so there is this concept in nixpkgs of the "regular staging-next PR".

@stale
Copy link

stale bot commented Sep 23, 2020

Hello, I'm a bot and I thank you in the name of the community for opening this issue.

To help our human contributors focus on the most-relevant reports, I check up on old issues to see if they're still relevant. This issue has had no activity for 180 days, and so I marked it as stale, but you can rest assured it will never be closed by a non-human.

The community would appreciate your effort in checking if the issue is still valid. If it isn't, please close it.

If the issue persists, and you'd like to remove the stale label, you simply need to leave a comment. Your comment can be as simple as "still important to me". If you'd like it to get more attention, you can ask for help by searching for maintainers and people that previously touched related code and @ mention them in a comment. You can use Git blame or GitHub's web interface on the relevant files to find them.

Lastly, you can always ask for help at our Discourse Forum or at #nixos' IRC channel.

@stale stale bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Sep 23, 2020
@jtojnar
Copy link
Member

jtojnar commented Dec 5, 2020

We now have automatic reverse flow (master → staging-next → staging) every six hours: #105153

@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Dec 5, 2020
@stale
Copy link

stale bot commented Jun 3, 2021

I marked this as stale due to inactivity. → More info

@stale stale bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jun 3, 2021
@nixos-discourse
Copy link

This issue has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/2022-10-22-documentation-team-meeting-notes-12-nixcon-edition/22689/1

@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Oct 22, 2022
@samueldr
Copy link
Member

There is now documentation here:

Hopefully this fulfills what @worldofpeace wanted to see.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants