Enable poetry workflow

[jdkandersson]

Applicable spec: ISD115

Overview

• Install poetry in the unit/integ test runner with the poetry-plugin-export plugin
• Optionally uses the tox -e pack command instead of charmcraft pack to enable the use of poetry
• Remove tox.ini's dependencies check since they will be located in pyproject.toml

Rationale

Enable workflows with full dependency pinning using poetry.
PRs using this branch:

• Switch to poetry bind-operator#19 (using poetry)
• Switch to poetry discourse-k8s-operator#250 (using poetry)
• Switch to poetry synapse-operator#223 (using poetry)

Workflow Changes

The integration test has been changed to optionally use tox -e pack command instead of charmcraft pack

Checklist

• The contributing guide was applied
• The PR is tagged with appropriate label (urgent, trivial, complex)

[nrobinaubertin]

This is blocked until I make it work with the following PR: canonical/synapse-operator#223

[jdkandersson]

What is the reason for removing this please?

[nrobinaubertin]

When using poetry, the dependencies will be handled in the pyproject.toml file and this test will always fail

[jdkandersson]

Could it somehow be adapted so that we still get the benefits please? It is checking here that the expected tools are being run which is quite helpful to make sure that the expected linting is actually being done in a given repo

[cbartz]

This is now marked ready for review, but I think the PR description is out of sync (e.g. Install poetry in the unit/integ test runner with the poetry-plugin-export plugin is not present).

[jdkandersson]

This is now marked ready for review, but I think the PR description is out of sync (e.g. Install poetry in the unit/integ test runner with the poetry-plugin-export plugin is not present).

Good point, perhaps @nrobinaubertin would you like to take over this PR please?

[nrobinaubertin]

This is now marked ready for review, but I think the PR description is out of sync (e.g. Install poetry in the unit/integ test runner with the poetry-plugin-export plugin is not present).

Good point, perhaps @nrobinaubertin would you like to take over this PR please?

I'll take it over.
But I did not want to put it in review because of #236 (comment)

[merkata]

@nrobinaubertin @jdkandersson is this being worked on? I suggest closing it if not and we can reopen it if needed..