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

Add containerinfra nodegroup support #1364

Merged

Commits on Jun 16, 2022

  1. Add containerinfra nodegroup resource

    Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
    MrFreezeex committed Jun 16, 2022
    Configuration menu
    Copy the full SHA
    8eae6b3 View commit details
    Browse the repository at this point in the history
  2. Add containerinfra nodegroup data source

    Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
    MrFreezeex committed Jun 16, 2022
    Configuration menu
    Copy the full SHA
    62af5fb View commit details
    Browse the repository at this point in the history
  3. Add containerinfra cluster zero node count support

    Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
    MrFreezeex committed Jun 16, 2022
    Configuration menu
    Copy the full SHA
    35d9444 View commit details
    Browse the repository at this point in the history
  4. Add containerinfra nodegroup zero node count support

    Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
    MrFreezeex committed Jun 16, 2022
    Configuration menu
    Copy the full SHA
    935caec View commit details
    Browse the repository at this point in the history
  5. Add containerinfra nodegroup resize node count support

    Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
    MrFreezeex committed Jun 16, 2022
    Configuration menu
    Copy the full SHA
    d851e2c View commit details
    Browse the repository at this point in the history

Commits on Jun 17, 2022

  1. Fix containerinfra cluster mergelabels behavior

    When you add mergelabels and you actually add labels, the magnum API
    will actually return you the complete set of labels under the "labels"
    key. It means that if you want to add/override a couple of label while
    setting merge_labels the creation will succeed but then terraform will
    detect the rest of the labels as changed and will try to rebuid the
    cluster.
    
    This commit instead use the keys labels_{added,skipped,overriden}
    when merge labels which contains the labels that the user has overriden.
    
    Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
    MrFreezeex committed Jun 17, 2022
    Configuration menu
    Copy the full SHA
    1228248 View commit details
    Browse the repository at this point in the history
  2. Add containerinfra nodegroup support for mergelabels

    Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
    MrFreezeex committed Jun 17, 2022
    Configuration menu
    Copy the full SHA
    d908d53 View commit details
    Browse the repository at this point in the history

Commits on Jun 20, 2022

  1. Fix container infra nodegroup golangci-lint errors

    Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
    MrFreezeex committed Jun 20, 2022
    Configuration menu
    Copy the full SHA
    cd0046d View commit details
    Browse the repository at this point in the history

Commits on Jun 22, 2022

  1. Fix container infra nodegroup image_id and flavor_id

    Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
    MrFreezeex committed Jun 22, 2022
    Configuration menu
    Copy the full SHA
    214e4c1 View commit details
    Browse the repository at this point in the history
  2. Fix containerinfra cluster template image/flavor_id

    Previously the code set image_id and flavor_id which doesn't exist
    instead of image/flavor. It would be way more meaningful to have argument
    image_id and flavor_id instead but for backward compatibility let's not
    do that for now.
    
    Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
    MrFreezeex committed Jun 22, 2022
    Configuration menu
    Copy the full SHA
    bb195a3 View commit details
    Browse the repository at this point in the history
  3. Add OS_MAGNUM_IMAGE to containerinfra acceptance tests

    As fedora atomic is no more available, we remove it from the code and
    add OS_MAGNUM_IMAGE environment variable so that we can a supported
    image in the CI later on.
    
    Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
    MrFreezeex committed Jun 22, 2022
    Configuration menu
    Copy the full SHA
    b9064e6 View commit details
    Browse the repository at this point in the history
  4. Make container infra cluster aware of default worker nodegroup

    The node_count returned in magnum for the cluster is the sum of all
    nodegroup. So once you add a node_count, the correct value is actually
    the node_count of the default worker node.
    
    This commit take the node_count of this default worker node group in
    priority and if there are any errors it takes the node_count of the
    cluster instead.
    
    Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
    MrFreezeex committed Jun 22, 2022
    Configuration menu
    Copy the full SHA
    447b02a View commit details
    Browse the repository at this point in the history
  5. Fix container infra nodegroup acceptance tests

    Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
    MrFreezeex committed Jun 22, 2022
    Configuration menu
    Copy the full SHA
    8d1e619 View commit details
    Browse the repository at this point in the history