Skip to content

Commit

Permalink
Merge pull request #15650 from SimonRichardson/fix-machine-test-logs-aws
Browse files Browse the repository at this point in the history
#15650

If we're using postgres charm it prevents us from tearing down because of storage issues. This test doesn't need to use postgres, we should just use a simpler charm for now.


## Checklist

- [x] [Integration tests](https://github.com/juju/juju/tree/main/tests), with comments saying what you're testing

## QA steps

```sh
$ cd tests && ./main.sh machine
```
  • Loading branch information
jujubot committed May 24, 2023
2 parents 1a2759d + 097abc1 commit c316413
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions tests/suites/machine/machine.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,12 +6,12 @@ test_log_permissions() {
file="${TEST_DIR}/test_log_permissions.log"
ensure "correct-log" "${file}"

juju deploy postgresql --series focal
juju deploy juju-qa-test source --series focal

wait_for "started" '.machines."0"."juju-status".current'

check_contains "$(juju ssh 0 -- stat -c '%G' /var/log/juju/unit-postgresql-0.log)" adm
check_contains "$(juju ssh 0 -- stat -c '%a' /var/log/juju/unit-postgresql-0.log)" 640
check_contains "$(juju ssh 0 -- stat -c '%G' /var/log/juju/unit-source-0.log)" adm
check_contains "$(juju ssh 0 -- stat -c '%a' /var/log/juju/unit-source-0.log)" 640

check_contains "$(juju ssh 0 -- stat -c '%a' /var/log/juju/machine-0.log)" 640
check_contains "$(juju ssh 0 -- stat -c '%G' /var/log/juju/machine-0.log)" adm
Expand Down

0 comments on commit c316413

Please sign in to comment.