Skip to content

Switch rolling to *always* use testing repositories.#122

Open
clalancette wants to merge 1 commit into
masterfrom
clalancette/rolling-use-testing
Open

Switch rolling to *always* use testing repositories.#122
clalancette wants to merge 1 commit into
masterfrom
clalancette/rolling-use-testing

Conversation

@clalancette

Copy link
Copy Markdown

The ROS PMC is considering a change to only support the testing repository for Rolling. While that change hasn't been finalized yet, it is not a bad idea for the Rolling docker images to always use testing; it is Rolling, after all, and whether it is unstable in testing in main or unstable in testing is mostly irrelevant. Update the code here to always have rolling point to the testing repositories.

The ROS PMC is considering a change to only support the
testing repository for Rolling.  While that change hasn't
been finalized yet, it is not a bad idea for the Rolling
docker images to always use testing; it is Rolling, after
all, and whether it is unstable in testing in main or
unstable in testing is mostly irrelevant.  Update the
code here to always have rolling point to the testing
repositories.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
@clalancette

Copy link
Copy Markdown
Author

Note for reviewers: I have no idea how to test this out, any ideas welcome.

@MatthijsBurgh MatthijsBurgh left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logic seems correct to me.

I also don't know how to test this.

@ruffsl

ruffsl commented Jun 23, 2026

Copy link
Copy Markdown
Member

The ROS PMC is considering a change to only support the testing repository for Rolling.

I've been out of the loop for a while, was this settled on by the ROS PMC?

@clalancette

Copy link
Copy Markdown
Author

I've been out of the loop for a while, was this settled on by the ROS PMC?

This was discussed in the ROS PMC meeting a few weeks ago:

https://docs.google.com/document/d/1TGXYp33N00ZJtilsRzfClFXcSyMa4QKVNmA03Lx9D7w/edit?tab=t.0#bookmark=id.tcw8mvm6ym8g

The short of it is that it is still up in the air. But I think that we should make this change now so that we can unblock the Rolling Docker images for now. If the PMC makes a different choice, we can always follow suit later on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants