[MCOL-5113] docker: make provision standard for both docker-compose and sky Created: 2022-06-02  Updated: 2022-12-19  Resolved: 2022-12-19

Status: Closed
Project: MariaDB ColumnStore
Component/s: Build
Affects Version/s: None
Fix Version/s: N/A

Type: Task Priority: Minor
Reporter: alexey vorovich (Inactive) Assignee: Todd Stoffel (Inactive)
Resolution: Fixed Votes: 0
Labels: triage

Epic Link: docker cleanup
Sprint: 2021-17, 2022-22

 Description   

We do have reports of sporadic readonly cluster with CS in Sky. I suggest we review startup

1. for docker-compose
docker-compose up -d && docker exec -it mcs1 provision
here we wait for all nodes to start and then execute cluster demo on mcs1
this is pretty simple https://github.com/mariadb-corporation/mariadb-skysql-columnstore-docker/blob/master/scripts/provision

2. for sky this is very different . look at the logic https://github.com/mariadb-corporation/mariadb-skysql-columnstore-docker/blob/12de7a737b47dc14c09321401c996fdce3c01ebe/scripts/skysql-specific-startup.sh#L80
it seems that SKY is using quite a complex logic and the timing is not clear.

Could we switch to the provision model where provision script allows to pass in the array of hosts

-operator start N nodes
-each node completes starting server and CMAPI server
-operator calls cluster_demo on node 1 and passes array of nodes . The cluster demo completes the startup

the pod shutdown should be handled unirformly. Ideally a new deprovision script that Todd provides and operator invokes at end



 Comments   
Comment by alexey vorovich (Inactive) [ 2022-06-07 ]

Good...

toddstoffel, to preserve uniformity between compose and sky, we should improve provision.sh script and allow passing of the list of hosts from Sky operator.
The same script technique should be used used for other events such stop/start

I think we should start on this with target 22.08 . I can help debug /provide feedback

Generated at Thu Feb 08 02:55:26 UTC 2024 using Jira 8.20.16#820016-sha1:9d11dbea5f4be3d4cc21f03a88dd11d8c8687422.