Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding aproxy support #325

Closed
wants to merge 28 commits into from

Conversation

phvalguima
Copy link

No description provided.

shayancanonical and others added 19 commits August 23, 2023 15:55
Minor bugfixes + proper usage of constants
* Preemptively switchover primary on scale-down

* addressing pr comments
* Update mysql image to 8.0.34 rock

* Update the rock hash in the upgrade test

* Deploy the latest/stable charm of the tls-operator in the tls integration test

* Update comment to correct version of mysql
…for 8.0.34 (#299)

* Update upgrade dependencies for mysqld and charm to account for 8.0.34

* Revert charm version to v1 in upgrade dependencies
* lib update

* render config file in the lib

* test fix

* refactor to use lib's new method for check connection

* won't check port before initializing users
* TDD for rollbacks

* split upgrade from stable

* trust

* set app data

* not doing upgrade from stable

* bump libjuju for nested assumes in metadata

* test fixes

* get first upgrading unit to check status
* use alternative name for mysql-test-app to avoid confusion with mysql app

* remove dup

* preemptively switchover on completion

* add missing resume

* split to reduce complexity and new mock

* bump lib

* pr feedback

* pr feedback plus more robust test to survice CI

* leader unit will stay in recovery state on rollback

* bump pytest_operator_cache

* fix build_charm fixture return as pathlib.Path

* fix rollback instruction

* address pr comments

* bump upgrade lib

* fix unit test

* more verbosity

* includes upgrade from stable

stable was bumped to version with upgrade support

* fix for rollback (data-platform-libs issue #97)

* extract and use helper function

* unneeded/wrong asserts

* fix fixture

* simplify fixture usage

* another fix

* fix call

* fix fixture reference

* revert mistakenly commited files
* CI matrix + env.vars

* pyproject updates (libjuju version, pytest-mock, etc)

* tox.ini equipped with Juju version selection vars

* Juju3 compatible tests

* Fixtures that allow detection and 'switching' between Juju versions

* snap_microk8s instead of old microk8s user
…312)

* Implement log rotation of error, general and slowquery logs

* Fix failing unit tests

* Avoid modifying the charm container's pebble layer

* Address PR + failing unit test

* Address PR feedback + add integration tests for log rotation

* Re-add jinja2 to pyproject dependencies

* Address PR feedback

* Port in fix to disable binlogging for flush log queries + make self-healing tests more consistent

* Add scripts directory to all paths in the tox file
* data_platform_libs/data_interfaces.py v18 (Relation Secrets)

* NO direct Relation databag usagegit add src/relations/mysql_provider.py!

* Checking the correct usage of relation databag vs. secrets

* Versions kept up-to-date for the pipeline
)

* Tests are running on Juju2 by default

* Juju 3.1/stable + Juju3 as default

* Updating CONTRIBUTING doc
…initialized (#323)

* Prevent starting logrotate dispatcher or flush mysql logs until unit initialized

* Always setup logrotate files in ephemeral storage during mysql-pebble ready in case pod is killed
* first passing

* using reboot_from_complete_outage instead
@phvalguima phvalguima force-pushed the self-hosted-runners-with-aproxy branch from 868db7b to 0e6475e Compare October 18, 2023 08:27
@phvalguima phvalguima force-pushed the self-hosted-runners-with-aproxy branch from 8254091 to c87b874 Compare October 18, 2023 09:29
@phvalguima phvalguima closed this Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants