eve icon indicating copy to clipboard operation
eve copied to clipboard

Main changes for kube/cluster-init.sh for multi-node cluster handling

Open naiming-zededa opened this issue 1 year ago • 1 comments

  • create the cluster-utils.sh in pkg/kube for a number of functions
  • get EdgeNodeClusterStatus from 'zedkube' for cluster status, cluster nodeip, cluster prefix, etc
  • a background process to monitor the cluster mode changes
  • implement cluster mode change logic and operations for change from single-node to cluster, or from cluster back to single-node
  • save kube /var/lib at single-node first time setup for later use when the mode changes back from cluster mode to single node
  • display eve-release string for get node OS-IMAGE field
  • this patch tested still works in kubevirt single-node case.
  • some of the external_boot_img changes in the patch is from Andrew's diff.

naiming-zededa avatar Oct 18 '24 18:10 naiming-zededa

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 20.93%. Comparing base (dd27fe1) to head (c030dfa). Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4385   +/-   ##
=======================================
  Coverage   20.93%   20.93%           
=======================================
  Files          13       13           
  Lines        2895     2895           
=======================================
  Hits          606      606           
  Misses       2163     2163           
  Partials      126      126           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Oct 18 '24 18:10 codecov[bot]

Looks like it needs to be rebased.

deitch avatar Nov 07 '24 10:11 deitch

Looks like it needs to be rebased.

Hi @deitch rebased in the update. hi @andrewd-zededa, i have resolved some conflicts, in particular this:

        if ! external_boot_image_import; then
                continue
        fi

please check to see if this is in the intended place.

naiming-zededa avatar Nov 07 '24 20:11 naiming-zededa

hi @andrewd-zededa, i have resolved some conflicts, in particular this:

@andrewd-zededa can you respond to this and take another look?

eriknordmark avatar Nov 19 '24 21:11 eriknordmark