[GitHub] cloudstack pull request: Update 4.5 branch with L10N resources fil...

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1099#issuecomment-158642997
  
LGTM, thanks @milamberspace 


---
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: Fixed return type Void to void in DataMot...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/969#issuecomment-158673227
  
Hi @rodrigo93 Thanks for rebasing, but please make sure you have one commit 
(and get rid of the merge commit). Thanks!


---
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: Create test cases to getPatchFilePath met...

[pedro-martins]

Github user pedro-martins commented on the pull request:

https://github.com/apache/cloudstack/pull/944#issuecomment-158667466
  
merged with master top version, someone can check this PR? 
=)  thx guys. 


---
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: Update 4.5 branch with L10N resources fil...

[milamberspace]

GitHub user milamberspace opened a pull request:

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

Update 4.5 branch with L10N resources files from Transifex 20151120


Open PR before the 4.5.3 release to update the L10N resources files from 
Transifex to the 4.5 branch.

Merge this PR just before the start of the release process.

cc @bhaisaab 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/milamberspace/cloudstack 
L10N-L10N-update-4.5-20151120

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1099.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1099


commit 85bffef61755022a140266c5d95da203da6e0166
Author: Milamber 
Date:   2015-11-21T11:49:52Z

Update 4.5 branch with L10N resources files from Transifex 20151120




---
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-9065: Packaging RPM, add optio...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1075#issuecomment-158668313
  
LGTM, based on a set of tests that I run on this branch. Screenshot is from 
an experimental Jenkins job (that runs the same tests I did manually until 
now). The tests were run on KVM and used the RPMs from this branch so at least 
those work fine and nothing else broke.

https://cloud.githubusercontent.com/assets/1630096/11319800/da296640-9083-11e5-9702-514a06307a46.png;>



---
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: Removal of DefaultUserAuthenticator empty...

[alexandrelimassantana]

GitHub user alexandrelimassantana opened a pull request:

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

Removal of DefaultUserAuthenticator empty class.

The DefaultUserAuthenticator is an empty class, extending from the
AdapterBase and implementing the UserAuthenticator interface. The class
is not being used as a marker and it's sole use is to be extended by
other UserAuthenticators. Noticing that the class had no purpose, I
removed it and made it's children extend from it's superclass and
implement it's interface instead. Also, I removed the @Local tags from
those classes, since EJB is not used anymore.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/rafaelweingartner/cloudstack 
master-lrg-cs-hackday-005

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1100.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1100


commit cd57451f4b0ea7e42ad5408486e80f43a55367af
Author: alexandre de Limas Santana 
Date:   2015-11-21T18:27:07Z

Removal of DefaultUserAuthenticator empty class.

The DefaultUserAuthenticator is an empty class, extending from the
AdapterBase and implementing the UserAuthenticator interface. The class
is not being used as a marker and it's sole use is to be extended by
other UserAuthenticators. Noticing that the class had no purpose, I
removed it and made it's children extend from it's superclass and
implement it's interface instead. Also, I removed the @Local tags from
those classes, since EJB is not used anymore.




---
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-8988: Removed unused adapters ...

[rodrigo93]

Github user rodrigo93 commented on the pull request:

https://github.com/apache/cloudstack/pull/943#issuecomment-158671944
  
PR merged with master.


---
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: Fixed return type Void to void in DataMot...

[rodrigo93]

Github user rodrigo93 commented on the pull request:

https://github.com/apache/cloudstack/pull/969#issuecomment-158671871
  
Just to update, I merged the commit with master.


---
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.
---

Getting re-started

[Wolf Halton]

I am a little embarrassed, but I cannot log in to the analysis site with my
apache username.
1. I could have just forgotten the password, so I reset it.
2. My account may have been approved to get access to a different apache
project, and I missed a step to get set up here.
3. Could be something totally else.
In OpenOffice, I was an initial committer, and focused most on user
support, strategy and marketing.  I never committed any code, so I never
bothered to learn the committing drill.  It seems lazy now, but at the
time, it made sense.  I want to work on bugs, like cleaning up the code,
and I might be technically up to the challenge.
Next steps?

Wolf Halton
Mobile/Text 678-687-6104
--
Expand Your Vision = Enhance Your Impact
Disaster Recovery in the Cloud
 -
http://atlantacloudtech.com/painless-disaster-recovery/
cloudtech.com/disaster-recovery

[GitHub] cloudstack pull request: CLOUDSTACK-8969 VPN customer gateway can'...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/955#issuecomment-158673310
  
@SudharmaJain Any update on this? Please respond to the comments, thanks!


---
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: L10N : Updated locale files with latest t...

[milamberspace]

Github user milamberspace commented on the pull request:

https://github.com/apache/cloudstack/pull/1097#issuecomment-158636704
  
@barunhalder1234 I thinks you don't use the Transifex website to update the 
translations?

For example on Transifex (for 4.6/4.7 resources) the Japanese translation 
for the key label.ha.enabled (HA Enable in US string) is: 
高可用性有効
(translate in Google Translator: Enable high availability)

But in your PR the same key is translated with:
高可用性を有効にする
(translate in Google Translator: We want to enable high availability)

(curiously the current japanese string on transifex seems better than your 
update string...)

The issue is that the transifex Japanese / Chinese resources files aren't 
up-to-date, so next time, when we update the resources files from transifex to 
the git repo, we lost all your updated strings.

The right way to translate the Web UI of CloudStack is to use the Transifex 
project [1] and use the script inside the cloudstack project to download and 
prepare the resources files (unicode/native conversion and alphabetical order) 
[2]

To deal with this issue, you can update the transifex JA and CH resources 
files with the Transifex client. (See this script [3]) I can do this if you 
prefer.

[1] https://www.transifex.com/ke4qqq/CloudStack_UI/
[2] 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Update+L10N+files+from+Transifex+to+git+repo
[3] 
https://github.com/apache/cloudstack/blob/master/tools/transifex/sync-transifex-ui.sh

cc @remibergsma 


---
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: Getting re-started

[Daan Hoogland]

Wolf, I am not sure if access to analysis is default for apache account
holders. Ask infra ;)

On Sat, Nov 21, 2015 at 3:55 PM, Wolf Halton  wrote:

> I am a little embarrassed, but I cannot log in to the analysis site with my
> apache username.
> 1. I could have just forgotten the password, so I reset it.
> 2. My account may have been approved to get access to a different apache
> project, and I missed a step to get set up here.
> 3. Could be something totally else.
> In OpenOffice, I was an initial committer, and focused most on user
> support, strategy and marketing.  I never committed any code, so I never
> bothered to learn the committing drill.  It seems lazy now, but at the
> time, it made sense.  I want to work on bugs, like cleaning up the code,
> and I might be technically up to the challenge.
> Next steps?
>
> Wolf Halton
> Mobile/Text 678-687-6104
> --
> Expand Your Vision = Enhance Your Impact
> Disaster Recovery in the Cloud
>  -
> http://atlantacloudtech.com/painless-disaster-recovery/
> cloudtech.com/disaster-recovery
>



-- 
Daan

[GitHub] cloudstack pull request: Cwe 190

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1057#issuecomment-158664606
  
@remibergsma for me test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL 
fails. That is not the one you meant is it? I also saw the failure you had.


---
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: Add47xmessagesproperties transifex

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1098#issuecomment-158667887
  
@milamberspace looks ok, but ii know not enough of the transifex mechs to 
be sure. Can you add some explanation?


---
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-9065: Packaging RPM, add optio...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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: Create test cases to getPatchFilePath met...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/944#issuecomment-158673838
  
Hi @pedro-martins I can run some tests but please remove the merge commit. 
Thanks!


---
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: SR_BACKEND_FAILURE_61File SR creation error [opterr=fail to mount FileSR. Errno is 32]

[B Prakash]

Thanks Devdeep, I had missed seeding secondary storage, it is working fine 
after doing the same.
regards,
Prakash

From: Devdeep 
Sent: Saturday, November 21, 2015 3:22 AM
To: dev@cloudstack.apache.org
Subject: Re: SR_BACKEND_FAILURE_61File SR creation error [opterr=fail to mount 
FileSR. Errno is 32]

Have you seeded the secondary storage with the default systemvm template?

Regards,
Devdeep

On Sat, Nov 21, 2015 at 8:33 AM, B Prakash  wrote:

> Thanks Jayapal, Abhinandan.
>
> log SMlog doesnt giveout much, I am able to mount and edit file on mount
> from both server and agent. I get the same error as follows (timestamp is
> different because I tried it again), is there anything else I am missing?:
>
> Nov 20 16:22:01 agent SM: [5432] ['mount', '--bind',
> '/var/cloud_mount/b61521bd-ad7b-3e95-91e8-00762b28aeeb/template/tmpl/1/1',
> '/var/run/sr-mount/e18ee8da-a45d-46b8-b361-ec50e9ddcb22']
> Nov 20 16:22:01 agent SM: [5432] FAILED in util.pread: (rc 32) stdout: '',
> stderr: 'mount: special device
> /var/cloud_mount/b61521bd-ad7b-3e95-91e8-00762b28aeeb/template/tmpl/1/1
> does not exist
> Nov 20 16:22:01 agent SM: [5432] '
> Nov 20 16:22:02 agent SM: [5432] Raising exception [61, File SR creation
> error [opterr=fail to mount FileSR. Errno is 32]]
> Nov 20 16:22:02 agent SM: [5432] lock: released
> /var/lock/sm/e18ee8da-a45d-46b8-b361-ec50e9ddcb22/sr
> Nov 20 16:22:02 agent SM: [5432] * generic exception: sr_attach:
> EXCEPTION SR.SROSError, File SR creation error [opterr=fail to mount
> FileSR. Errno is 32]
> Nov 20 16:22:02 agent SM: [5432]   File "/opt/xensource/sm/SRCommand.py",
> line 110, in run
> Nov 20 16:22:02 agent SM: [5432] return self._run_locked(sr)
> Nov 20 16:22:02 agent SM: [5432]   File "/opt/xensource/sm/SRCommand.py",
> line 159, in _run_locked
> Nov 20 16:22:02 agent SM: [5432] rv = self._run(sr, target)
> Nov 20 16:22:02 agent SM: [5432]   File "/opt/xensource/sm/SRCommand.py",
> line 322, in _run
> Nov 20 16:22:02 agent SM: [5432] return
> sr.attach(self.params['sr_uuid'])
> Nov 20 16:22:02 agent SM: [5432]   File "/opt/xensource/sm/FileSR", line
> 164, in attach
> Nov 20 16:22:02 agent SM: [5432] opterr='fail to mount FileSR. Errno
> is %s' % inst.code)
> Nov 20 16:22:02 agent SM: [5432]   File "/opt/xensource/sm/xs_errors.py",
> line 52, in __init__
> Nov 20 16:22:02 agent SM: [5432] raise SR.SROSError(errorcode,
> errormessage)
> Nov 20 16:22:02 agent SM: [5432]
> Nov 20 16:22:02 agent SM: [5432] lock: closed
> /var/lock/sm/e18ee8da-a45d-46b8-b361-ec50e9ddcb22/sr
> Nov 20 16:22:30 agent SM: [5587] lock: opening lock file
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/sr
> Nov 20 16:22:30 agent SM: [5587] lock: acquired
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/sr
> Nov 20 16:22:30 agent SM: [5587] sr_attach {'sr_uuid':
> '485be965-9bcd-4e76-8fa8-5881f0203645', 'subtask_of':
> 'DummyRef:|fd400425-8c78-85c2-949c-230c0c39e03f|SR.attach', 'args': [],
> 'host_ref': 'OpaqueRef:274f3e03-3513-61a8-40de-5b1ece53c1d7',
> 'session_ref': 'OpaqueRef:5706f0b6-0fcb-7c5c-defa-36e537655059',
> 'device_config': {'SRmaster': 'true', 'location':
> '/var/cloud_mount/b61521bd-ad7b-3e95-91e8-00762b28aeeb/template/tmpl/1/1'},
> 'command': 'sr_attach', 'sr_ref':
> 'OpaqueRef:5906daec-9b86-5fa3-58bc-e95969b77a4d'}
> Nov 20 16:22:30 agent SM: [5587] lock: opening lock file
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/running
> Nov 20 16:22:30 agent SM: [5587] lock: opening lock file
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/sr
> Nov 20 16:22:30 agent SM: [5587] lock: acquired
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/running
> Nov 20 16:22:30 agent SM: [5587] lock: acquired
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/sr
> Nov 20 16:22:30 agent SM: [5587] RESET for SR
> 485be965-9bcd-4e76-8fa8-5881f0203645 (master: True)
> Nov 20 16:22:30 agent SM: [5587] lock: released
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/sr
> Nov 20 16:22:30 agent SM: [5587] lock: released
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/running
> Nov 20 16:22:30 agent SM: [5587] lock: closed
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/sr
> Nov 20 16:22:30 agent SM: [5587] lock: closed
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/running
> Nov 20 16:22:30 agent SM: [5587] set_dirty
> 'OpaqueRef:5906daec-9b86-5fa3-58bc-e95969b77a4d' succeeded
> Nov 20 16:22:30 agent SM: [5587] ['mount', '--bind',
> '/var/cloud_mount/b61521bd-ad7b-3e95-91e8-00762b28aeeb/template/tmpl/1/1',
> '/var/run/sr-mount/485be965-9bcd-4e76-8fa8-5881f0203645']
> Nov 20 16:22:30 agent SM: [5587] FAILED in util.pread: (rc 32) stdout: '',
> stderr: 'mount: special device
> /var/cloud_mount/b61521bd-ad7b-3e95-91e8-00762b28aeeb/template/tmpl/1/1
> does not exist
> Nov 20 16:22:30 agent SM: [5587] '
> Nov 20 16:22:30 agent SM: [5587] Raising exception [61, File SR creation
> error 

[GitHub] cloudstack pull request: CLOUDSTACK-8751 minimise downtime of netw...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/866#issuecomment-158685221
  
good that you found the time to implement this feature @bvbharatk , thanks. 
an integrations test for this is needed as we will judge future PRs against it. 
As in, they may not break this. I have no problem with the code if it can be 
shown to do what it is supposed to. If I find time I will help to prove it but 
that won't be the coming weeks.


---
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-8988: Removed unused adapters ...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/943#issuecomment-158675017
  
Hi @rodrigo93 in the comments above it was suggested to change the commit 
title, can you do that please? The commit title is the first line of the commit 
message, and that should be max about 70 characters long. Followed by an empty 
line, and then whatever it is you want to say more (probably what you start 
with now). In short: add a short descriptive title and an empty line and force 
push the commit again. Ping me and I'll run soms tests on this branch. Thanks!


---
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: [FUTURE]CLOUDSTACK-8907: POD Gateway opti...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/886#issuecomment-158676481
  
@borisroman Can you have a look and resolve the conflict? When we find 
another reviewer we can merge this in 4.7. Pinging @DaanHoogland @bhaisaab 


---
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: Add select template dropdown when reinsta...

[ustcweizhou]

GitHub user ustcweizhou opened a pull request:

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

Add select template dropdown when reinstall VM



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ustcweizhou/cloudstack 
add-select-template-dropdown

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1101.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1101


commit 6e5a05d5c97f96afa656596016fb11df6c222dfb
Author: Wei Zhou 
Date:   2015-01-16T12:53:19Z

Add select template dropdown when reinstall VM




---
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: Add47xmessagesproperties transifex

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1098#issuecomment-158686667
  
Tnx Do we have a page on cwiki on this already?


---
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-8751 minimise downtime of netw...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/866#issuecomment-158676663
  
Pinging @wilderrodrigues to have a look.

@bvbharatk Is there any (integration) test to test this feature?


---
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-9068: Listing Port Forwarding ...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1081#issuecomment-158678341
  
Thanks for the fix @nitin-maharana. Tried it and it works great! LGTM

https://cloud.githubusercontent.com/assets/1630096/11320287/64aad05a-9094-11e5-90b8-3a69ff68b307.png;>

Pinging @DaanHoogland @bhaisaab to review.


---
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: Removed unnecessary @Local annotations an...

[cristofolini]

GitHub user cristofolini opened a pull request:

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

Removed unnecessary @Local annotations and their respective imports

Following @rafaelweingartner 's findings in PR #714 that many spring beans 
contained an @Local annotation, we've decided to remove said annotations and 
their imports from the ComponentLifecycleBase class and its subclasses seeking 
a reduction of a few hundred lines of useless code.

I had already opened a pull request for this (#853) but at some point my 
commit disappeared from the PR entirely, showing no new changes in code, which 
caused it to be merged automatically (with no changes).

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/rafaelweingartner/cloudstack 
master-lrg-cs-hackday-006

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1102.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1102


commit 1a64c247adbeb326f7ddbe6e4feaf3159e9e740d
Author: cirstofolini 
Date:   2015-11-21T20:31:11Z

Removed unnecessary @Local annotations and their respective imports from 
the ComponentLifecycleBase class and its subclasses.




---
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: Removal of DefaultUserAuthenticator empty...

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1100#issuecomment-158688620
  
Hi @DaanHoogland, 
That class was created 5 years ago by “Manuel Amador”, at that time the 
class had some logic. Then, someone called “Darren Shepherd” while doing 
some clean up left the class empty.



---
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: Create test cases to getPatchFilePath met...

[DaanHoogland]

Github user DaanHoogland commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/944#discussion_r45550989
  
--- Diff: 
plugins/hypervisors/xenserver/test/com/cloud/hypervisor/xenserver/resource/CitrixResourceBaseTest.java
 ---
@@ -0,0 +1,37 @@
+package com.cloud.hypervisor.xenserver.resource;
+
+import java.io.File;
+import java.util.List;
+
+import org.junit.Assert;
+import org.mockito.Mockito;
+import org.powermock.api.mockito.PowerMockito;
+
+import com.cloud.utils.script.Script;
+
+public abstract class CitrixResourceBaseTest {
+
+public void testGetPathFilesExeption(CitrixResourceBase 
citrixResourceBase){
+String patch = citrixResourceBase.getPatchFilePath();
+
+PowerMockito.mockStatic(Script.class);
+Mockito.when(Script.findScript("", patch)).thenReturn(null);   
+
+citrixResourceBase.getPatchFiles();
+
--- End diff --

no checks?


---
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: Create test cases to getPatchFilePath met...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/944#issuecomment-158683799
  
@pedro-martins like the code one question about CitrixResourceBaseTest. And 
also: can you rebase on a commit beyond 4.6 and not merge, please?


---
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: Removal of DefaultUserAuthenticator empty...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1100#issuecomment-158684056
  
@alexandrelimassantana Did you check who created the 
DefaultUserAuthenticator? Your change makes sense to me but I would like to 
know why it was created before deleting it.

code lgtm, btw.


---
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: Add47xmessagesproperties transifex

[milamberspace]

Github user milamberspace commented on the pull request:

https://github.com/apache/cloudstack/pull/1098#issuecomment-158686367
  
@DaanHoogland I don' known the best way to explain the transifex process.

For all versions of CloudStack, we have a version of resources files on 
Transifex [1].

When you update the main resource file with the new version, the transifex 
config file is updated with all languages with already translated.

I don't know the best way to explain the internal behavior of the transifex 
client, but the config file is update after the first download of the main 
resource file (en_US) and the upload of the current L10N files to the new L10N 
files for the new versions. This PR reflect the changes.

You can retry the behavior from the master (up to date 2015/11/21) with 
theses commands:

To fetch the latest original L10N file form Transifex (already done because 
I'm put the 4.7 main resource files on the TX website today)
cd tools/transifex/
 ./sync-transifex-ui.sh download-source-language 
CloudStack_UI.47xmessagesproperties

To upload all L10N translations files (from 4.6 files) to transifex 
(already done by me today, but you can re-made the work during few days before 
the translator team changes the strings on Transifex)
./sync-transifex-ui.sh upload-l10n-languages 
CloudStack_UI.47xmessagesproperties


[1]https://www.transifex.com/ke4qqq/CloudStack_UI/content/



---
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: Removal of DefaultUserAuthenticator empty...

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1100#issuecomment-158688747
  
Sure ;)
He is one of my team mates of that project we talked about.
Today was one of the days that we meet and try to clean some code. We 
normally do that once a month.


---
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: Removal of DefaultUserAuthenticator empty...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1100#issuecomment-158688687
  
ah @rafaelweingartner you did the check for @alexandrelimassantana , 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: [4.7] CLOUDSTACK-7984: Collect network st...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/926#issuecomment-158676063
  
@wido I see some comments above. Not sure if they are all addressed. Can 
you please review this and let us know if you are OK with merging? I'll run 
soms tests again since it's been a while.


---
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: [FUTURE]CLOUDSTACK-8907: POD Gateway opti...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/886#issuecomment-158684576
  
code looks good but I don't see the use case; in an isolated network 
without internet access there would still be a gateway, would there?


---
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-8751 minimise downtime of netw...

[DaanHoogland]

Github user DaanHoogland commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/866#discussion_r45551366
  
--- Diff: 
server/test/com/cloud/network/element/VirtualRouterElementTest.java ---
@@ -225,6 +235,62 @@ public void testPrepare() {
 
 }
 
+@Test
+public void testGetRouters1(){
--- End diff --

testGetRoutersUnderThisOrTheseCircumstances() ould be a more appropriate 
name.
testGetRouters1, testGetRouters2 and testGetRouters3 is not really clear.


---
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-9068: Listing Port Forwarding ...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1081#issuecomment-158685974
  
less is more, if it works it's fine. How does it behave on resizing?


---
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: [DISCUSS] ACS 4.5.3 release

[Milamber]

Hello Rohit,

I think that the PR 626 [1] on the debian/rules must be apply on the 4.5 
branch to release 4.5.3 version to allow the good building of CS 4.53 on 
Ubuntu. (snapshot suffix issue on cloud-agent package)



$ git diff
diff --git a/debian/rules b/debian/rules
index 45461bb..78dd487 100755
--- a/debian/rules
+++ b/debian/rules
@@ -11,7 +11,7 @@
 # build-arch and build-indep targets  by Bill Allombert 2001

 DEBVERS := $(shell dpkg-parsechangelog | sed -n -e 's/^Version: //p')
-VERSION := $(shell echo '$(DEBVERS)' | sed -e 's/^[[:digit:]]*://' -e 
's/[~-].*//')

+VERSION := $(shell cat pom.xml |sed '22!d'| cut -d'>' -f2 |cut -d'<' -f1)
 MVNADD := $(shell if echo '$(DEBVERS)' | grep -q snapshot; then echo ; 
fi )

 PACKAGE = $(shell dh_listpackages|head -n 1|cut -d '-' -f 1)
 SYSCONFDIR = "/etc"

Milamber

[1] https://github.com/apache/cloudstack/pull/626/files



On 20/11/2015 15:16, Stephan Seitz wrote:

Hi Rohit,

I've reported one Bug[1], that affects the network.throttling.rate. I
assume, this is just some bad calculation.

Maybe this could be fixed for 4.5.3? :)

[1] https://issues.apache.org/jira/browse/CLOUDSTACK-8936


Am Freitag, den 20.11.2015, 06:27 + schrieb Rohit Yadav:

Hi all,


I want to ask how happy people are with the last 4.5.2 release and if
there are any issues they want to report or want to be fixed in a
future minor release. If we’ve enough demand, we can work towards a
last 4.5 minor release. Thanks.


Rohit Yadav
Software Architect





D: +44 20 3642 6102 | S: +44 20 3603 0540 | M: +91 88 262 30892

  
rohit.ya...@shapeblue.com | www.shapeblue.com | Twitter:@ShapeBlue
  
ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS



Find out more about ShapeBlue and our range of CloudStack related
services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Software Engineering
CloudStack Infrastructure Support
CloudStack Bootcamp Training Courses


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. 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.

[GitHub] cloudstack pull request: Add47xmessagesproperties transifex

[milamberspace]

Github user milamberspace commented on the pull request:

https://github.com/apache/cloudstack/pull/1098#issuecomment-158688518
  
@DaanHoogland I've just update this wiki page:

https://cwiki.apache.org/confluence/display/CLOUDSTACK/Update+L10N+files+from+Transifex+to+git+repo

I hope that this page explains the process, otherwise tell me the issue 
with the transifex client.


---
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: L10N : Updated locale files with latest t...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1097#issuecomment-158613518
  
Pinging @milamberspace to have a look.


---
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: 4.6 release

[Sebastien Goasguen]

Talking about 4.6

There seems to be an issue when people upgrade from 4.5.2 and the awsapi 
package is missing.

Can someone (ping @pdion) upgrade the release notes to mention the fact that 
awsapi is removed and add a step in the upgrade process ?

> On Nov 20, 2015, at 2:39 PM, Wido den Hollander  wrote:
> 
> 
> 
> On 11/20/2015 02:04 PM, Paul Angus wrote:
>> http://docs.cloudstack.apache.org/projects/cloudstack-release-notes/en/4.6.0/upgrade/upgrade-4.5.html#hypervisor-vmware
>> states:
>> 'Warning
>> For VMware hypervisor CloudStack management server packages must be build 
>> using “noredist”. Refer to Building from Source.'
>> I can't find the same warning in the 4.6 installation guide(s) anywhere.
>> 
>> Which version (noredist or oss) have the packages on cloudstack.apt-get.eu 
>> been built with?
>> 
>> And actually is there any difference between /rhel and /centos ?
>> 
> 
> No, there is not. It's just a symlink. /rhel was the old path and
> /centos is the new one.
> 
>> 
>> Regards,
>> Paul Angus
>> VP Technology/Cloud Architect
>> ShapeBlue Ltd
>> S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus
>> paul.an...@shapeblue.com | www.shapeblue.com | Twitter:@shapeblue
>> ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS
>> 
>> 
>> 
>> 
>> -Original Message-
>> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
>> Sent: Friday, November 20, 2015 11:56 AM
>> To: dev 
>> Subject: Re: 4.6 release
>> 
>> Paul, this is not helpful. please supply changes for what you would like to 
>> see instead so we can discuss.
>> 
>> On Fri, Nov 20, 2015 at 10:10 AM, Paul Angus 
>> wrote:
>> 
>>> Guys,
>>> 
>>> 
>>> 
>>> I’m out and about this morning, but I’ve noticed that the release
>>> notes state that baseurl for 4.6 is
>>> http://cloudstack.apt-get.eu/rhel/4.5/
>>> 
>>> And that somewhat buried in the vmware install information is the fact
>>> that the apt-get repo only includes the oss build.
>>> 
>>> 1.   I fundamentally disagree with this practice.
>>> 
>>> 2.   If we’re going to do this then it needs to needs to be put front
>>> and center on the download links page with an explanation of the
>>> differences to save confusion. And a link to where rpms which DO
>>> contain the noredist can be found.
>>> 
>>> It looks like we’re actively trying to make it difficult for people to
>>> start using CloudStack.
>>> 
>>> 
>>> 
>>> 
>>> 
>>> Regards,
>>> 
>>> *Paul Angus*
>>> 
>>> 
>>> *VP Technology/Cloud Architect *
>>> 
>>> *[image: ShapeBlue_col_logo_sml]*
>>> 
>>> S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus
>>> paul.an...@shapeblue.com | www.shapeblue.com | Twitter:@shapeblue
>>> ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS
>>> 
>>> 
>>> 
>>> 
>>> Find out more about ShapeBlue and our range of CloudStack related
>>> services
>>> 
>>> IaaS Cloud Design & Build
>>> 
>>> CSForge – rapid IaaS deployment framework
>>>  CloudStack Consulting
>>> 
>>> CloudStack Software Engineering
>>> 
>>> CloudStack Infrastructure Support
>>> 
>>> CloudStack Bootcamp Training Courses
>>> 
>>> 
>>> 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. 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.
>>> 
>> 
>> 
>> 
>> --
>> Daan
>> Find out more about ShapeBlue and our range of CloudStack related services
>> 
>> IaaS Cloud Design & Build
>> CSForge – rapid IaaS deployment framework
>> CloudStack Consulting
>> CloudStack Software 
>> Engineering
>> CloudStack Infrastructure 
>> 

Re: Read/Write splitting on database

[Remi Bergsma]

No, I don't think that can be done or works well. The best approach is to use 
MariaDB with Galera. There was a talk in Dublin about this, you might want to 
check the video. 

Regards, Remi 

Sent from my iPhone

> On 18 Nov 2015, at 10:01, Alireza Eskandari  
> wrote:
> 
> Hi all,
> According to link below, does CloudStack split read/write request between 
> master and slaves?
> https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=34838207
> 
> regards,
> Alireza 

[GitHub] cloudstack pull request: CLOUDSTACK-9076: Changed ownership of dir...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1095#issuecomment-158613584
  
@borisroman Did you build the packages from this branch? Can you share some 
details to show it works properly now?


---
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: SR_BACKEND_FAILURE_61File SR creation error [opterr=fail to mount FileSR. Errno is 32]

[Devdeep]

Have you seeded the secondary storage with the default systemvm template?

Regards,
Devdeep

On Sat, Nov 21, 2015 at 8:33 AM, B Prakash  wrote:

> Thanks Jayapal, Abhinandan.
>
> log SMlog doesnt giveout much, I am able to mount and edit file on mount
> from both server and agent. I get the same error as follows (timestamp is
> different because I tried it again), is there anything else I am missing?:
>
> Nov 20 16:22:01 agent SM: [5432] ['mount', '--bind',
> '/var/cloud_mount/b61521bd-ad7b-3e95-91e8-00762b28aeeb/template/tmpl/1/1',
> '/var/run/sr-mount/e18ee8da-a45d-46b8-b361-ec50e9ddcb22']
> Nov 20 16:22:01 agent SM: [5432] FAILED in util.pread: (rc 32) stdout: '',
> stderr: 'mount: special device
> /var/cloud_mount/b61521bd-ad7b-3e95-91e8-00762b28aeeb/template/tmpl/1/1
> does not exist
> Nov 20 16:22:01 agent SM: [5432] '
> Nov 20 16:22:02 agent SM: [5432] Raising exception [61, File SR creation
> error [opterr=fail to mount FileSR. Errno is 32]]
> Nov 20 16:22:02 agent SM: [5432] lock: released
> /var/lock/sm/e18ee8da-a45d-46b8-b361-ec50e9ddcb22/sr
> Nov 20 16:22:02 agent SM: [5432] * generic exception: sr_attach:
> EXCEPTION SR.SROSError, File SR creation error [opterr=fail to mount
> FileSR. Errno is 32]
> Nov 20 16:22:02 agent SM: [5432]   File "/opt/xensource/sm/SRCommand.py",
> line 110, in run
> Nov 20 16:22:02 agent SM: [5432] return self._run_locked(sr)
> Nov 20 16:22:02 agent SM: [5432]   File "/opt/xensource/sm/SRCommand.py",
> line 159, in _run_locked
> Nov 20 16:22:02 agent SM: [5432] rv = self._run(sr, target)
> Nov 20 16:22:02 agent SM: [5432]   File "/opt/xensource/sm/SRCommand.py",
> line 322, in _run
> Nov 20 16:22:02 agent SM: [5432] return
> sr.attach(self.params['sr_uuid'])
> Nov 20 16:22:02 agent SM: [5432]   File "/opt/xensource/sm/FileSR", line
> 164, in attach
> Nov 20 16:22:02 agent SM: [5432] opterr='fail to mount FileSR. Errno
> is %s' % inst.code)
> Nov 20 16:22:02 agent SM: [5432]   File "/opt/xensource/sm/xs_errors.py",
> line 52, in __init__
> Nov 20 16:22:02 agent SM: [5432] raise SR.SROSError(errorcode,
> errormessage)
> Nov 20 16:22:02 agent SM: [5432]
> Nov 20 16:22:02 agent SM: [5432] lock: closed
> /var/lock/sm/e18ee8da-a45d-46b8-b361-ec50e9ddcb22/sr
> Nov 20 16:22:30 agent SM: [5587] lock: opening lock file
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/sr
> Nov 20 16:22:30 agent SM: [5587] lock: acquired
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/sr
> Nov 20 16:22:30 agent SM: [5587] sr_attach {'sr_uuid':
> '485be965-9bcd-4e76-8fa8-5881f0203645', 'subtask_of':
> 'DummyRef:|fd400425-8c78-85c2-949c-230c0c39e03f|SR.attach', 'args': [],
> 'host_ref': 'OpaqueRef:274f3e03-3513-61a8-40de-5b1ece53c1d7',
> 'session_ref': 'OpaqueRef:5706f0b6-0fcb-7c5c-defa-36e537655059',
> 'device_config': {'SRmaster': 'true', 'location':
> '/var/cloud_mount/b61521bd-ad7b-3e95-91e8-00762b28aeeb/template/tmpl/1/1'},
> 'command': 'sr_attach', 'sr_ref':
> 'OpaqueRef:5906daec-9b86-5fa3-58bc-e95969b77a4d'}
> Nov 20 16:22:30 agent SM: [5587] lock: opening lock file
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/running
> Nov 20 16:22:30 agent SM: [5587] lock: opening lock file
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/sr
> Nov 20 16:22:30 agent SM: [5587] lock: acquired
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/running
> Nov 20 16:22:30 agent SM: [5587] lock: acquired
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/sr
> Nov 20 16:22:30 agent SM: [5587] RESET for SR
> 485be965-9bcd-4e76-8fa8-5881f0203645 (master: True)
> Nov 20 16:22:30 agent SM: [5587] lock: released
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/sr
> Nov 20 16:22:30 agent SM: [5587] lock: released
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/running
> Nov 20 16:22:30 agent SM: [5587] lock: closed
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/sr
> Nov 20 16:22:30 agent SM: [5587] lock: closed
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/running
> Nov 20 16:22:30 agent SM: [5587] set_dirty
> 'OpaqueRef:5906daec-9b86-5fa3-58bc-e95969b77a4d' succeeded
> Nov 20 16:22:30 agent SM: [5587] ['mount', '--bind',
> '/var/cloud_mount/b61521bd-ad7b-3e95-91e8-00762b28aeeb/template/tmpl/1/1',
> '/var/run/sr-mount/485be965-9bcd-4e76-8fa8-5881f0203645']
> Nov 20 16:22:30 agent SM: [5587] FAILED in util.pread: (rc 32) stdout: '',
> stderr: 'mount: special device
> /var/cloud_mount/b61521bd-ad7b-3e95-91e8-00762b28aeeb/template/tmpl/1/1
> does not exist
> Nov 20 16:22:30 agent SM: [5587] '
> Nov 20 16:22:30 agent SM: [5587] Raising exception [61, File SR creation
> error [opterr=fail to mount FileSR. Errno is 32]]
> Nov 20 16:22:30 agent SM: [5587] lock: released
> /var/lock/sm/485be965-9bcd-4e76-8fa8-5881f0203645/sr
> Nov 20 16:22:30 agent SM: [5587] * generic exception: sr_attach:
> EXCEPTION SR.SROSError, File SR creation error [opterr=fail to mount
> FileSR. Errno is 32]
> 
> From: 

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1021#issuecomment-158617888
  
@bhaisaab @jlk want to review give a second lgtm?


---
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: Cwe 190

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1057#issuecomment-158624430
  
I was busy starting ;)


---
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: Cwe 190

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1057#issuecomment-158624231
  
@DaanHoogland It could be a result of another failure. On master this test 
passes, so maybe you can run just this test yourself and look at the details?


---
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: Cwe 190

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1057#issuecomment-158613419
  
@DaanHoogland @rafaelweingartner Much better now, just one failed test to 
resolve:

Failed test: `TestVPCRouterOneNetwork`

```
Execute cmd: createstaticroute failed, due to: errorCode: 530, 
errorText:cidr is not formatted correctly: cloud
```

Details:
```
requests.packages.urllib3.connectionpool: DEBUG: "GET 
/client/api?apiKey=Mn6U73dZiNUEuY1GR7hwRNlwoPAiIeEbPnNZojmWoLNr31Y3b9SpBvuOP_xH_Pn7pm-BS1DuOrV8BffcLP35Uw=createStaticRoute=QKOVZRSgas3u2sOUus5t8JncRbs%3D=0177e150-3dc2-4454-85be-f43b9b093b65=11.1.1.1%2F24=json
 HTTP/1.1" 530 133
test_09_create_redundant_vpc_offering 
(integration.component.test_vpc_offerings.TestVPCOffering): ERROR: 
Exception:['Traceback (most recent call last):\n', '  File 
"/usr/lib/python2.7/site-packages/marvin/cloudstackConnection.py", line 308, in 
__parseAndGetResponse\nresponse_cls)\n', '  File 
"/usr/lib/python2.7/site-packages/marvin/jsonHelper.py", line 150, in 
getResultObj\nraise cloudstackException.CloudstackAPIException(respname, 
errMsg)\n', 'CloudstackAPIException: Execute cmd: createstaticroute failed, due 
to: errorCode: 530, errorText:cidr is not formatted correctly: cloud\n']
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/marvin/cloudstackConnection.py", 
line 308, in __parseAndGetResponse
response_cls)
  File "/usr/lib/python2.7/site-packages/marvin/jsonHelper.py", line 150, 
in getResultObj
raise cloudstackException.CloudstackAPIException(respname, errMsg)
CloudstackAPIException: Execute cmd: createstaticroute failed, due to: 
errorCode: 530, errorText:cidr is not formatted correctly: cloud
test_09_create_redundant_vpc_offering 
(integration.component.test_vpc_offerings.TestVPCOffering): ERROR: 
marvinRequest : CmdName: 
 Exception: ['Traceback (most recent call last):\n', '  File 
"/usr/lib/python2.7/site-packages/marvin/cloudstackConnection.py", line 374, in 
marvinRequest\nraise self.__lastError\n', 'CloudstackAPIException: Execute 
cmd: createstaticroute failed, due to: errorCode: 530, errorText:cidr is not 
formatted correctly: cloud\n']
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/marvin/cloudstackConnection.py", 
line 374, in marvinRequest
raise self.__lastError
CloudstackAPIException: Execute cmd: createstaticroute failed, due to: 
errorCode: 530, errorText:cidr is not formatted correctly: cloud
test_09_create_redundant_vpc_offering 
(integration.component.test_vpc_offerings.TestVPCOffering): CRITICAL: 
EXCEPTION: test_09_create_redundant_vpc_offering: ['Traceback (most recent call 
last):\n', '  File "/usr/lib/python2.7/site-packages/nose/suite.py", line 209, 
in run\nself.setUp()\n', '  File 
"/usr/lib/python2.7/site-packages/nose/suite.py", line 292, in setUp\n
self.setupContext(ancestor)\n', '  File 
"/usr/lib/python2.7/site-packages/nose/suite.py", line 315, in setupContext\n   
 try_run(context, names)\n', '  File 
"/usr/lib/python2.7/site-packages/nose/util.py", line 471, in try_run\n
return func()\n', '  File 
"/data/jenkins-slave/workspace/acs-ci-build/901-run-marvin-tests/test/integration/component/test_vpc_routers.py",
 line 701, in setUpClass\ngatewayid=private_gateway.id\n', '  File 
"/usr/lib/python2.7/site-packages/marvin/lib/base.py", line 4262, in create\n   
 return StaticRoute(apiclient.createStaticRoute(cmd).__dict__)\n', '  File 
"/usr/lib/python2.7
 /site-packages/marvin/cloudstackAPI/cloudstackAPIClient.py", line 2277, in 
createStaticRoute\nresponse = self.connection.marvinRequest(command, 
response_type=response, method=method)\n', '  File 
"/usr/lib/python2.7/site-packages/marvin/cloudstackConnection.py", line 379, in 
marvinRequest\nraise e\n', 'CloudstackAPIException: Execute cmd: 
createstaticroute failed, due to: errorCode: 530, errorText:cidr is not 
formatted correctly: cloud\n']
```


---
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: Cwe 190

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1057#issuecomment-158625241
  
:-)


---
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: [4.7] CLOUDSTACK-8746: VM Snapshotting im...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/977#issuecomment-158615300
  
Hi @wido can you review this again please? Thanks!


---
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-8485: listAPIs are taking too ...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1021#issuecomment-158617403
  

[1021.network.results.txt](https://github.com/apache/cloudstack/files/40781/1021.network.results.txt)

[1021.vpc.results.txt](https://github.com/apache/cloudstack/files/40782/1021.vpc.results.txt)
all tests passed


---
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: Cwe 190

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1057#issuecomment-158620306
  
thanks @remibergsma ```cidr is not formatted correctly: cloud``` sound like 
a valid error message. not sure if it needs fixing in the test or in the code.


---
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: Add47xmessagesproperties transifex

[milamberspace]

GitHub user milamberspace opened a pull request:

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

Add47xmessagesproperties transifex

Now master branch is the next 4.7 release. Open the 4.7 resources files on 
Transifex.

Update the transifex client config file.

Small PR outside the build path.

cc @remibergsma @DaanHoogland 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/milamberspace/cloudstack 
Add47xmessagespropertiesTransifex

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1098.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1098


commit 6b3daf4eb744221eaf645bd303a9cad5166caf23
Author: Milamber 
Date:   2015-11-21T12:03:01Z

Update transifex client config file with the 4.7 L10N config

commit 9219cbcd385c72ca2172f7ce03cd89d2103f33f6
Author: Milamber 
Date:   2015-11-21T12:05:41Z

Add languages for 4.7




---
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.
---