[GitHub] [cloudstack-primate] davidjumani commented on issue #847: [BUG] add ldap user button missing

2020-11-09 Thread GitBox


davidjumani commented on issue #847:
URL: 
https://github.com/apache/cloudstack-primate/issues/847#issuecomment-723844734


   @utchoang Please go ahead and raise a PR for the change. Thanks



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.

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




[GitHub] [cloudstack-primate] utchoang opened a new pull request #851: FIX - Show clear the add LDAP account button

2020-11-09 Thread GitBox


utchoang opened a new pull request #851:
URL: https://github.com/apache/cloudstack-primate/pull/851


   @rhtyd @davidjumani cc @svenvogel 
   Fixes #847 
   
![image](https://user-images.githubusercontent.com/13766648/98516333-f4131300-229e-11eb-9ec6-6c19cd484e95.png)
   



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.

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




[GitHub] [cloudstack-primate] utchoang commented on pull request #851: FIX - Show clear the add LDAP account button

2020-11-09 Thread GitBox


utchoang commented on pull request #851:
URL: 
https://github.com/apache/cloudstack-primate/pull/851#issuecomment-723862189


   @blueorangutan package



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.

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




[GitHub] [cloudstack-primate] blueorangutan commented on pull request #851: FIX - Show clear the add LDAP account button

2020-11-09 Thread GitBox


blueorangutan commented on pull request #851:
URL: 
https://github.com/apache/cloudstack-primate/pull/851#issuecomment-723862979


   @utchoang a Jenkins job has been kicked to build primate packages. I'll keep 
you posted as I make progress.



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.

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




[GitHub] [cloudstack-primate] blueorangutan commented on pull request #851: FIX - Show clear the add LDAP account button

2020-11-09 Thread GitBox


blueorangutan commented on pull request #851:
URL: 
https://github.com/apache/cloudstack-primate/pull/851#issuecomment-723868850


   Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian 
:heavy_check_mark:archive.
   QA: http://primate-qa.cloudstack.cloud:8080/client/pr/851 (JID-3649)



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.

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




[GitHub] [cloudstack-primate] DaanHoogland opened a new issue #852: [BUG] ldap add account does not return after selecting AnyDomain filter

2020-11-09 Thread GitBox


DaanHoogland opened a new issue #852:
URL: https://github.com/apache/cloudstack-primate/issues/852


   **Describe the bug**
   https://user-images.githubusercontent.com/2486961/98532340-dbe0cb00-2281-11eb-87eb-06f4b1bafaa9.png";>
   selecting AnyDomain as filter does not return even with empty result set.
   
   as opposed to: 
   https://user-images.githubusercontent.com/2486961/98532495-13e80e00-2282-11eb-9abe-14ecc78c9e86.png";>
   or : 
   https://user-images.githubusercontent.com/2486961/98532596-3548fa00-2282-11eb-8666-54d151810277.png";>
   
   it is not clear to me yet, what the problematic circumstances are. some 
combinations of filter and domain do not return. It seems the an empty result 
set might be the problem.
   
   
   **To Reproduce**
   Steps to reproduce the behavior:
   
   1. Log in as an Admin
   1. Go to 'Accounts --> Add LDAP Account'
   1. select different filter/domain combinations
   
   **Expected behavior**
   A result set should be shown within reasonable time.
   
   **Screenshots**
   see description
   
   **Desktop (please complete the following information):**
- OS: [e.g. Windows, Mac, iOS, Android with device/screen details if 
applicable]
- Browser [e.g. chrome, safari]
- Version [e.g. 22]
   
   **Additional context**
   Add any other context about the problem here.
   



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.

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




[GitHub] [cloudstack-primate] davidjumani commented on pull request #853: addldapaccount: Fix defensive check

2020-11-09 Thread GitBox


davidjumani commented on pull request #853:
URL: 
https://github.com/apache/cloudstack-primate/pull/853#issuecomment-723955079


   @blueorangutan package



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.

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




[GitHub] [cloudstack-primate] davidjumani opened a new pull request #853: addldapaccount: Fix defensive check

2020-11-09 Thread GitBox


davidjumani opened a new pull request #853:
URL: https://github.com/apache/cloudstack-primate/pull/853


   Fixes https://github.com/apache/cloudstack-primate/issues/852



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.

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




[GitHub] [cloudstack-primate] blueorangutan commented on pull request #853: addldapaccount: Fix defensive check

2020-11-09 Thread GitBox


blueorangutan commented on pull request #853:
URL: 
https://github.com/apache/cloudstack-primate/pull/853#issuecomment-723955628


   @davidjumani a Jenkins job has been kicked to build primate packages. I'll 
keep you posted as I make progress.



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.

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




[GitHub] [cloudstack-primate] davidjumani commented on a change in pull request #853: addldapaccount: Fix defensive check

2020-11-09 Thread GitBox


davidjumani commented on a change in pull request #853:
URL: https://github.com/apache/cloudstack-primate/pull/853#discussion_r519741977



##
File path: src/views/iam/AddLdapAccount.vue
##
@@ -283,7 +283,7 @@ export default {
   params.domainid = store.getters.userInfo.domainid
   if (domain) {
 const result = this.listDomains.filter(item => item.name === domain)
-if (result) {
+if (result.length > 0) {

Review comment:
   Can't be none. Either an empty list or a single item





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.

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




[GitHub] [cloudstack-primate] DaanHoogland commented on a change in pull request #853: addldapaccount: Fix defensive check

2020-11-09 Thread GitBox


DaanHoogland commented on a change in pull request #853:
URL: https://github.com/apache/cloudstack-primate/pull/853#discussion_r519741714



##
File path: src/views/iam/AddLdapAccount.vue
##
@@ -283,7 +283,7 @@ export default {
   params.domainid = store.getters.userInfo.domainid
   if (domain) {
 const result = this.listDomains.filter(item => item.name === domain)
-if (result) {
+if (result.length > 0) {

Review comment:
   how about result == None?





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.

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




[GitHub] [cloudstack-primate] davidjumani commented on a change in pull request #853: addldapaccount: Fix defensive check

2020-11-09 Thread GitBox


davidjumani commented on a change in pull request #853:
URL: https://github.com/apache/cloudstack-primate/pull/853#discussion_r519741977



##
File path: src/views/iam/AddLdapAccount.vue
##
@@ -283,7 +283,7 @@ export default {
   params.domainid = store.getters.userInfo.domainid
   if (domain) {
 const result = this.listDomains.filter(item => item.name === domain)
-if (result) {
+if (result.length > 0) {

Review comment:
   Can't be none. Either an empty list of not





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.

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




[GitHub] [cloudstack-primate] blueorangutan commented on pull request #853: addldapaccount: Fix defensive check

2020-11-09 Thread GitBox


blueorangutan commented on pull request #853:
URL: 
https://github.com/apache/cloudstack-primate/pull/853#issuecomment-723961422


   Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian 
:heavy_check_mark:archive.
   QA: http://primate-qa.cloudstack.cloud:8080/client/pr/853 (JID-3650)



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.

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




[GitHub] [cloudstack-primate] DaanHoogland merged pull request #851: FIX - Show clear the add LDAP account button

2020-11-09 Thread GitBox


DaanHoogland merged pull request #851:
URL: https://github.com/apache/cloudstack-primate/pull/851


   



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.

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




[GitHub] [cloudstack-primate] DaanHoogland merged pull request #853: addldapaccount: Fix defensive check

2020-11-09 Thread GitBox


DaanHoogland merged pull request #853:
URL: https://github.com/apache/cloudstack-primate/pull/853


   



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.

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




[GitHub] [cloudstack-primate] DaanHoogland closed issue #847: [BUG] add ldap user button missing

2020-11-09 Thread GitBox


DaanHoogland closed issue #847:
URL: https://github.com/apache/cloudstack-primate/issues/847


   



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.

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




[GitHub] [cloudstack-primate] DaanHoogland closed issue #852: [BUG] ldap add account does not return after selecting AnyDomain filter

2020-11-09 Thread GitBox


DaanHoogland closed issue #852:
URL: https://github.com/apache/cloudstack-primate/issues/852


   



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.

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




[GitHub] [cloudstack-primate] sm97 opened a new issue #854: [BUG] DomainAdmin user has no Configure Limits tab in Project

2020-11-09 Thread GitBox


sm97 opened a new issue #854:
URL: https://github.com/apache/cloudstack-primate/issues/854


   **Describe the bug**
   DomainAdmin role allows to update project resource limits by default but 
Primate hides Configure Limit tab in WEB UI.
   Only SuperAdmin has such tab.
   
   **Expected behavior**
   It's expected that every role with updateResourceLimit permission can use 
Configure Limit tab for project.
   
   **Screenshots**
   Affected account props:
   ![2020-11-09 15_46_30-Accounts - 
CloudStack](https://user-images.githubusercontent.com/9654317/98544827-9df59e00-22a5-11eb-9412-065b0398ea4f.png)
   
   Screen view for admin account;
   ![2020-11-09 15_51_49-Projects - admin - 
CloudStack](https://user-images.githubusercontent.com/9654317/98544828-9f26cb00-22a5-11eb-9927-3d6c7dc2c4fc.png)
   
   Screen view for DomainAdmin role account:
   ![2020-11-09 15_53_24-Projects - domainadmin - 
CloudStack](https://user-images.githubusercontent.com/9654317/98544831-9fbf6180-22a5-11eb-8c8e-cf48ced3205e.png)
   
   



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.

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




[GitHub] [cloudstack-primate] Pearl1594 opened a new pull request #855: Allow domain admins to configure resource limits

2020-11-09 Thread GitBox


Pearl1594 opened a new pull request #855:
URL: https://github.com/apache/cloudstack-primate/pull/855


   Fixes: https://github.com/apache/cloudstack-primate/issues/854



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.

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




[GitHub] [cloudstack-primate] Pearl1594 commented on pull request #855: Allow domain admins to configure resource limits

2020-11-09 Thread GitBox


Pearl1594 commented on pull request #855:
URL: 
https://github.com/apache/cloudstack-primate/pull/855#issuecomment-724010541


   @blueorangutan package



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.

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




[GitHub] [cloudstack-primate] blueorangutan commented on pull request #855: Allow domain admins to configure resource limits

2020-11-09 Thread GitBox


blueorangutan commented on pull request #855:
URL: 
https://github.com/apache/cloudstack-primate/pull/855#issuecomment-724010977


   @Pearl1594 a Jenkins job has been kicked to build primate packages. I'll 
keep you posted as I make progress.



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.

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




[GitHub] [cloudstack-primate] blueorangutan commented on pull request #855: Allow domain admins to configure resource limits

2020-11-09 Thread GitBox


blueorangutan commented on pull request #855:
URL: 
https://github.com/apache/cloudstack-primate/pull/855#issuecomment-724017274


   Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian 
:heavy_check_mark:archive.
   QA: http://primate-qa.cloudstack.cloud:8080/client/pr/855 (JID-3651)



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.

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




[GitHub] [cloudstack-primate] sm97 commented on pull request #855: Allow domain admins to configure resource limits

2020-11-09 Thread GitBox


sm97 commented on pull request #855:
URL: 
https://github.com/apache/cloudstack-primate/pull/855#issuecomment-724039924


   IMHO the better approach is to enable tab based on user permissons not role 
names.



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.

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




[GitHub] [cloudstack-primate] alexandru-bagu closed pull request #839: showing ostypeid input even if we use vmware because cloudstack api requires it

2020-11-09 Thread GitBox


alexandru-bagu closed pull request #839:
URL: https://github.com/apache/cloudstack-primate/pull/839


   



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.

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




[GitHub] [cloudstack-primate] davidjumani commented on pull request #855: Allow domain admins to configure resource limits

2020-11-09 Thread GitBox


davidjumani commented on pull request #855:
URL: 
https://github.com/apache/cloudstack-primate/pull/855#issuecomment-724467666


   @blueorangutan package



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.

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




[GitHub] [cloudstack-primate] blueorangutan commented on pull request #855: Allow domain admins to configure resource limits

2020-11-09 Thread GitBox


blueorangutan commented on pull request #855:
URL: 
https://github.com/apache/cloudstack-primate/pull/855#issuecomment-724468065


   @davidjumani a Jenkins job has been kicked to build primate packages. I'll 
keep you posted as I make progress.



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.

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




[GitHub] [cloudstack-primate] blueorangutan commented on pull request #855: Allow domain admins to configure resource limits

2020-11-09 Thread GitBox


blueorangutan commented on pull request #855:
URL: 
https://github.com/apache/cloudstack-primate/pull/855#issuecomment-724472371


   Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian 
:heavy_check_mark:archive.
   QA: http://primate-qa.cloudstack.cloud:8080/client/pr/855 (JID-3653)



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.

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




[GitHub] [cloudstack-primate] utchoang opened a new pull request #856: FIX - Project invitations - Add new filter state `expired`

2020-11-09 Thread GitBox


utchoang opened a new pull request #856:
URL: https://github.com/apache/cloudstack-primate/pull/856


   @rhtyd @davidjumani cc @svenvogel 
   
![image](https://user-images.githubusercontent.com/13766648/98636511-942d7280-2359-11eb-8d47-b2b3a449803e.png)
   
![image](https://user-images.githubusercontent.com/13766648/98636524-9ee80780-2359-11eb-8837-4ffd7c2da279.png)
   



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.

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




[GitHub] [cloudstack-primate] utchoang opened a new pull request #857: FIX - Domain - Reset tree selected when resetting search.

2020-11-09 Thread GitBox


utchoang opened a new pull request #857:
URL: https://github.com/apache/cloudstack-primate/pull/857


   @rhtyd @davidjumani cc @svenvogel 
   
   - Before:
   
![image](https://user-images.githubusercontent.com/13766648/98636947-5a10a080-235a-11eb-8550-d0de3ac14f62.png)
   
![image](https://user-images.githubusercontent.com/13766648/98636986-67c62600-235a-11eb-94fd-7c1216fe9339.png)
   
   - After:
   
![image](https://user-images.githubusercontent.com/13766648/98637006-73b1e800-235a-11eb-9b58-b85910abe762.png)
   
![image](https://user-images.githubusercontent.com/13766648/98637016-7b718c80-235a-11eb-841a-5451b6459f9a.png)
   



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.

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




[GitHub] [cloudstack-primate] utchoang commented on pull request #856: FIX - Project invitations - Add new filter state `expired`

2020-11-09 Thread GitBox


utchoang commented on pull request #856:
URL: 
https://github.com/apache/cloudstack-primate/pull/856#issuecomment-724496134


   @blueorangutan package



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.

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




[GitHub] [cloudstack-primate] blueorangutan commented on pull request #856: FIX - Project invitations - Add new filter state `expired`

2020-11-09 Thread GitBox


blueorangutan commented on pull request #856:
URL: 
https://github.com/apache/cloudstack-primate/pull/856#issuecomment-724497218


   @utchoang a Jenkins job has been kicked to build primate packages. I'll keep 
you posted as I make progress.



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.

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




[GitHub] [cloudstack-primate] blueorangutan commented on pull request #856: FIX - Project invitations - Add new filter state `expired`

2020-11-09 Thread GitBox


blueorangutan commented on pull request #856:
URL: 
https://github.com/apache/cloudstack-primate/pull/856#issuecomment-724501559


   Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian 
:heavy_check_mark:archive.
   QA: http://primate-qa.cloudstack.cloud:8080/client/pr/856 (JID-3654)



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.

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




[GitHub] [cloudstack-documentation] sureshanaparti opened a new pull request #169: Updated docs with the new storage plugin for Dell EMC PowerFlex/ScaleIO (formerly VxFlexOS)

2020-11-09 Thread GitBox


sureshanaparti opened a new pull request #169:
URL: https://github.com/apache/cloudstack-documentation/pull/169


   Added new storage plugin (Dell EMC PowerFlex/ScaleIO) to the documentation.



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.

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




[GitHub] [cloudstack-primate] utchoang commented on pull request #857: FIX - Domain - Reset tree selected when resetting search.

2020-11-09 Thread GitBox


utchoang commented on pull request #857:
URL: 
https://github.com/apache/cloudstack-primate/pull/857#issuecomment-724509214


   @blueorangutan package



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.

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




[GitHub] [cloudstack-primate] blueorangutan commented on pull request #857: FIX - Domain - Reset tree selected when resetting search.

2020-11-09 Thread GitBox


blueorangutan commented on pull request #857:
URL: 
https://github.com/apache/cloudstack-primate/pull/857#issuecomment-724509724


   @utchoang a Jenkins job has been kicked to build primate packages. I'll keep 
you posted as I make progress.



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.

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




[GitHub] [cloudstack-primate] blueorangutan commented on pull request #857: FIX - Domain - Reset tree selected when resetting search.

2020-11-09 Thread GitBox


blueorangutan commented on pull request #857:
URL: 
https://github.com/apache/cloudstack-primate/pull/857#issuecomment-724513571


   Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian 
:heavy_check_mark:archive.
   QA: http://primate-qa.cloudstack.cloud:8080/client/pr/857 (JID-3655)



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.

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




[GitHub] [cloudstack-primate] davidjumani opened a new pull request #858: kubernetesservicetab: Fixing instance tab

2020-11-09 Thread GitBox


davidjumani opened a new pull request #858:
URL: https://github.com/apache/cloudstack-primate/pull/858


   Show ip or ssh port based on the network type



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.

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