[Bug]: Unable to deploy charm with branch in channel

[carlcsaposs-canonical]

Description

Unable to deploy charm with branch (e.g. 8.0/edge/pr-3)

python-libjuju/juju/origin.py

Lines 69 to 71 in d809caf

	def parse(s):
	"""parse a channel from a given string.
	Parse does not take into account branches.

Urgency

Annoying bug in our test suite

Python-libjuju version

2.9.38.1

Juju version

2.9.29

Reproduce / Test

from pytest_operator.plugin import OpsTest

MYSQL_APP_NAME = "mysql-k8s"

async def test_deploy(ops_test: OpsTest):
    await ops_test.model.deploy(
        MYSQL_APP_NAME,
        channel="8.0/edge/mysqlrouter-user",
        application_name=MYSQL_APP_NAME,
        num_units=3,
        trust=True,
    )

[juanmanuel-tirado]

I will add this to the wishlist.

[carlcsaposs-canonical]

I will add this to the wishlist.

thank you!

[github-actions]

This issue is marked as incomplete because it has been open 30 days with no activity. Please remove incomplete label or comment or this will be closed in 5 days.

[carlcsaposs-canonical]

Still an issue

[ca-scribner]

Kubeflow team is hit by this too. For cases where two charms need to change at once (like if we change a relation), we'd like to use branches in tests so we do not have to always build all charms locally.

[cderici]

Looks like this has fallen through the cracks, thanks for the ping, we'll get on it (in the next pulse). I might ask you later @ca-scribner about some branches we can test this on.

[ca-scribner]

ty @cderici! Happy to help