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

Better handling of optional roles (min/default cardinality 0) #1019

Closed
OpenNebulaProject opened this issue Nov 20, 2017 · 9 comments
Closed

Comments

@OpenNebulaProject
Copy link


Author Name: EOLE Team (EOLE Team)
Original Redmine Issue: 4310, https://dev.opennebula.org/issues/4310
Original Date: 2016-01-27
Original Assignee: Abel Coronado


Hello,

I just created a service with optional roles:

<code class="javascript">
    {
      "name": "PC_admin",
      "cardinality": 0,
      "vm_template": 12780,
      "parents": [
        "Horus"
      ],
      "min_vms": 0,
      "max_vms": 5,
      "elasticity_policies": [

      ],
      "scheduled_policies": [

      ]
    },
</code>

Here is my review:

  • the cloud view does not support well:
    ** the @0@ cardinality, the cursor is outside the cardinality bar
    ** it's not possible to reset the cardinality @0@
  • starting the service works as expected, the optional VMs are not deployed but their status is @running@:
ROLE PC_admin
ROLE STATE          : RUNNING
PARENTS             : Horus
VM TEMPLATE         : 12780
CARDINALITY         : 0
MIN VMS             : 0
MAX VMS             : 5
NODES INFORMATION
 VM_ID NAME                    STAT UCPU    UMEM HOST                       TIME


  • scaling the optional VMs works, but I can not set it to @0@

Regards.

@OpenNebulaProject
Copy link
Author


Original Redmine Comment
Author Name: EOLE Team (EOLE Team)
Original Date: 2016-07-19T10:19:00Z


I tested with 5.0 and the issue is still present.

@OpenNebulaProject
Copy link
Author


Original Redmine Comment
Author Name: EOLE Team (EOLE Team)
Original Date: 2017-06-08T14:30:35Z


I tested with 5.4 and things are a little better except for cloud view:

  • We can define a role with @min VMs@ at @0@: attachment:flow-define-role-with-min-0-max-1.png
  • The service can be deployed with a cardinality of @0@: attachment:flow-service-OK-without-VM.png
  • The cloud view does not permit to scale the role, either up or down: attachment:flow-cloud-view-can-not-scale.png
  • The user view permit to scale the role
    ** Scale up
    *** attachment:flow-user-view-can-scale-up.png
    *** attachment:flow-user-view-scaling-up.png
    ** Scale down
    *** attachment:flow-user-view-can-scale-down.png
    *** attachment:flow-user-view-scaling-down.png
  • The user view permit to choose a value outside @min VMs@ and @max VMs@: attachment:flow-user-view-do-not-limit-scale.png

@OpenNebulaProject
Copy link
Author


Original Redmine Comment
Author Name: EOLE Team (EOLE Team)
Original Date: 2017-06-08T14:31:15Z


Add missing screenshots.

@OpenNebulaProject
Copy link
Author


Original Redmine Comment
Author Name: EOLE Team (EOLE Team)
Original Date: 2017-06-08T14:32:40Z


Note that the error message when I tried to scale too much is not very informative.

Regards.

@stale
Copy link

stale bot commented Mar 6, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. The OpenNebula Dev Team

@tinova
Copy link
Member

tinova commented Mar 6, 2019

Relevant for OneFlow revamp

@rsmontero rsmontero modified the milestones: Release 5.10, Release 5.12 Sep 5, 2019
@tinova tinova modified the milestones: Release 5.12, Release 5.12.1 Jun 16, 2020
@tinova tinova removed this from the Release 5.12.1 milestone Jul 8, 2020
@rsmontero
Copy link
Member

Onflow handles this type of flows. This issue is about rendering 0 VMs roles in Sunstone/Cloud view

@sbbarragan
Copy link

sbbarragan commented Jan 28, 2021

PRs to merge:

tinova added a commit to OpenNebula/docs that referenced this issue Jan 28, 2021
Co-authored-by: Tino Vázquez <cvazquez@opennebula.io>
@tinova
Copy link
Member

tinova commented Jan 28, 2021

needs testing

rsmontero pushed a commit that referenced this issue Feb 10, 2021
Co-authored-by: Tino Vázquez <cvazquez@opennebula.io>
rsmontero pushed a commit that referenced this issue Feb 10, 2021
rsmontero pushed a commit that referenced this issue Mar 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants