[Proposal] Concerning open PRs

2016-02-17 Thread Wilder Rodrigues
Hi all,


It’s been a long time, but the Wolverine is not dead yet. ;)

Currently we have 175 opened PRs, which we all agree to be a lot, given the 
fact that few people, if any, are testing/merging them. I have been a bit off 
the radar, but from next week I will start helping to get some of those PRs 
tested and, hopefully, merged.

In order to get the community working as an unit, I would like to propose the 
following:

* One should only create a PR after testing an existing PR.
  - By testing I mean… testing. Not just looking into it and saying “LGTM”. 
Manual tests should also count, with screenshots attached to the PR.

That will make those with test environment pitch in and help, and in addition 
might also decrease the frenzy for creating PRs which occasionally won’t be 
tested within a month time - or longer.

For others not creating PRs that often, like me, we should help testing at 
least 1 PR per week.

Being a bit more blunt now, if a PR is created but the person does not 
contribute with testing an existing one, the new PR should be closed.

What do you think?

Cheers,
Wilder

[GitHub] cloudstack pull request: SystemVM cleanups

2016-02-17 Thread resmo
Github user resmo commented on the pull request:

https://github.com/apache/cloudstack/pull/1414#issuecomment-185579250
  
Actually the logrotating is running every hour (by size), which can not be 
done with logrotate time interval. We have had problems in the past filling up 
/var (and / when it wasn't a separate partition).

This PR does not change functionality, it just make it explicitely. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9168: Testpath to check if wro...

2016-02-17 Thread pritisarap12
Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1245#issuecomment-185550931
  
Updated the testpath with review comments:
--Validation of netmask 
--check it is from correct CIDR



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9168: Testpath to check if wro...

2016-02-17 Thread pritisarap12
Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1245#issuecomment-185550227
  
Test result after modifications:

Check Netmask value in database ... === TestName: 
test_01_netmask_value_check | Status : SUCCESS ===
ok

--
Ran 1 test in 121.854s

OK



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Strongswan vpn feature

2016-02-17 Thread pdion891
Github user pdion891 commented on the pull request:

https://github.com/apache/cloudstack/pull/872#issuecomment-185506918
  
Hi John,
Here is the l2tp configuration file, IP address voluntarily sabotage:
```
conn L2TP-PSK
authby=secret
pfs=no
rekey=no
keyingtries=3
#
# --
# The VPN server.
#
# Allow incoming connections on the external network interface.
# If you want to use a different interface or if there is no
# defaultroute, you can use:   left=your.ip.addr.ess
#
 left=394.937.948.473
#
leftprotoport=17/1701
# If you insist on supporting non-updated Windows clients,
# you can use:leftprotoport=17/%any
#
# --
# The remote user(s).
#
# Allow incoming connections only from this IP address.
right=%any
# If you want to allow multiple connections from any IP address,
# you can use:right=%any
#
rightprotoport=17/%any
#
# --
# Change 'ignore' to 'add' to enable this configuration.
#
auto=add
```


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8830 CLONE - VM snapshot fails...

2016-02-17 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/798#discussion_r53255088
  
--- Diff: 
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareStorageManagerImpl.java
 ---
@@ -1464,4 +1440,45 @@ private String deleteDir(String dir) {
 private static String getVolumeRelativeDirInSecStroage(long volumeId) {
 return "volumes/" + volumeId;
 }
+
+/**
+ * This method is to check if a given TaskInfo Object is valid( and 
has name and entity name assigned).It return true if TaskInfo Object is valid 
and false otherwise.
+ *
+ * @param TaskInfo
+ *info
+ * @return boolean(true or false)
+ **/
+private boolean isvalidTaskInfoObj(TaskInfo info){
+return !(info == null || info.getEntityName() == null || 
info.getName() == null);
+}
+
+private boolean waitForRunningTaskOnVM(String vmName, String taskName, 
VmwareContext context) throws Exception {
+try {
+ManagedObjectReference taskmgr = 
context.getServiceContent().getTaskManager();
+List tasks = 
context.getVimClient().getDynamicProperty(taskmgr, "recentTask");
+
+for (ManagedObjectReference taskMor : tasks) {
+TaskInfo info = 
(TaskInfo)(context.getVimClient().getDynamicProperty(taskMor, "info"));
+
+if (!isvalidTaskInfoObj(info)) {
--- End diff --

I noticed you are only suggesting ;), no need to say that, when we review a 
PR we suggest changes to improve it, which is normal.

I am going to ask again, Do you mean remove the “isvalidTaskInfoObj” 
and add that condition to the (if)?
That would not reduce the code complexity, in the worst case scenario the 
complexity would be the same. At the line you commented, when we use that 
“isvalidTaskInfoObj” to mask the complexity on the logic to check if a 
task, we can remove some complexity from one method to another; doing that, we 
facilitate code documentation and unit tests writing.

For instance, if you want to suggest something to reduce code complexity or 
the so called cyclomatic complexity, you should take a look at lines 1467 and 
1468, there you have a lot of room for improvements.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8830 CLONE - VM snapshot fails...

2016-02-17 Thread rodrigo93
Github user rodrigo93 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/798#discussion_r53253724
  
--- Diff: 
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareStorageManagerImpl.java
 ---
@@ -1464,4 +1440,45 @@ private String deleteDir(String dir) {
 private static String getVolumeRelativeDirInSecStroage(long volumeId) {
 return "volumes/" + volumeId;
 }
+
+/**
+ * This method is to check if a given TaskInfo Object is valid( and 
has name and entity name assigned).It return true if TaskInfo Object is valid 
and false otherwise.
+ *
+ * @param TaskInfo
+ *info
+ * @return boolean(true or false)
+ **/
+private boolean isvalidTaskInfoObj(TaskInfo info){
+return !(info == null || info.getEntityName() == null || 
info.getName() == null);
+}
+
+private boolean waitForRunningTaskOnVM(String vmName, String taskName, 
VmwareContext context) throws Exception {
+try {
+ManagedObjectReference taskmgr = 
context.getServiceContent().getTaskManager();
+List tasks = 
context.getVimClient().getDynamicProperty(taskmgr, "recentTask");
+
+for (ManagedObjectReference taskMor : tasks) {
+TaskInfo info = 
(TaskInfo)(context.getVimClient().getDynamicProperty(taskMor, "info"));
+
+if (!isvalidTaskInfoObj(info)) {
--- End diff --

Yes, those if's statements could be merged into one if statement. This is 
just one suggestion to make a minor improvement in the complexity.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: question on certificate upload

2016-02-17 Thread Wei ZHOU
Daan,

the new certificate and key are stored in datab
ase, and imported int
o realhostip.keystore
if t
he certificate is u
ploaded together with a key.


2016年2月17日星期三,Daan Hoogland  写道:

> H,
>
> If I upload a new certificate from the infrastructure tab, the timestamp of
> the certificates in the consoleproxy in /usr/local/cloud/systemvm/certs
> stay the same, even after destroying teh cpvm. Am I doing something wrong
> or expecting the wrong thing to happen?
>
> ​thanks,​
> --
> Daan
>


RE: [update] ACS management unable to connect to xenserver hosts after reboot

2016-02-17 Thread Paul Angus
The msid is generated from the MAC address of the host when the service starts, 
the two IDs are subtly different do you have some bonding in place that is 
maybe miss-configured, which is generating the 2nd MAC?



Paul Angus
VP Technology   ,   ShapeBlue


t:  @cloudyangus

e:  paul.an...@shapeblue.com|  
w:  www.shapeblue.com





-Original Message-
From: Simon Weller [mailto:swel...@ena.com]
Sent: Wednesday, February 17, 2016 6:11 PM
To: dev@cloudstack.apache.org
Cc: Glenn Wagner 
Subject: Re: [update] ACS management unable to connect to xenserver hosts after 
reboot

Stephan,

When you restart the management process, do you see any logs indicating it's 
trying to peer with another management server?

- Si


From: Stephan Seitz 
Sent: Wednesday, February 17, 2016 9:28 AM
To: dev@cloudstack.apache.org
Cc: Glenn Wagner
Subject: Re: [update] ACS management unable to connect to xenserver hosts after 
reboot

Glenn,

thanks for your reply. Unfortunately the SSVM has been destroyed.

We don't have any firewall in between. ACS and XenServers are located in the 
same /22. I've double checked every connection and there's no iptables or 
similar in the way.
Instead of the SSVM, I've just successfully checked if the consoleproxy VM is 
able to connect to Port 8250.

To me it looks, like there's some strange "identity" problem.

mysql> select * from mshost;
+++---+--+---+-++--+-+-+-+
| id | msid | runid | name | state |
version | service_ip | service_port | last_update | removed |
alert_count |
+++---+--+---+-++--+-+-+-+
| 1 | 57177340185274 | 1455209855143 | acs-management-1 | Up | 4.7.1
| 10.97.13.1 | 9090 | 2016-02-12 16:55:56 | NULL |
0 |
| 3 | 57177340185273 | 1455639355379 | acs-management-1 | Up | 4.7.1
| 10.97.13.1 | 9090 | 2016-02-17 11:31:50 | NULL |
0 |
+++---+--+---+-++--+-+-+-+
2 rows in set (0.00 sec)

Indeed, there is (and always has been) only one management host in this 
infrastructure.

With sqldumps at hand, we removed the second row and purged all the related 
jobs to that id, but after restarting cloudstack-management, this entry wasi 
created again.

Maybe, I'm completely wrong, but is it possible that our management host 
"thinks" there's another management host responsible for our cluster?

Since we're fiddling at least two days without any success here, I'm willing to 
get a few consulting hours thrown on that.

cheers,

- Stephan

btw. sorry, if this is a double post, but I think the list ate my last mail...


Am Dienstag, den 16.02.2016, 20:39 + schrieb Glenn Wagner:
> Hi Stephan,
>
> Check that you can telnet port 8250 on the management server from SSVM
> , check that iptables has been setup correctly Looks like it’s a
> firewall issue on the ACS Management server
>
> Thanks
> Glenn
>
>
>
>
>
> ShapeBlue
> Glenn Wagner
> Senior
> Consultant
> ,
> ShapeBlue
> d:
> | s: +27 21 527 0091
> |
> m:
> +27 73 917 4111
> e:
> glenn.wag...@shapeblue.com | t:
> |
> w:
> www.shapeblue.com
> a:
> 2nd Floor, Oudehuis Centre, 122 Main Rd, Somerset West Cape Town 7130
> South Africa
>
> Shape Blue Ltd is a company incorporated in England & Wales. ShapeBlue
> Services India LLP is a company incorporated in India and is operated
> under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda
> is a company incorporated in Brasil and is operated under license from
> Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company registered by The
> Republic of South Africa and is traded under license from Shape Blue
> Ltd. ShapeBlue is a registered trademark.
> This email and any attachments to it may be confidential and are
> intended solely for the use of the individual to whom it is addressed.
> Any views or opinions expressed are solely those of the author and do
> not necessarily represent those of Shape Blue Ltd or related
> companies. If you are not the intended recipient of this email, you
> must neither take any action based upon its contents, nor copy or show
> it to anyone. Please contact the sender if you believe you have
> received this email in error.
>
>
>
>
>
> -Original Message-
> From: Stephan Seitz [mailto:s.se...@secretresearchfacility.com]
> Sent: Tuesday, 16 February 2016 5:19 PM
> To: us...@cloudstack.apache.org
> Cc: dev@cloudstack.apache.org
> Subject: [update] ACS management unable to connect to xenserver hosts
> after reboot
>
> Hi again!
>
> I think we've found the root source, but are unable 

Re: question on certificate upload

2016-02-17 Thread Daan Hoogland
Both work but the certificate never gets installed in the cpvm, it gets
loaded in memory directly through the command. thanks.

On Wed, Feb 17, 2016 at 7:01 PM, Remi Bergsma 
wrote:

> Hi Daan,
>
> Did you try the api call instead of the UI? I believe the UI has a bug (it
> filters the private key and it doesn't appear in the db). API call worked
> for me.
>
> Regards, Remi
>
> Sent from my iPhone
>
> > On 17 Feb 2016, at 18:52, Daan Hoogland  wrote:
> >
> > forgot to mention: version 4.7.1 I don't think that matters but still.
> > On Wed, Feb 17, 2016 at 2:19 PM, Daan Hoogland 
> > wrote:
> >
> >> H,
> >>
> >> If I upload a new certificate from the infrastructure tab, the timestamp
> >> of the certificates in the consoleproxy in
> /usr/local/cloud/systemvm/certs
> >> stay the same, even after destroying teh cpvm. Am I doing something
> wrong
> >> or expecting the wrong thing to happen?
> >>
> >> ​thanks,​
> >> --
> >> Daan
> >
> >
> >
> > --
> > Daan
>



-- 
Daan


Re: [update] ACS management unable to connect to xenserver hosts after reboot

2016-02-17 Thread Simon Weller
Stephan,

When you restart the management process, do you see any logs indicating it's 
trying to peer with another management server?

- Si


From: Stephan Seitz 
Sent: Wednesday, February 17, 2016 9:28 AM
To: dev@cloudstack.apache.org
Cc: Glenn Wagner
Subject: Re: [update] ACS management unable to connect to xenserver hosts after 
reboot

Glenn,

thanks for your reply. Unfortunately the SSVM has been destroyed.

We don't have any firewall in between. ACS and XenServers are located in
the same /22. I've double checked every connection and there's no
iptables or similar in the way.
Instead of the SSVM, I've just successfully checked if the consoleproxy
VM is able to connect to Port 8250.

To me it looks, like there's some strange "identity" problem.

mysql> select * from mshost;
+++---+--+---+-++--+-+-+-+
| id | msid   | runid | name | state |
version | service_ip | service_port | last_update | removed |
alert_count |
+++---+--+---+-++--+-+-+-+
|  1 | 57177340185274 | 1455209855143 | acs-management-1 | Up| 4.7.1
| 10.97.13.1 | 9090 | 2016-02-12 16:55:56 | NULL|
0 |
|  3 | 57177340185273 | 1455639355379 | acs-management-1 | Up| 4.7.1
| 10.97.13.1 | 9090 | 2016-02-17 11:31:50 | NULL|
0 |
+++---+--+---+-++--+-+-+-+
2 rows in set (0.00 sec)

Indeed, there is (and always has been) only one management host in this
infrastructure.

With sqldumps at hand, we removed the second row and purged all the
related jobs to that id, but after restarting cloudstack-management,
this entry wasi created again.

Maybe, I'm completely wrong, but is it possible that our management host
"thinks" there's another management host responsible for our cluster?

Since we're fiddling at least two days without any success here, I'm
willing to get a few consulting hours thrown on that.

cheers,

- Stephan

btw. sorry, if this is a double post, but I think the list ate my last
mail...


Am Dienstag, den 16.02.2016, 20:39 + schrieb Glenn Wagner:
> Hi Stephan,
>
> Check that you can telnet port 8250 on the management server from
> SSVM , check that iptables has been setup correctly
> Looks like it’s a firewall issue on the ACS Management server
>
> Thanks
> Glenn
>
>
>
>
>
> ShapeBlue
> Glenn Wagner
> Senior
> Consultant
> ,
> ShapeBlue
> d:
>  | s: +27 21 527 0091
>  |
> m:
> +27 73 917 4111
> e:
> glenn.wag...@shapeblue.com | t:
>  |
> w:
> www.shapeblue.com
> a:
> 2nd Floor, Oudehuis Centre, 122 Main Rd, Somerset West Cape Town 7130 South 
> Africa
>
> Shape Blue Ltd is a company incorporated in England & Wales. ShapeBlue
> Services India LLP is a company incorporated in India and is operated
> under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda
> is a company incorporated in Brasil and is operated under license from
> Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company registered by The
> Republic of South Africa and is traded under license from Shape Blue
> Ltd. ShapeBlue is a registered trademark.
> This email and any attachments to it may be confidential and are
> intended solely for the use of the individual to whom it is addressed.
> Any views or opinions expressed are solely those of the author and do
> not necessarily represent those of Shape Blue Ltd or related
> companies. If you are not the intended recipient of this email, you
> must neither take any action based upon its contents, nor copy or show
> it to anyone. Please contact the sender if you believe you have
> received this email in error.
>
>
>
>
>
> -Original Message-
> From: Stephan Seitz [mailto:s.se...@secretresearchfacility.com]
> Sent: Tuesday, 16 February 2016 5:19 PM
> To: us...@cloudstack.apache.org
> Cc: dev@cloudstack.apache.org
> Subject: [update] ACS management unable to connect to xenserver hosts
> after reboot
>
> Hi again!
>
> I think we've found the root source, but are unable to mitigate that:
>
> 2016-02-16 16:13:22,217 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-8:null) Seq 6--1: MgmtId 57177340185273: Req:
> Routing to peer
> 2016-02-16 16:13:22,217 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-9:null) Seq 6--1: MgmtId 57177340185273: Req:
> Cancel request received
> 2016-02-16 16:13:22,899 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-10:null) Seq 1-4458000681143369786: MgmtId
> 57177340185273: Req: Resource [Host:1] is unreachable: Host 1: Link is
> closed
> 2016-02-16 16:13:22,899 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> 

question on certificate upload

2016-02-17 Thread Daan Hoogland
H,

If I upload a new certificate from the infrastructure tab, the timestamp of
the certificates in the consoleproxy in /usr/local/cloud/systemvm/certs
stay the same, even after destroying teh cpvm. Am I doing something wrong
or expecting the wrong thing to happen?

​thanks,​
-- 
Daan


Re: [RESULT][VOTE] Apache CloudStack 4.7.1/4.8 (and call for new RM)

2016-02-17 Thread Sebastien Goasguen

> On Feb 16, 2016, at 7:01 PM, Simon Weller  wrote:
> 
> Team,
> 
> We have a couple of things we need collaborate on:
> 
> 1. Per Remi's request below, we need some folks familiar with the release 
> process to either help, or document steps required to completed the release 
> notes, docs and website updates.
> 2 Remi has spearheaded the charge for the last couple of releases and has 
> done a fantastic job getting 4.7.x and 4.8 to a very stable point. We now 
> need someone else (preferably more than one) to take over the RM 
> responsibilities for the next release.
> I suggest more than one, so the burden can be shared and we don't burn 
> individuals out.
> 
> i'm more than happy to do my share and help the process along, but I'm not a 
> committer, so we individuals who are to step forward and lead the charge.

It’s worth mentioning a few things that the PMC looks at for committership 
(getting write access to the repos):
- participation on ML
- submitting pull requests
- voting on releases
- helping users on ML or slack or IRC
- helping with testing.

For instance we have lots of PRs in the pipe, anyone with a cloudstack 
installation can help testing those PRs (pull them, build, report test results) 
etc…

For the website, same thing anyone can submit a pull request….

For release votes, anyone can vote (even none committers)

Bottom line, while not being a committer prevents you from writing to the repos 
directly it should not be seen as an impediment.

Say someone starts reviewing all the pending PRs, providing good feedback, 
running tests etc….that would be a good case for committership.



> 
> - Si
> 
> 
> 
> 
> From: Remi Bergsma 
> Sent: Monday, February 8, 2016 3:43 AM
> To: dev@cloudstack.apache.org
> Subject: Re: [RESULT][VOTE] Apache CloudStack 4.7.0
> 
> It would be great if we could work together to complete the remaining items. 
> Including release notes, docs, website updates etc. Haven’t done these myself 
> and until now someone always stepped in and helped :-)
> 
> Thanks,
> Remi
> 
> 
> 
> On 06/02/16 19:29, "Sebastien Goasguen"  wrote:
> 
>> 
>>> On Feb 5, 2016, at 3:08 AM, John Kinsella  wrote:
>>> 
>>> Did the announcements for 4.7/4.8 go out? I don’t see them on the mailing 
>>> lists or elsewhere?
>>> 
>> 
>> 
>> I don’t think it went out, nor do I think there were RN for them or an 
>> update to the website
>> 
 On Dec 17, 2015, at 8:37 AM, Remi Bergsma  
 wrote:
 
 Hi all,
 
 After 72 hours, the vote for CloudStack 4.7.0 [1] *passes* with 5 PMC + 1 
 non-PMC votes.
 
 +1 (PMC / binding)
 * Wilder
 * Wido
 * Milamber
 * Rohit
 * Remi
 
 +1 (non binding)
 * Boris
 
 0
 * Abhinandan
 * Dag
 * Glenn
 
 -1
 Raja (has been discussed, seems local test configure issue)
 
 Thanks to everyone participating.
 
 I will now prepare the release announcement to go out after 24 hours to 
 give the mirrors time to catch up.
 
 [1] http://cloudstack.markmail.org/message/aahz3ajryvd7wzec
 
>>> 
>> 



Re: question on certificate upload

2016-02-17 Thread Remi Bergsma
Hi Daan,

Did you try the api call instead of the UI? I believe the UI has a bug (it 
filters the private key and it doesn't appear in the db). API call worked for 
me. 

Regards, Remi 

Sent from my iPhone

> On 17 Feb 2016, at 18:52, Daan Hoogland  wrote:
> 
> forgot to mention: version 4.7.1 I don't think that matters but still.
> On Wed, Feb 17, 2016 at 2:19 PM, Daan Hoogland 
> wrote:
> 
>> H,
>> 
>> If I upload a new certificate from the infrastructure tab, the timestamp
>> of the certificates in the consoleproxy in /usr/local/cloud/systemvm/certs
>> stay the same, even after destroying teh cpvm. Am I doing something wrong
>> or expecting the wrong thing to happen?
>> 
>> ​thanks,​
>> --
>> Daan
> 
> 
> 
> -- 
> Daan


Re: [update] ACS management unable to connect to xenserver hosts after reboot

2016-02-17 Thread Stephan Seitz
Glenn,

thanks for your reply. Unfortunately the SSVM has been destroyed.

We don't have any firewall in between. ACS and XenServers are located in
the same /22. I've double checked every connection and there's no
iptables or similar in the way.
Instead of the SSVM, I've just successfully checked if the consoleproxy
VM is able to connect to Port 8250.

To me it looks, like there's some strange "identity" problem.

mysql> select * from mshost;
+++---+--+---+-++--+-+-+-+
| id | msid   | runid | name | state |
version | service_ip | service_port | last_update | removed |
alert_count |
+++---+--+---+-++--+-+-+-+
|  1 | 57177340185274 | 1455209855143 | acs-management-1 | Up| 4.7.1
| 10.97.13.1 | 9090 | 2016-02-12 16:55:56 | NULL|
0 |
|  3 | 57177340185273 | 1455639355379 | acs-management-1 | Up| 4.7.1
| 10.97.13.1 | 9090 | 2016-02-17 11:31:50 | NULL|
0 |
+++---+--+---+-++--+-+-+-+
2 rows in set (0.00 sec)

Indeed, there is (and always has been) only one management host in this
infrastructure.

With sqldumps at hand, we removed the second row and purged all the
related jobs to that id, but after restarting cloudstack-management,
this entry wasi created again.

Maybe, I'm completely wrong, but is it possible that our management host
"thinks" there's another management host responsible for our cluster?

Since we're fiddling at least two days without any success here, I'm
willing to get a few consulting hours thrown on that.

cheers,

- Stephan

btw. sorry, if this is a double post, but I think the list ate my last
mail...


Am Dienstag, den 16.02.2016, 20:39 + schrieb Glenn Wagner: 
> Hi Stephan,
> 
> Check that you can telnet port 8250 on the management server from
> SSVM , check that iptables has been setup correctly 
> Looks like it’s a firewall issue on the ACS Management server
> 
> Thanks
> Glenn
> 
> 
> 
> 
> 
> ShapeBlue
> Glenn Wagner
> Senior
> Consultant
> , 
> ShapeBlue
> d: 
>  | s: +27 21 527 0091
>  | 
> m: 
> +27 73 917 4111
> e: 
> glenn.wag...@shapeblue.com | t: 
>  | 
> w: 
> www.shapeblue.com
> a: 
> 2nd Floor, Oudehuis Centre, 122 Main Rd, Somerset West Cape Town 7130 South 
> Africa
> 
> Shape Blue Ltd is a company incorporated in England & Wales. ShapeBlue
> Services India LLP is a company incorporated in India and is operated
> under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda
> is a company incorporated in Brasil and is operated under license from
> Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company registered by The
> Republic of South Africa and is traded under license from Shape Blue
> Ltd. ShapeBlue is a registered trademark.
> This email and any attachments to it may be confidential and are
> intended solely for the use of the individual to whom it is addressed.
> Any views or opinions expressed are solely those of the author and do
> not necessarily represent those of Shape Blue Ltd or related
> companies. If you are not the intended recipient of this email, you
> must neither take any action based upon its contents, nor copy or show
> it to anyone. Please contact the sender if you believe you have
> received this email in error.
> 
> 
> 
> 
> 
> -Original Message-
> From: Stephan Seitz [mailto:s.se...@secretresearchfacility.com] 
> Sent: Tuesday, 16 February 2016 5:19 PM
> To: us...@cloudstack.apache.org
> Cc: dev@cloudstack.apache.org
> Subject: [update] ACS management unable to connect to xenserver hosts
> after reboot
> 
> Hi again!
> 
> I think we've found the root source, but are unable to mitigate that:
> 
> 2016-02-16 16:13:22,217 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-8:null) Seq 6--1: MgmtId 57177340185273: Req:
> Routing to peer
> 2016-02-16 16:13:22,217 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-9:null) Seq 6--1: MgmtId 57177340185273: Req:
> Cancel request received
> 2016-02-16 16:13:22,899 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-10:null) Seq 1-4458000681143369786: MgmtId
> 57177340185273: Req: Resource [Host:1] is unreachable: Host 1: Link is
> closed
> 2016-02-16 16:13:22,899 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-10:null) Seq 1--1: MgmtId 57177340185273: Req:
> Routing to peer
> 2016-02-16 16:13:22,900 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-11:null) Seq 1--1: MgmtId 57177340185273: Req:
> Cancel request received
> 2016-02-16 16:13:22,905 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-12:null) Seq 3-2144839322535198778: MgmtId
> 57177340185273: Req: Resource 

Re: [update] ACS management unable to connect to xenserver hosts after reboot

2016-02-17 Thread Stephan Seitz
Glenn,

thanks for your reply. Unfortunately the SSVM has been destroyed.

We don't have any firewall in between. ACS and XenServers are located in
the same /22. I've double checked every connection and there's no
iptables or similar in the way.
Instead of the SSVM, I've just successfully checked if the consoleproxy
VM is able to connect to Port 8250.

To me it looks, like there's some strange "identity" problem.

mysql> select * from mshost;
+++---+--+---+-++--+-+-+-+
| id | msid   | runid | name | state |
version | service_ip | service_port | last_update | removed |
alert_count |
+++---+--+---+-++--+-+-+-+
|  1 | 57177340185274 | 1455209855143 | acs-management-1 | Up| 4.7.1
| 10.97.13.1 | 9090 | 2016-02-12 16:55:56 | NULL|
0 |
|  3 | 57177340185273 | 1455639355379 | acs-management-1 | Up| 4.7.1
| 10.97.13.1 | 9090 | 2016-02-17 11:31:50 | NULL|
0 |
+++---+--+---+-++--+-+-+-+
2 rows in set (0.00 sec)

Indeed, there is (and always has been) only one management host in this
infrastructure.

With sqldumps at hand, we removed the second row and purged all the
related jobs to that id, but after restarting cloudstack-management,
this entry wasi created again.

Maybe, I'm completely wrong, but is it possible that our management host
"thinks" there's another management host responsible for our cluster?

Since we're fiddling at least two days without any success here, I'm
willing to get a few consulting hours thrown on that.

cheers,

- Stephan

Am Dienstag, den 16.02.2016, 20:39 + schrieb Glenn Wagner: 
> Hi Stephan,
> 
> Check that you can telnet port 8250 on the management server from
> SSVM , check that iptables has been setup correctly 
> Looks like it’s a firewall issue on the ACS Management server
> 
> Thanks
> Glenn
> 
> 
> 
> 
> 
> ShapeBlue
> Glenn Wagner
> Senior
> Consultant
> , 
> ShapeBlue
> d: 
>  | s: +27 21 527 0091
>  | 
> m: 
> +27 73 917 4111
> e: 
> glenn.wag...@shapeblue.com | t: 
>  | 
> w: 
> www.shapeblue.com
> a: 
> 2nd Floor, Oudehuis Centre, 122 Main Rd, Somerset West Cape Town 7130 South 
> Africa
> 
> Shape Blue Ltd is a company incorporated in England & Wales. ShapeBlue
> Services India LLP is a company incorporated in India and is operated
> under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda
> is a company incorporated in Brasil and is operated under license from
> Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company registered by The
> Republic of South Africa and is traded under license from Shape Blue
> Ltd. ShapeBlue is a registered trademark.
> This email and any attachments to it may be confidential and are
> intended solely for the use of the individual to whom it is addressed.
> Any views or opinions expressed are solely those of the author and do
> not necessarily represent those of Shape Blue Ltd or related
> companies. If you are not the intended recipient of this email, you
> must neither take any action based upon its contents, nor copy or show
> it to anyone. Please contact the sender if you believe you have
> received this email in error.
> 
> 
> 
> 
> 
> -Original Message-
> From: Stephan Seitz [mailto:s.se...@secretresearchfacility.com] 
> Sent: Tuesday, 16 February 2016 5:19 PM
> To: us...@cloudstack.apache.org
> Cc: dev@cloudstack.apache.org
> Subject: [update] ACS management unable to connect to xenserver hosts
> after reboot
> 
> Hi again!
> 
> I think we've found the root source, but are unable to mitigate that:
> 
> 2016-02-16 16:13:22,217 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-8:null) Seq 6--1: MgmtId 57177340185273: Req:
> Routing to peer
> 2016-02-16 16:13:22,217 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-9:null) Seq 6--1: MgmtId 57177340185273: Req:
> Cancel request received
> 2016-02-16 16:13:22,899 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-10:null) Seq 1-4458000681143369786: MgmtId
> 57177340185273: Req: Resource [Host:1] is unreachable: Host 1: Link is
> closed
> 2016-02-16 16:13:22,899 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-10:null) Seq 1--1: MgmtId 57177340185273: Req:
> Routing to peer
> 2016-02-16 16:13:22,900 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-11:null) Seq 1--1: MgmtId 57177340185273: Req:
> Cancel request received
> 2016-02-16 16:13:22,905 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (AgentManager-Handler-12:null) Seq 3-2144839322535198778: MgmtId
> 57177340185273: Req: Resource [Host:3] is unreachable: Host 3: Link is
> closed
> 
> Here's a longer excerpt 

Re: question on certificate upload

2016-02-17 Thread Daan Hoogland
forgot to mention: version 4.7.1 I don't think that matters but still.

On Wed, Feb 17, 2016 at 2:19 PM, Daan Hoogland 
wrote:

> H,
>
> If I upload a new certificate from the infrastructure tab, the timestamp
> of the certificates in the consoleproxy in /usr/local/cloud/systemvm/certs
> stay the same, even after destroying teh cpvm. Am I doing something wrong
> or expecting the wrong thing to happen?
>
> ​thanks,​
> --
> Daan
>



-- 
Daan


[GitHub] cloudstack pull request: [master/4.9+] vmware: upgrade mvn sdk dep...

2016-02-17 Thread bhaisaab
Github user bhaisaab closed the pull request at:

https://github.com/apache/cloudstack/pull/1415


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: [master/4.9+] vmware: upgrade mvn sdk dep...

2016-02-17 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1415#issuecomment-185273600
  
Merging this change on #1397 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9252: Support configurable NFS...

2016-02-17 Thread serg38
Github user serg38 commented on the pull request:

https://github.com/apache/cloudstack/pull/1361#issuecomment-185256350
  
If requirements to use particular NFS version is known beforehand  it can 
be specified during the image store creation using details argument 
details[0].key=nfs.version[0].value=3  e.g.

