Skip to content

Conversation

Martin-Rehr
Copy link
Contributor

Cleanup lustre source code after pylustrequota build since it's not needed and just consumes space in the image.

Copy link
Contributor

@jonasbardino jonasbardino left a comment

Choose a reason for hiding this comment

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

I think you want to move the trailing semi-colons (;) in the previous line down to the newly added lines to get the intended behaviour.

@jonasbardino jonasbardino added the enhancement New feature or request label Aug 5, 2025
@jonasbardino
Copy link
Contributor

Running python setup.py ... directly is now deprecated, btw.
https://packaging.python.org/en/latest/discussions/setup-py-deprecated/

@Martin-Rehr
Copy link
Contributor Author

Running python setup.py ... directly is now deprecated, btw. https://packaging.python.org/en/latest/discussions/setup-py-deprecated/

Thanks

Copy link
Contributor

@jonasbardino jonasbardino left a comment

Choose a reason for hiding this comment

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

I haven't tried the old or new install in practice, yet, but I suppose you already did.
It looks correct with semi-colons 👍

@Martin-Rehr Martin-Rehr merged commit ae34504 into master Aug 5, 2025
1 of 4 checks passed
@Martin-Rehr Martin-Rehr deleted the adjust/lustre-quota-cleanup branch August 5, 2025 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants