You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2022/11/25 15:23:17 UTC

[GitHub] [cloudstack] alexandremattioli opened a new issue, #6923: Adding new ESX cluster allows for operator to enter data in immutable fields

alexandremattioli opened a new issue, #6923:
URL: https://github.com/apache/cloudstack/issues/6923

   <!--
   Verify first that your issue/request is not already reported on GitHub.
   Also test if the latest release and main branch are affected too.
   Always add information AFTER of these HTML comments, but no need to delete the comments.
   -->
   
   ##### ISSUE TYPE
   <!-- Pick one below and delete the rest -->
    * Enhancement Request
   
   ##### COMPONENT NAME
   <!--
   UI
   -->
   ~~~
   
   ~~~
   
   ##### CLOUDSTACK VERSION
   <!--
   4.17.2.0
   -->
   
   ~~~
   
   ~~~
   
   
   
   ##### OS / ENVIRONMENT
   <!--
   ESX
   -->
   
   
   ##### SUMMARY
   A Cloudstack Zone is associated to a single vCenter, ACS holds the details for vCenter IP, username and password in its database.
   When adding a new cluster to a pod the dialog allows for an end user to enter the following non mandatory fields:
   vCenter Host
   vCenter Username
   vCenter password
   vCenter datacenter
   
   If this information is not entered the information in the database is used.
   If the operator tries to enter a different vCenter host and credentials, then the operation fails with an error message telling the user to make sure the cluster being added belongs to the correct vCenter and VMWare DC
   
   If no different values can be used, then the fields shouldn't be displayed. 
   The presence of this fields leads to an operator to think that multiple vCenters are supported
   
   ##### STEPS TO REPRODUCE
   <!--
   In the UI navigate to Infrastructure->Pods
   Add new POD
   Select Hypervisor = VMware
   -->
   
   <!-- Paste example playbooks or commands between quotes below -->
   ~~~
   
   ~~~
   
   <!-- You can also paste gist.github.com links for larger files -->
   
   ##### EXPECTED RESULTS
   <!-- What did you expect to happen when running the steps above? -->
   For immutable fields to not being shown as mutable.
   ~~~
   
   ~~~
   
   ##### ACTUAL RESULTS
   <!-- What actually happened? -->
   Immutable fields are shown as mutable to the operator
   <!-- Paste verbatim command output between quotes below -->
   ~~~
   
   ~~~
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [cloudstack] weizhouapache commented on issue #6923: Adding new ESX cluster allows for operator to enter data in immutable fields

Posted by GitBox <gi...@apache.org>.
weizhouapache commented on issue #6923:
URL: https://github.com/apache/cloudstack/issues/6923#issuecomment-1387081217

   @alexandremattioli 
   
   do you mean that we should not allow to add another vmware cluster, if the zone already has a vmware cluster ?
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [cloudstack] DaanHoogland commented on issue #6923: Adding new ESX cluster allows for operator to enter data in immutable fields

Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on issue #6923:
URL: https://github.com/apache/cloudstack/issues/6923#issuecomment-1365920274

   @alexandremattioli is this just the UI or also the API? (also I would also argue it might be a bug, but I donĀ“t fully understand the issue, yet)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [cloudstack] alexandremattioli commented on issue #6923: Adding new ESX cluster allows for operator to enter data in immutable fields

Posted by GitBox <gi...@apache.org>.
alexandremattioli commented on issue #6923:
URL: https://github.com/apache/cloudstack/issues/6923#issuecomment-1387119809

   @weizhouapache definitely should allow for more clusters. The issue is that the UI asks for information that it shoudn't. 
   @DaanHoogland I'm not sure if that's also the case for the API.  I can show you the UI issue if you want.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org