charm-logrotated:master

Last commit made on 2023-10-11
Get this branch:
git clone -b master https://git.launchpad.net/charm-logrotated
Members of Llama (LMA) Charmers can upload to this branch. Log in for directions.

Branch merges

Branch information

Name:
master
Repository:
lp:charm-logrotated

Recent commits

aa1115f... by Ashley James

Add juju 3.1 support.

Also removes juju 2.9 support.

Reviewed-on: https://code.launchpad.net/~dashmage/charm-logrotated/+git/charm-logrotated/+merge/452136
Reviewed-by: Eric Chen <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-bootstack <email address hidden>
Reviewed-by: Tianqi Xiao <email address hidden>

0e34497... by Ashley James

func: Remove juju version pin

9087925... by Chi Wai CHAN

Fix lint test by running `make black`

Reviewed-on: https://code.launchpad.net/~raychan96/charm-logrotated/+git/charm-logrotated/+merge/445949
Reviewed-by: Ramesh Sattaru <email address hidden>
Reviewed-by: Eric Chen <email address hidden>

70690a3... by Chi Wai CHAN

Run `make black` to fix lint test.

d0c2447... by Wong Hong Han

Add unit test coverage

Reviewed-on: https://code.launchpad.net/~honghan/charm-logrotated/+git/charm-logrotated/+merge/445015
Reviewed-by: Tianqi Xiao <email address hidden>
Reviewed-by: Eric Chen <email address hidden>

b9e564d... by Wong Hong Han

fix wrong and missing docstring

37732c3... by Wong Hong Han

separate invalid checking to another test

b5f8d73... by Wong Hong Han

add some unit test coverage

b5d0cd9... by Ashley James

Implement random time functionality for daily cronjob.

Reviewed-on: https://code.launchpad.net/~dashmage/charm-logrotated/+git/charm-logrotated/+merge/443391
Reviewed-by: Eric Chen <email address hidden>
Reviewed-by: JamesLin <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-bootstack <email address hidden>

af917de... by Ashley James

Implement random time functionality for daily cronjob.

- The "random" functionality was not working earlier since the Ubuntu
  cron package does not support the feature. So a new method was
  added to generate a random time within a range. This will provide
  the random time to be placed in /etc/crontab for the daily cronjob.
- Remove condition to check if `logrotate-cronjob-frequency` is
 `daily` before applying the cron daily schedule.
- Modify existing unit tests to accomodate the changes with the random
  time functionality.
- Fix validation of random time and add another unit test for the
  validation.
- Edit config.yaml to add a clear disclaimer about
  `update-cron-daily-schedule`.