Skip to content

[9.0] chore: added sweep rule#8576

Closed
fstagni wants to merge 1 commit into
DIRACGrid:rel-v9r0from
fstagni:v9_config_2
Closed

[9.0] chore: added sweep rule#8576
fstagni wants to merge 1 commit into
DIRACGrid:rel-v9r0from
fstagni:v9_config_2

Conversation

@fstagni

@fstagni fstagni commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@fstagni fstagni requested a review from atsareg as a code owner June 4, 2026 12:23
@fstagni fstagni added the sweep:ignore Prevent sweeping from being ran for this PR label Jun 4, 2026
@aldbr

aldbr commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

I thought that only specific patches would go from integration to rel-v9r0, am I wrong? That would force us to test them well before going into rel-v9r0, which is expected to stay very stable.
Shall we let people target v9 first before integration? (because it sounds that this sweep rule would allow that)

@fstagni

fstagni commented Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

We are "done", I hope, with backportings.

The pattern should stay:

  • work on integration for most of things
  • target v9r0 for urgent bug fixes for the "lily pad" release. These bug fixes normally should be swept to integration.

@sfayer

sfayer commented Jun 4, 2026

Copy link
Copy Markdown
Member

Hi,

I think like @aldbr, I was kinda expecting the sweep to go the other way... i.e. development all happens on integration and very specific patches may get specially[*] marked as sweepable to be merged back into the rel-v9r0 branch? (Rather than encouraging people to develop against v9.0 and then forward-port).

Regards,
Simon

[*] e.g. the tag would be something like sweep:include on special stuff and the default "no tag" behaviour would be sweep:ignore (so unless you actively want backporting you don't have to think/worry about it).

@fstagni

fstagni commented Jun 5, 2026

Copy link
Copy Markdown
Contributor Author

I am trying to ease your way when the "inevitable" patch will need to be applied to v9. Otherwise we just remove completely the sweeper.

@chaen

chaen commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

I think we should remove the sweeper. Every development goes to integration, and is backported manually

@fstagni

fstagni commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

Fine for me!

@fstagni fstagni closed this Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sweep:ignore Prevent sweeping from being ran for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants