Merge ~mertkirpici/juju-lint:lp/1995479 into juju-lint:master

[Mert Kirpici]

Close LP #1995479

Functests seem to get stuck with libjuju 3.0.4. Also PYTHONPATH variables were missing from the functests that use the python package.

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

[Eric Chen (eric-chen)]

This modification seems conflicts with the conclusion in LP #1993334 that we need to use pylibjuju >= 3.0.4. I prefer to dig into deeper and find out the root cause.

review: Needs Fixing

[🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack)]

PASSED: Continuous integration, rev:a60dc92ca10892d07bfa21ab42a213de75ca610f
https://jenkins.canonical.com/bootstack/job/lp-juju-lint-ci/25/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/bootstack/job/lp-charm-test-functest/1164/
    None: https://jenkins.canonical.com/bootstack/job/lp-update-mp/1582/

Click here to trigger a rebuild:
https://jenkins.canonical.com/bootstack/job/lp-juju-lint-ci/25//rebuild

review: Approve (continuous-integration)

[Gabriel Cocenza (gabrielcocenza)]

The problem was an incompatibility issue between python-libjuju and the version of the controller. Updating the controller to the latest stable version solved the issue.

review: Disapprove

Unmerged commits

a60dc92... by Mert Kirpici on 2022-11-02

tests/functional: pin juju to 3.0.2

Works-on: 1995479
Signed-off-by: Mert Kırpıcı <email address hidden>

diff --git a/tests/functional/requirements.txt b/tests/functional/requirements.txt
index f7fed82..da81487 100644
--- a/tests/functional/requirements.txt
+++ b/tests/functional/requirements.txt
@@ -1,2 +1,3 @@
 pytest
 pytest-operator
+juju==3.0.2
diff --git a/tox.ini b/tox.ini
index 08183e2..9a9b8f3 100644
--- a/tox.ini
+++ b/tox.ini
@@ -56,6 +56,7 @@ commands =
     --log-cli-level=WARNING \
     --ignore={toxinidir}/tests/unit \
     -m smoke
+setenv = PYTHONPATH={toxinidir}
 
 [testenv:func-dev]
 deps =
@@ -65,6 +66,7 @@ commands =
     pytest -v \
     --log-cli-level=WARNING \
     --ignore={toxinidir}/tests/unit
+setenv = PYTHONPATH={toxinidir}
 
 [testenv:lint]
 deps =