Skip to content

Allow --ignore-installed flag for setuptools and wheel upgrades in dev container.#479

Merged
Geogouz merged 1 commit into
rucio:masterfrom
Geogouz:patch-478-ignore-installed-setuptools
Feb 11, 2026
Merged

Allow --ignore-installed flag for setuptools and wheel upgrades in dev container.#479
Geogouz merged 1 commit into
rucio:masterfrom
Geogouz:patch-478-ignore-installed-setuptools

Conversation

@Geogouz

@Geogouz Geogouz commented Jan 30, 2026

Copy link
Copy Markdown
Contributor

Handles #478

To get things unstuck, I am pushing it now to get things going. Nevertheless, there is the #475 under review that uses venv, addressing the problem in a better way. But I think this might take some time.

@Geogouz
Geogouz requested a review from bziemons January 30, 2026 14:16
@Geogouz Geogouz self-assigned this Jan 30, 2026

@rdimaio rdimaio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is all temporary until the venv change anyway, but could also be worth doing the same change used in the webui container here #467 so that we use the same fix in all containers

@Geogouz

Geogouz commented Feb 11, 2026

Copy link
Copy Markdown
Contributor Author

This is all temporary until the venv change anyway, but could also be worth doing the same change used in the webui container here #467 so that we use the same fix in all containers

My opinion would be to actually better switch that into this approach. It sounds like having less side effects. But since all that is temporary, we can just leave them as is till we do it for both in the proper way. For now, since it has been approved, I'm going ahead to merge it because we have been too long without a working dev image. People are noticing problems on our support channel already :D.

@Geogouz
Geogouz merged commit 257c243 into rucio:master Feb 11, 2026
1 check passed
@Geogouz
Geogouz deleted the patch-478-ignore-installed-setuptools branch February 11, 2026 17:23
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.

2 participants