Skip to content
This repository has been archived by the owner on Apr 25, 2022. It is now read-only.

Add host #22

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add host #22

wants to merge 2 commits into from

Conversation

hbinkim
Copy link
Contributor

@hbinkim hbinkim commented Oct 8, 2021

  • refactor provisioner code
    • verify each step is completed instead of setting provisioner state based on the deploy node condition
    • execute package installation per node instead of looping through nodes to execute each commend
  • refactor controller code
    • separate condition and state update function
  • add deployNode information on CephCluster status
  • add e2e test cases
    • for adding extra osd host
    • for adding extra osd disks and osd host order is different (bootstrap node is mentioned underneath the added osd node)

@github-actions github-actions bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 8, 2021
@github-actions github-actions bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Oct 29, 2021
@github-actions github-actions bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Dec 29, 2021
@hbinkim hbinkim marked this pull request as ready for review January 3, 2022 02:44
- refactor provisioner code
  - verify each step is completed instead of setting provisioner state based on the deploy node condition
  - execute package installation per node instead of looping through nodes to execute each commend
- refactor controller code
  - separate condition and state update function
- add deployNode information on CephCluster status
- add Bootstrapped state on CephCluster
- add e2e test cases
  - for adding extra osd host
  - for adding extra osd disks and osd host order is different (bootstrap node is mentioned underneath the added osd node)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant