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

PAD-173: 2 node HA - Tech Preview #131

Merged
merged 90 commits into from
Jun 11, 2024
Merged

PAD-173: 2 node HA - Tech Preview #131

merged 90 commits into from
Jun 11, 2024

Conversation

TylerGillson and others added 30 commits October 4, 2023 11:49
Error: lexer error: Earthfile
syntax error: line 263:4: token recognition error at: 'ELSE '
This handles both suse and ubuntu
Signed-off-by: Tyler Gillson <[email protected]>
* WIP: Add script to automate testing of two node cluster

* Add template for cluster profile

* Remove separate function to build stylus framework image

Earthly stylus goal is now building this goal two

* WIP: automation of cluster launch

* Move test script to test/

* Exract user data template

* Add creation of all machines

* Refactor main

* Add missing required variable

* wip: e2e vmware test

Signed-off-by: Tyler Gillson <[email protected]>

* finish automating e2e provisioning

Signed-off-by: Tyler Gillson <[email protected]>

* tidy whitespace & remove oz's changes to default user-data template

Signed-off-by: Tyler Gillson <[email protected]>

* fixes & docs

Signed-off-by: Tyler Gillson <[email protected]>

* add isTwoNodeCandidate flag to cluster template

Signed-off-by: Tyler Gillson <[email protected]>

* fix: remove invalid node-status-update-frequency flag

Signed-off-by: Tyler Gillson <[email protected]>

* fix: ensure unique VM names in vSphere; docs

Signed-off-by: Tyler Gillson <[email protected]>

* remove invalid arg from cluster profile template

Signed-off-by: Tyler Gillson <[email protected]>

* add livenessSeconds & VIP config

Signed-off-by: Tyler Gillson <[email protected]>

* feat: add destroy_edge_hosts

Signed-off-by: Tyler Gillson <[email protected]>

* install ping for two node

Signed-off-by: Tyler Gillson <[email protected]>

* feat: configurable NIC_NAME

Signed-off-by: Tyler Gillson <[email protected]>

* fix test-two-node.sh

* add debug scripts, cleanup funcs, remove two-node env hack

Signed-off-by: Tyler Gillson <[email protected]>

---------

Signed-off-by: Tyler Gillson <[email protected]>
Co-authored-by: Oz Tiram <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
We are using external kine so that we can share as much as code
between postgresql and sqlite and also between kubeadm and k3s.

Signed-off-by: Oz Tiram <[email protected]>
* Add postgresql installation if the backend requires it

Signed-off-by: Oz N Tiram <[email protected]>

* Add preparation of postgresql to ubuntu base

* Configure postgresql on suse too

Signed-off-by: Oz N Tiram <[email protected]>

---------

Signed-off-by: Oz N Tiram <[email protected]>
* DRY configuration of postgresql

Signed-off-by: Oz N Tiram <[email protected]>

* Fix Earthfile syntax

Signed-off-by: Oz Tiram <[email protected]>

---------

Signed-off-by: Oz Tiram <[email protected]>
* Parameterize build branches

If I want to merge a branch into the two-node brach,
I should be able to test that branch, hence I should build from
that branch.

Signed-off-by: Oz Tiram <[email protected]>

* Move the variables for the test script to own file

This allows merging changes to the test script without
risking git merge havoc if these are overriden in the script.
Also, it prevents people from accidentally commiting secrets.

Signed-off-by: Oz Tiram <[email protected]>

* Make use of absolute path to env file

This allows running the script from the test dir and
also any other path (e.g. top dir of the repo)

Signed-off-by: Oz Tiram <[email protected]>

* Remove extra curly brace

* Fix how PG_CONF_DIR is referenced

Also, used explicit RUN command for each shell line

Signed-off-by: Oz Tiram <[email protected]>

* Fix IF ELSE for PG_CONF_FILE

* Update test-two-node.sh

* Update Earthfile

* Update env.example

* Update Earthfile

---------

Signed-off-by: Oz Tiram <[email protected]>
Co-authored-by: Tyler Gillson <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
Signed-off-by: Tyler Gillson <[email protected]>
@TylerGillson TylerGillson merged commit bc880b3 into main Jun 11, 2024
@TylerGillson TylerGillson deleted the 2-node branch June 11, 2024 21:35
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.

3 participants