api?command=addImageStore=json=test1=NFS=0d074f25-ed31-482f-8bc5-44c9314fc417=nfs%3A%2F%2F10.1.1.1%2Fnfs%5B0%5D.key=nfs.version%5B0%5D.value=3"
Unfortunately updateImageStore is not yet implemented so for an existing 
image store it needs to be inserted to the table manaully.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: SystemVM cleanups

2016-02-17 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1414#issuecomment-185251677
  
Do we want to rotate logs only based on size?
What about using that “minsize” and the timestamp (daily) 
configuration? I do not know the system vms log growth rate, but that seems a 
better approach.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9252: Support configurable NFS...

2016-02-17 Thread nvazquez
Github user nvazquez commented on the pull request:

https://github.com/apache/cloudstack/pull/1361#issuecomment-185194859
  
Sure @kishankavala, to set NFS version for a store with id 
STORE_ID you'll need to insert a record in 
image_store_details_table which has:
* store_id = STORE_ID
* name = nfs.version
* value = X where X is the desired NFS version e.g. 2, 3, 4


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9252: Support configurable NFS...

2016-02-17 Thread kishankavala
Github user kishankavala commented on the pull request:

https://github.com/apache/cloudstack/pull/1361#issuecomment-185176731
  
@nvazquez
Can you please repond to my earlier question also.
>> How is the nfs.version set in image_store_details table?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9280: Allow system VM volumes ...

2016-02-17 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1406#discussion_r53150413
  
--- Diff: 
engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeObject.java
 ---
@@ -92,7 +92,7 @@ public static VolumeObject getVolumeObject(DataStore 
dataStore, VolumeVO volumeV
 public String getAttachedVmName() {
 Long vmId = volumeVO.getInstanceId();
 if (vmId != null) {
-VMInstanceVO vm = vmInstanceDao.findById(vmId);
+VMInstanceVO vm = vmInstanceDao.findByIdIncludingRemoved(vmId);
--- End diff --

Sorry @ProjectMoon, my mistake (it is all right, the method is implemented 
on GenericDao).


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9280: Allow system VM volumes ...

2016-02-17 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1406#discussion_r53149620
  
--- Diff: 
engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeObject.java
 ---
@@ -92,7 +92,7 @@ public static VolumeObject getVolumeObject(DataStore 
dataStore, VolumeVO volumeV
 public String getAttachedVmName() {
 Long vmId = volumeVO.getInstanceId();
 if (vmId != null) {
-VMInstanceVO vm = vmInstanceDao.findById(vmId);
+VMInstanceVO vm = vmInstanceDao.findByIdIncludingRemoved(vmId);
--- End diff --

@ProjectMoon I couldn't find the "findByIdIncludingRemoved" method 

(https://github.com/apache/cloudstack/blob/master/engine/schema/src/com/cloud/vm/dao/VMInstanceDao.java).

Am I missing something?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9026: Modifying testpath for a...

2016-02-17 Thread pritisarap12
Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1031#issuecomment-185115938
  
Testcase result:

Test migrate Volume (root and data disk) ... === TestName: 
test_01_migrate_root_and_data_disk_nonlive | Status : SUCCESS ===
ok

--
Ran 1 test in 1688.400s

OK



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9066: Update testpath to delet...

2016-02-17 Thread pritisarap12
Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1078#issuecomment-185115973
  
Testcase result:

snapshot hardning ... === TestName: test_01_snapshot_hardning_kvm | Status 
: SUCCESS ===
ok
snapshot hardning ... === SKIP: Skip test for server other than XenServer

--
Ran 2 tests in 1259.500s

OK (SKIP=1)




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8717: Failed to start instance...

2016-02-17 Thread pritisarap12
Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1416#issuecomment-185092942
  
@remibergsma and @runseb  Please give one LGTM to close the PR as it is 
just the change in commit message and PR title .


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8717: Failed to start instance...

2016-02-17 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1416#issuecomment-185092625
  
please ask @remibergsma and @runseb , as they had opinions on it before.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8717: Failed to start instance...

2016-02-17 Thread pritisarap12
Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1416#issuecomment-185092215
  
@DaanHoogland  Please give one LGTM to close the PR as it is just the 
change in commit message and PR title .




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---