search

canonical / cassandra-k8s-operator

Apache License 2.0
1 stars 4 forks source link

Rewrite First PR #42

Closed dstathis closed 3 years ago

jnsgruk commented 3 years ago

Had a few minutes so have updated the tox and pyproject config

dstathis commented 3 years ago

Fixes #37

dstathis commented 3 years ago

This is not the final version of the Cassandra rewrite but I think it is a good checkpoint with lots of improvements

dstathis commented 3 years ago

Also, I am aware the integration tests are failing. This is a test failure and not a charm failure as it succeeds when run locally. I'm looking in to it but don't want to delay for this.

dstathis commented 3 years ago

Fixes #32

dstathis commented 3 years ago

Fixes #27

mmanciop commented 3 years ago

@sed-i can you please have a look at the integration test?

sed-i commented 3 years ago

@sed-i can you please have a look at the integration test?

The most recent failed test says Unit in error: cassandra-k8s/0, which means the charm's status was not idle/active but error.

dstathis commented 3 years ago

@sed-i can you please have a look at the integration test?

The most recent failed test says Unit in error: cassandra-k8s/0, which means the charm's status was not idle/active but error.

Yeah I see this and I haven't been able to figure out why. It always passes when run locally. Also, I briefly looked at some other repos and they seem to be having the same error.

sed-i commented 3 years ago

Yeah I see this and I haven't been able to figure out why.

In your [testenv:integration], change the command to pytest -v --tb native --log-cli-level=INFO -s {posargs} {toxinidir}/tests/integration and you will be able to see more log messages