Skip to content

[9.0] chore: added sweep rule#8576

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

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

Conversation

@fstagni
Copy link
Copy Markdown
Contributor

@fstagni fstagni commented Jun 4, 2026

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
Copy link
Copy Markdown
Contributor

aldbr commented Jun 4, 2026

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
Copy link
Copy Markdown
Contributor Author

fstagni commented Jun 4, 2026

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
Copy link
Copy Markdown
Member

sfayer commented Jun 4, 2026

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
Copy link
Copy Markdown
Contributor Author

fstagni commented Jun 5, 2026

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
Copy link
Copy Markdown
Contributor

chaen commented Jun 5, 2026

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

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