Skip to content

Conversation

anoopcs9
Copy link
Collaborator

These packages are automatically pulled in with python3-sambacc extras which gets built by default and available for installation as weak dependencies. If we know that these packages exists and available, then we shoud be good here to assume that they are pulled in during the installation of corresponding python3-sambacc extras.

$ rpm -q --requires python3-sambacc+toml-0.8~1.g52a3e0101-1.el9.noarch.rpm | grep tomli
python3.9dist(tomli)

$ rpm -q --requires python3-sambacc+yaml-0.8~1.g52a3e0101-1.el9.noarch.rpm | grep pyyaml
python3.9dist(pyyaml) >= 5.4

$ rpm -q --requires python3-sambacc+rados-0.8~1.g52a3e0101-1.el9.noarch.rpm | grep rados
python3.9dist(rados)

@anoopcs9
Copy link
Collaborator Author

anoopcs9 commented Oct 2, 2025

@Mergifyio rebase

These packages[1] are automatically pulled in with python3-sambacc
extras which gets built by default and available for installation
as weak dependencies. If we know that these packages[1] exists and
available, then we should be good here to assume that they are pulled
in during the installation of corresponding python3-sambacc extras.

$ rpm -q --requires \
    python3-sambacc+toml-0.8~1.g52a3e0101-1.el9.noarch.rpm | grep tomli
python3.9dist(tomli)

$ rpm -q --requires \
    python3-sambacc+yaml-0.8~1.g52a3e0101-1.el9.noarch.rpm | grep pyyaml
python3.9dist(pyyaml) >= 5.4

$ rpm -q --requires \
    python3-sambacc+rados-0.8~1.g52a3e0101-1.el9.noarch.rpm | grep rados
python3.9dist(rados)

[1] python3-pyyaml, python3-tomli and python3-rados

Signed-off-by: Anoop C S <[email protected]>
Copy link

mergify bot commented Oct 2, 2025

rebase

✅ Branch has been successfully rebased

@anoopcs9 anoopcs9 force-pushed the rm-explicit-install branch from cc9be63 to 37f1eb5 Compare October 2, 2025 11:42
Copy link
Collaborator

@phlogistonjohn phlogistonjohn left a comment

Choose a reason for hiding this comment

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

lgtm. the list was out of date and since everything else was working that proves this is not strictly needed any more

@anoopcs9 anoopcs9 requested review from obnoxxx and xhernandez October 3, 2025 06:05
@mergify mergify bot added the queued label Oct 3, 2025
@mergify mergify bot merged commit 11d486d into samba-in-kubernetes:master Oct 3, 2025
43 checks passed
@mergify mergify bot removed the queued label Oct 3, 2025
@anoopcs9 anoopcs9 deleted the rm-explicit-install branch October 8, 2025 06:33
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