[GitHub] cloudstack issue #2037: CLOUDSTACK-9871: Set SQL Mode in SQL Session for MyS...

2017-04-12 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/2037
  
Do you know if this have an impact if we still use MySQL 5.6 or MySQL 5.5 
on another host and have CloudStack installed on Ubuntu or CentOS 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.
---


[GitHub] cloudstack issue #1834: CLOUDSTACK-9679:Allow master user to manage subordin...

2017-04-04 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1834
  
@pdumbre @karuturi @syed 
I thinks this introduce an issue where a domain admin cannot get a template 
by id. 
a domain admin can list feature template and get a template via `ids=` but 
not with `id=`
The example bellow is perform with a domain admin account to list 
public+featured template at root domain:

```
(beta2r1-ninja) > list templates templatefilter=featured filter=name,id
count = 9
template:
+--+-+
|  id  |   name  |
+--+-+
| 513b3a6d-c011-46f0-a4a3-2a954cadb673 |  CoreOS Alpha 1367.5.0  |
| 0c04d876-1f85-45a7-b6f4-504de435bf12 |Debian 8.5 PV base (64bit)   |
| 285f2203-449a-428f-997a-1ffbebbf1382 |   CoreOS Alpha  |
| 332b6ca8-b3d6-42c7-83e5-60fe87be6576 |  CoreOS Stable  |
| 3b705008-c186-464d-ad59-312d902420af |   Windows Server 2016 std SPLA  |
| 4256aebe-a1c1-4b49-9993-de2bc712d521 |   Ubuntu 16.04.01 HVM   |
| 59e6b00a-b88e-4539-aa3c-75c9c7e9fa6c | Ubuntu 14.04.5 HVM base (64bit) |
| 3ab936eb-d8c2-44d8-a64b-17ad5adf8a51 |  CentOS 6.8 PV  |
| 7de5d423-c91e-49cc-86e8-9d6ed6abd997 |  CentOS 7.2 HVM |
+--+-+
(beta2r1-ninja) > list templates templatefilter=featured 
id=7de5d423-c91e-49cc-86e8-9d6ed6abd997 filter=name,id
Error 531: Acct[b285d62e-0ec2-4a7c-b773-961595ec6356-Ninja-5664] does not 
have permission to operate within domain id=c9b4f83d-16eb-11e7-a8b9-367e6fe958a9
cserrorcode = 4365
errorcode = 531
errortext = Acct[b285d62e-0ec2-4a7c-b773-961595ec6356-Ninja-5664] does not 
have permission to operate within domain id=c9b4f83d-16eb-11e7-a8b9-367e6fe958a9
uuidList:
(beta2r1-ninja) > list templates templatefilter=featured 
ids=7de5d423-c91e-49cc-86e8-9d6ed6abd997 filter=name,id
count = 1
template:
+--++
|  id  |  name  |
+--++
| 7de5d423-c91e-49cc-86e8-9d6ed6abd997 | CentOS 7.2 HVM |
+--++
```


---
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 issue #1582: CLOUDSTACK-9408 for the move away from download.clou...

2017-03-14 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1582
  
Effectively we are due for an updated systemvm for strongswan + jdk8


---
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 #:

2017-02-27 Thread pdion891
Github user pdion891 commented on the pull request:


https://github.com/apache/cloudstack/commit/fa85151be962824cc88776a0264e1ac6ef90560c#commitcomment-21080945
  
In server/src/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java:
In server/src/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java on line 295:
@karuturi this seams to break master. the line 294 `{` does not seams to be 
close.




---
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 issue #1919: CLOUDSTACK-9763: set ssh public key when creating vm...

2017-02-27 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1919
  
This is usefull when using VPC.

+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-www pull request #34: Remove link to “planet.apache.org” and ...

2017-02-13 Thread pdion891
Github user pdion891 commented on a diff in the pull request:

https://github.com/apache/cloudstack-www/pull/34#discussion_r100800298
  
--- Diff: content/about.html ---
@@ -79,7 +78,7 @@
 Get 
Involved
 Developers
 Mailing 
Lists
-http://lanyrd.com/topics/apache-cloudstack/; target="_blank">Events  
Meetups 
--- End diff --

I'm not sure we want to ditch lanyrd link, it's usefull to publish other 
event that collabs, like meetups.


---
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-www issue #34: Remove link to “planet.apache.org” and update ...

2017-02-13 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-www/pull/34
  
Let's change the model but not in this PR ;)


---
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-www issue #34: Remove link to “planet.apache.org” and update ...

2017-02-13 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-www/pull/34
  
sorry,  so yeah,   no html in the PR, only in asf-site when merge is 
aprouve, than html generated by the commiter merging the PR.  make sense?   
since `content` folder already exist, no need to change this PR for this ;-)



---
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-www issue #34: Remove link to “planet.apache.org” and update ...

2017-02-13 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-www/pull/34
  
I haven't follow this whole thread but shouldn't we not include .html files 
in PR and master branch and only push them in the asf-site branch once approved 
?  this would make master and PR cleaners.


---
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-docs-admin issue #41: 50% of the doc strings are translated into ...

2017-02-04 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/41
  
@lmrv  I've extract the PR diff file from github, but there is no 
information about you in the git commis, you name and email address. 

I think you just need to do follow:

```
git config --global user.name "your_name"
git config --global user.email "your_em...@example.com"
```

Then you will need to amend all commits of the PR so it will be commit 
under your name, refer to : 
http://stackoverflow.com/questions/4981126/how-to-amend-several-commits-in-git-to-change-author
 seams good information.
ex: `git rebase -i YOUR_SHA -x "git commit --amend --reset-author -CHEAD"`

I can merge the PR as is but your name will not figure it the `git log`



---
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 #:

2017-02-03 Thread pdion891
Github user pdion891 commented on the pull request:


https://github.com/apache/cloudstack/commit/eabf862ba9751bcdbe18a6cc9b91f2ae48472cab#commitcomment-20741478
  
In setup/db/db/schema-4910to4920.sql:
In setup/db/db/schema-4910to4920.sql on line 217:
could it be possible there is a missing `;` at the last line ?


---
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 issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-03 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1928
  
Please merge it into master as well, it also mean we need to make sure 
master does have the same file `setup/db/db/schema-4920to4930.sql` too.

We will do some testing around xenserver7 soon also  :-)


---
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 issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-02 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1928
  
I think this should be push into master as well to make sure it will exist 
in future version.


---
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-www issue #34: Remove link to “planet.apache.org” and update ...

2017-02-02 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-www/pull/34
  
we have builds.cloudstack.org, but we would need to validate how to push 
files on webserver which I think the current case if a pull process.


---
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-www issue #34: Remove link to “planet.apache.org” and update ...

2017-02-02 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-www/pull/34
  
The website is not build via jenkins yet. manual for now :-S


---
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-www issue #34: Remove link to “planet.apache.org” and update ...

2017-02-02 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-www/pull/34
  
Hi, I think we miss-managed our repo. The original idea was to have only 
source files in the master branch for the website, and when the PR is aprouve 
we build it in the asf-site branch and then include the html version of it, 
this is the branch that get pushed into webservers.

I might worth update the Readme about this :-S


---
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 #1921: Dockerfile: Upgrade base distro to Ubuntu 16....

2017-01-30 Thread pdion891
GitHub user pdion891 opened a pull request:

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

Dockerfile: Upgrade base distro to Ubuntu 16.04, fix support for JDK8

Fix current broken build introduce with JDK upgrade from #1888 of docker 
simulator because of missing dependencies and JDK8. This PR upgrade the distro 
to Ubuntu 16.04 and use default jdk version 8.

I've quickly test the build locally, no error in the UI and logs.

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

$ git pull https://github.com/pdion891/cloudstack dockerfile_ubuntu1604_jdk8

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

https://github.com/apache/cloudstack/pull/1921.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 #1921


commit 450d8d00fb5966c5cda8f2ed878f1b3add1adaf1
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2017-01-31T01:35:10Z

Upgrade base distro to Ubuntu 16.04, fix support for JDK8, fix 
dependencies, should fix current broken build of docker simulator




---
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-docs-admin issue #41: 50% of the doc strings are translated into ...

2017-01-30 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/41
  
LGTM,  I'll merge this PR this week... 

Thanks a lot @lmrv for the translation ! 


---
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-docs-admin pull request #41: 50% of the doc strings are translate...

2017-01-30 Thread pdion891
Github user pdion891 commented on a diff in the pull request:

https://github.com/apache/cloudstack-docs-admin/pull/41#discussion_r98378667
  
--- Diff: source/locale/fr/LC_MESSAGES/virtual_machines.po ---
@@ -0,0 +1,2188 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+#
+# Translators:
+msgid ""
+msgstr ""
+"Project-Id-Version: Apache CloudStack Administration RTD\n"
+"Report-Msgid-Bugs-To: \n"
+"POT-Creation-Date: 2016-08-22 13:55+0200\n"
+"PO-Revision-Date: \n"
+"Last-Translator: \n"
+"Language-Team: \n"
+"MIME-Version: 1.0\n"
+"Content-Type: text/plain; charset=iso-8859-1\n"
+"Content-Transfer-Encoding: 8bit\n"
+"Language: fr\n"
+"X-Generator: Poedit 1.8.11\n"
+
+#: ../../virtual_machines.rst:18
+msgid "Working with Virtual Machines"
+msgstr "Travailler avec les Machines Virtuelles"
+
+#: ../../virtual_machines.rst:21
+msgid "About Working with Virtual Machines"
+msgstr "A propos du travail avec les Machines Virtuelles"
+
+#: ../../virtual_machines.rst:23
+msgid ""
+"CloudStack provides administrators with complete control over the 
lifecycle of all "
+"guest VMs executing in the cloud. CloudStack provides several guest 
management "
+"operations for end users and administrators. VMs may be stopped, started, 
rebooted, "
+"and destroyed."
+msgstr ""
+"CloudStack fournit aux administrateurs le contr�le complet du cycle de 
vie de toutes "
+"les VM invit�es s'ex�cutant dans le cloud. CloudStack fournit 
plusieurs op�rations "
+"de gestion des invit�s aux utilisateurs finaux et aux administrateurs. 
Les VM "
+"peuvent �tre arr�t�es, d�marr�es, red�marr�es et 
d�truites."
+
+#: ../../virtual_machines.rst:28
+msgid ""
+"Guest VMs have a name and group. VM names and groups are opaque to 
CloudStack and "
+"are available for end users to organize their VMs. Each VM can have three 
names for "
+"use in different contexts. Only two of these names can be controlled by 
the user:"
+msgstr ""
+"Les VM invit�es ont un nom et un groupe. Les noms et groupes de VM sont 
opaques � "
+"CloudStack et sont disponibles pour que les utilisateurs finaux 
organisent leurs VM. "
+"Chaque VM peut avoir trois noms � utiliser dans diff�rents contextes. 
Seuls deux de "
+"ces noms peuvent �tre contr�l�s par l'utilisateur :"
+
+#: ../../virtual_machines.rst:33
+msgid ""
+"Instance name – a unique, immutable ID that is generated by CloudStack 
and can not "
+"be modified by the user. This name conforms to the requirements in IETF 
RFC 1123."
+msgstr ""
+"Nom de l'instance - un ID unique et non modifiable qui est g�n�r� 
par CloudStack et "
+"ne peut pas �tre modifi� par l'utilisateur. Ce nom est conforme aux 
exigences de la "
+"RFC 1123 de l'IETF."
+
+#: ../../virtual_machines.rst:37
+msgid ""
+"Display name – the name displayed in the CloudStack web UI. Can be set 
by the user. "
+"Defaults to instance name."
+msgstr ""
+"Nom affiché - le nom affiché dans l'interface web CloudStack. Peut 
être fixé par "
+"l'utilisateur. Prend par défaut le nom de l'instance."
+
+#: ../../virtual_machines.rst:40
+msgid ""
+"Name – host name that the DHCP server assigns to the VM. Can be set by 
the user. "
+"Defaults to instance name"
+msgstr ""
+"Nom - nom de machine que le serveur DHCP assigne à la VM. Peut être 
fixé par "
--- End diff --

@lmrv  is it normal to see weird character such as these?


---
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 #:

2017-01-29 Thread pdion891
Github user pdion891 commented on the pull request:


https://github.com/apache/cloudstack/commit/e6cc78f531d0201aed004fd9e003a1ae2ab9a816#commitcomment-20657409
  
In tools/docker/Dockerfile:
In tools/docker/Dockerfile on line 25:
unfortunately it doesn't work because add-apt-repository is not present. I 
did a quick tests and it seams to work better on Ubuntu 16.04. so I'll submit a 
PR changing the base to Ubuntu 16.04.


---
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-docs-admin issue #41: 50% of the doc strings are translated into ...

2017-01-29 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/41
  
Hi @lmrv,  since the last commit,  if I try to build the project in `fr` 
there is an issue with `virtual_machines.mo`:  

I've run `make gettext` which worked fined but not sphinx-intl:
```
$ sphinx-intl build --locale-dir source/locale
Project name loaded from .tx/config: apache-cloudstack-administration-rtd
Build: source/locale/fr/LC_MESSAGES/networking_and_traffic.mo
Build: source/locale/fr/LC_MESSAGES/projects.mo
Build: source/locale/fr/LC_MESSAGES/networking2.mo
Build: source/locale/fr/LC_MESSAGES/troubleshooting.mo
Build: source/locale/fr/LC_MESSAGES/systemvm.mo
Build: source/locale/fr/LC_MESSAGES/hosts.mo
Build: source/locale/fr/LC_MESSAGES/ui.mo
Build: source/locale/fr/LC_MESSAGES/administration.mo
Build: source/locale/fr/LC_MESSAGES/virtual_machines.mo
Traceback (most recent call last):
  File "/usr/local/bin/sphinx-intl", line 11, in 
sys.exit(main())
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 722, in 
__call__
return self.main(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 697, in 
main
rv = self.invoke(ctx)
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 1066, 
in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 895, in 
invoke
return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 535, in 
invoke
return callback(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/sphinx_intl/commands.py", 
line 247, in build
basic.build(locale_dir, output_dir, languages)
  File "/usr/local/lib/python2.7/dist-packages/sphinx_intl/basic.py", line 
101, in build
cat = c.load_po(po_file)
  File "/usr/local/lib/python2.7/dist-packages/sphinx_intl/catalog.py", 
line 17, in load_po
cat = pofile.read_po(f)
  File "/usr/lib/python2.7/dist-packages/babel/messages/pofile.py", line 
212, in read_po
_add_message()
  File "/usr/lib/python2.7/dist-packages/babel/messages/pofile.py", line 
165, in _add_message
catalog[msgid] = message
  File "/usr/lib/python2.7/dist-packages/babel/messages/catalog.py", line 
619, in __setitem__
self.mime_headers = _parse_header(message.string).items()
  File "/usr/lib/python2.7/dist-packages/babel/messages/catalog.py", line 
415, in _set_mime_headers
value, tzoffset, _ = re.split('([+-]\d{4})$', value, 1)
ValueError: need more than 1 value to unpack
```

Do you have the same behavior on your side?


---
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 issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-01-28 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1813
  
Quickly tested with 4.10 pre jdk8 and it work creating vm and resize the 
root volume on XenServer 6.5.  tested using lvmiscsi and SolidFire storage 
plugins.



---
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-www issue #33: Updated the 2017 conference details

2017-01-27 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-www/pull/33
  
+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-www issue #33: Updated the 2017 conference details

2017-01-26 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-www/pull/33
  
otherwise, 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-www issue #33: Updated the 2017 conference details

2017-01-26 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-www/pull/33
  
Can you take the oportunity to upgrade the Year in this page too ?

https://github.com/apache/cloudstack-www/blob/master/source/layouts/layout.erb#L131


---
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 issue #1797: CLOUDSTACK-9630: Cannot use listNics API as advertis...

2017-01-25 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1797
  
Have you tested as @jburwell comments, `broadcasturi` cannot be part of the 
response for user, only for root-admin. Does this PR make `broadcasturi` 
visible to users?


---
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 issue #1797: CLOUDSTACK-9630: Cannot use listNics API as advertis...

2017-01-24 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1797
  
as it been confirmed the the attribute `broadcasturi = vlan://743` is only 
visible as admin and not as user? because the example have it.


---
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-docs-admin issue #41: 50% of the doc strings are translated into ...

2017-01-10 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/41
  
Also, I think we need to keep the `Makefile` with an empty `SPHINXOPS` So 
the default build will be for english version and RTD.org will detect the 
translation files and will do them automatically. from what I can see with the 
Chinese translation.

Please just fix those 2 things and it's a LGTM for me!

Thanks again!


---
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-docs-admin issue #41: 50% of the doc strings are translated into ...

2017-01-10 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/41
  
@lmrv , I've quickly reviewed the translated content.  but I'm wondering 
why the `make.sh` file is removed? 


---
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-docs-admin issue #41: 50% of the doc strings are translated into ...

2017-01-03 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/41
  
@rhtyd @swill @jburwell,

Does any of you would know if the header file is correct, for .po files?

I think the copyright line must be removed, 

```
# SOME DESCRIPTIVE TITLE.
# Copyright (C)
# This file is distributed under the same license as the Apache CloudStack 
Administration Documentation package.
#
```

And should be replace by something like this:

```
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements.  See the NOTICE file
# distributed with this work for additional information
# regarding copyright ownership.  The ASF licenses this file
# to you under the Apache License, Version 2.0 (the
# "License"); you may not use this file except in compliance
# with the License.  You may obtain a copy of the License at
#
#   http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing,
# software distributed under the License is distributed on an
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
# KIND, either express or implied.  See the License for the
# specific language governing permissions and limitations
# under the License.
```


Am I right or it's ok in the current state ?

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-docs-admin issue #41: 50% of the doc strings are translated into ...

2017-01-03 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/41
  
Hi @lmrv, I've built the site in FR and reviewing the doc...  So far I've 
only found a minor issue: header of .po files are improper, it should be the 
apache2 license.




---
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-docs-admin issue #40: index.po and ui.po translated in FR

2016-12-23 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/40
  
@lmrv I think you are now at 50% completed, right ?  could it be possible 
to update this PR so we cloud merge this first step ?

Continue the Good work !!


---
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 issue #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1858
  
👍 


---
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 issue #1769: tools: fix Dockerfile

2016-12-09 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1769
  
Hi @resmo,  I've made some changes on the Docker file so it might not be 
possible to merge this PR in the current state.

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 issue #1815: Fix docker image build for cloudstack-management

2016-12-09 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1815
  
Thanks @rhtyd !


---
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 issue #1815: Fix docker image build for cloudstack-management

2016-12-06 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1815
  
Thanks @rhtyd, let merge it so we have the simulator back online on docker 
hub and let's improve our Docker images...

@jburwell, @rhtyd , do I merge it or you do it? I'm just wondering since we 
are close from a release. 

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 issue #1815: Fix docker image build for cloudstack-management

2016-12-05 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1815
  
@sebgoa , I agree with you, it's just the easiest way to have a cloudstack 
with simulator preconfigured ready to work. We should not use package for the 
image management also.

I've made basic tests while doing the PR has I got images that were 
working. But I haven't test using the management server managing real 
hypervisors.


---
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 issue #1435: Dockerfile4.9

2016-12-02 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1435
  
This PR is not valid anymore replaced by Squashed: #1815 


---
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 #1435: Dockerfile4.9

2016-12-02 Thread pdion891
Github user pdion891 closed the pull request at:

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


---
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 #1815: Fix docker image build for cloudstack-managem...

2016-12-02 Thread pdion891
GitHub user pdion891 opened a pull request:

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

Fix docker image build for cloudstack-management

CLOUDSTACK-9651

Fix Docker images build for 4.10.0.0:
- simulator
- marvin
- management_centos6




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

$ git pull https://github.com/pdion891/cloudstack dockerfile4.10.0

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

https://github.com/apache/cloudstack/pull/1815.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 #1815


commit f88a1082c256b7fe83d2ec3df070f5dd835b3f86
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2016-12-02T22:24:26Z

CLOUDSTACK-9651; fix docker image build for cloudstack-management, 
simulator and marvin for 4.10.0.0
close #1435




---
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 #1814: fix docker images build

2016-12-02 Thread pdion891
Github user pdion891 closed the pull request at:

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


---
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 #1814: fix docker images build

2016-12-02 Thread pdion891
GitHub user pdion891 opened a pull request:

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

 fix docker images build

CLOUDSTACK-9651;

Fix Docker images build for:
- cloudstack-simulator
- cloudstack-management
- marvin

Updated for version 4.10.0.0 and the new jenkins system.

close #1435

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

$ git pull https://github.com/pdion891/cloudstack dockerfile4.10

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

https://github.com/apache/cloudstack/pull/1814.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 #1814


commit fe8a03fad7501340ef3f17dfe67be0a3afff783f
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2016-12-02T22:24:26Z

CLOUDSTACK-9651; fix docker image build for cloudstack-management, 
simulator and marvin for 4.10.0.0
close #1435




---
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 issue #1435: Dockerfile4.9

2016-12-01 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1435
  
I've retested the management server, it work with Jenkins as the build is 
back online.

@jburwell, do you know if there is a way to squash a commit within the same 
PR ?


---
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 issue #1435: Dockerfile4.9

2016-11-30 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1435
  
Hi @jburwell, do you think we can merge this in master or you are in a code 
freeze for 4.10.0.0 ?

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 issue #1435: Dockerfile4.9

2016-11-30 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1435
  
@resmo I've updated the PR. I've tested the simulator and marvin. 



---
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 issue #1769: tools: fix Dockerfile

2016-11-30 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1769
  
ok, I'll update the PR and merge it I think the PR go all required LGTM. 
I'll confirmed with John, in case he is in a code freeze period.


---
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 issue #1769: tools: fix Dockerfile

2016-11-30 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1769
  
@resmo  could you do a PR on top of #1435 ? or this have to be merged 
first? so we can get your chances and some of mine.


---
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 issue #1769: tools: fix Dockerfile

2016-11-30 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1769
  
So, should we go forward and merge #1435 + #1769  to master?
I might just need to rebase and update version number of the current commit.

With those 2 PR, we would have master back online on dockerhub so we can 
get an updater docker image for the simulator!




---
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 issue #1769: tools: fix Dockerfile

2016-11-24 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1769
  
Could you look if it conflict with #1435 ?  Because I made a pull request 
to automatically increment version in docker file when changing release of 
master via ```tools/build/setnextversion.sh```.




---
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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-11-23 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
btw, I don't think this PR should be backport since it would change 
behavior for someone that would upgrade from 4.x.x to 4.9.0 or 4.9.x(that would 
include this PR)


---
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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-11-22 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
I haven't retested recently but the upgrade path will need documentation 
update I think because of the ``commands.properties``.

Otherwise it's LGTM for me too.


---
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 issue #1692: Fix Smoke Test Failures

2016-10-20 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1692
  
Could it be possible this PR is specific to blueorangutan tests or they are 
to fix marvin tests too? 
we could be able to test if marvin, need to know how for blueorangutan.

Since it fix test I'm all for this PR including #1708 and #1703. I did 
quickly review the code and don't see any issues.



---
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 issue #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-10-12 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
Hi @remibergsma,  

We found this issue CLOUDSTACK-9500 where Public IP's is not removed from a 
VR when the Public IP is release. this cause routing issues because the arp 
table is refresh at any operation on the VR which reset de MAC for the release 
IP that can be used into another VR in a VPC deployement model. 

Is this something you faced? 

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-docs-admin issue #40: index.po and ui.po translated in FR

2016-10-01 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/40
  
Hi @lmrv, I've look at your test site, those the page 
https://cloudstack.le-morvan.com/4.8/service_offerings.html  translation is 
completed? because I see it half in english.

Thanks again for that huge work ! 


---
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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-10-01 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
I've re-perform my upgrade test this morning with this PR, I just add to 
change ``dynamic.apichecker.enabled`` value have the authentication working, so 
nothing else have to change.

As @PaulAngus raised, in an upgrade scenario where the management-server is 
build for scratch to perform a cloud upgrade, ``dynamic.apichecker.enabled`` 
will cause issues and headache.

So to me it would make more sense if during and upgrade the value of 
``dynamic.apichecker.enabled`` is automatically set to "true" which is already 
the value of a new deployment, then we should update the release notes so if 
users have customized the ``commands.properties`` file, it's up to them to 
rollback ``dynamic.apichecker.enabled`` to false and make sure they have a copy 
of the commands.properties file. what do you think @PaulAngus ?

Maybe the ideal scenario would be that if the ``commands.properties`` file 
exist, cloudstack during is upgrade phase when it's started would update is 
dynamic apichecker values to those defined in the commands.properties 

Does this should go in this PR or we consider this PR is 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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-30 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
@rhtyd I agree with you for the documentation part.

For sure in-place upgrade must be transparent to user and not required 
manual steps.


---
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 issue #872: Strongswan vpn feature

2016-09-29 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/872
  
in case of HTTPs  issue registering the template 
http://objects-east.cloud.ca/v1/5ef827605f884961b94881e928e7a250/swill/systemvm64template-master-4.6.0-xen.vhd.bz2
can be use.


---
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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-29 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
Thanks @abhinandanprateek.  So I guest it's an LGTM for this PR but it's 
not solving the whole issue for CLOUDSTACK-9489 which is configuration 
inconsistency between fresh install and upgrade. So to me this PR cannot close 
CLOUDSTACK-9489 unless we create a new jira issue for the auth issue after an 
upgrade where we would need to manually change ``dynamic.apichecker.enabled``.




---
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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-28 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
Thanks @rhtyd, @jburwell @abhinandanprateek for the clarifications.

Look like doing the ``update cloud.configuration set value='true' where 
name='dynamic.apichecker.enabled';`` fixe the authentication issue.

Should we expect another issue post upgrade because of this db change? 
should we roll it back later? I haven't found the file command.properties when 
running 4.9.1.0-SNAPSHOT via jetty in the repo.

also, in the context a user did customized some values, will they be reset?

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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-27 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
@abhinandanprateek did you ran a fresh install of 4.9.x  or did an upgrade 
from example 4.4.4 ?

The way I'm testing is as follow, in case I do something wrong:

1) ``git checkout tags/4.4.4 -b 4.4.4``
2) Deploy and start management-server
```
mvn -DskipTests=true clean install -T 2
mvn -Pdeveloper -pl developer -Ddeploydb
mvn -pl client jetty:run
```
4) Auth test on the ui,  then stop jetty 
5) Change version to upgrade: ``git fetch origin pull/1684/head:pr-1684``
6) Build the new version but not deploy the DB to force the upgrade path:
```
mvn -DskipTests=true clean install -T 2
mvn -pl client jetty:run
```
7) test auth as admin in the UI and it's not working.



---
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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-26 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
@dagsonstebo for your scenario 2 can you get the count of global settings ? 

via cloudmonkey as admin:
```
list configurations
```
You should have a count of about 468.

Also this current PR did apply some changes from the Config.java that were 
not applied in your upgrades. I have test upgrade path without this fix and I 
had no issues, but we end up with missing configurations entrees.


---
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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-26 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
@dagsonstebo, did you upgrade to 4.9 ?  So far, I had no authentication 
issue upgrading to 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.
---


[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-26 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
@abhinandanprateek  I did replace the following 4 settings by the original 
values. But after restarting jetty I'm unable to login, but this time the 
behavior is a bit different, it look like it doesn't try to login, no log or 
message in the UI.

```
update configuration set value=null where 
name='user.password.encoders.exclude';
update configuration set value='SHA256SALT,MD5,LDAP,PLAINTEXT' where 
name='user.authenticators.order';
update configuration set value=null where 
name='user.authenticators.exclude';
update configuration set value='SHA256SALT,MD5,LDAP,PLAINTEXT' where 
name='user.password.encoders.order';
```

Could it be possible the  authentication order is not followed at the 
login? if the password does not meet ``PBKDF2`` requirement, which is the first 
in the order list of 4.9.0, shouldn't try with the next encoder in the list and 
so on?




---
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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-25 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
upgrade from 4.7.0 to 4.9.0.1 not working with same error:
```
WARN  [o.a.c.s.a.PBKDF2UserAuthenticator] (qtp204585401-22:ctx-9e4c5615) 
(logid:1ca9f551) The stored password for admin isn't in the right format for 
this authenticator
```


---
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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-25 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
I've applied this fix to 4.7.0 and I got the upgrade from 4.4.4 working.  
but when I upgrade 4.4.4 to this PR  I end with inability to logon into the UI.


---
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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-25 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
Unfortunately, Authentication is not working after an upgrade.  the 
configration table as been updated  but login is not working anymore.

Upgrade test from 4.4.4 to the PR (4.9), I can see that configuration table 
changes, origin confi count = 413, after starting 4.9 = 488

Here is the error display from jetty when trying to login into the UI or 
via cloudmonkey:

```
WARN  [o.a.c.s.a.PBKDF2UserAuthenticator] (qtp368929124-24:ctx-97af69ff) 
(logid:0d6ca986) The stored password for admin isn't in the right format for 
this authenticator
```


---
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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-25 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1684
  
I will test this PR, allow me few days..
This will be ported into master as well right? 

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 issue #872: Strongswan vpn feature

2016-09-15 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/872
  
@jayapalu  do you have a configuration example of strongswan for the remote 
management VPN? I would validate that the generated configuration will work 
with Windows and OSx clients.

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 issue #1658: Added an additional JSON diff output to the ApiXmlDo...

2016-09-09 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1658
  
lgtm, that's usefull


---
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-docs-admin issue #40: index.po and ui.po translated in FR

2016-08-24 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/40
  
@lmrv if you have a chance to update all files copyright header, to must 
have the apache2 v2 license header.

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-docs-admin issue #40: index.po and ui.po translated in FR

2016-08-24 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/40
  
@lmrv This is a great start! 

I'm willing to merge this PR, but it's a work in progress so it will not be 
visible on the main site until the translation is mostly completed.

At least the frame work is in place which is very great! 

Thanks a lot.  It's going to be awesome to have the full translation. 


---
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-docs-admin issue #40: index.po and ui.po translated in FR

2016-08-23 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/40
  
I'll review this PR, give me few days...


---
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 issue #1599: Marvin: Fix codegenerator to work with API discovery

2016-08-18 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1599
  
The code seams to make sense.  I like the idea of api auto-discovery so 
it's a LGTM I guest.
I'm a bit out of context on this one but if it's usefull lets merge it.

Cheers,


---
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 issue #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

2016-08-16 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1638
  
Make sense,  thanks, I hope it will get thru then to add support for Ubuntu 
16.04 for the management server host and KVM.



---
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 issue #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

2016-08-16 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1638
  
Shouldn't we consider doing a 5.0.0 release that would support Java 8 or we 
want to go  Java8 at 4.10.0 ?

Otherwise I can't review, but I should be able to help on testing.


---
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 issue #1619: Add the Transifex config for next version of CS (4.1...

2016-08-15 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1619
  
LGTM, 

Does this command most be run every build or one time per branch only ? 
Shouldn't we document this process in the building doc or the wiki ?

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 issue #1619: Add the Transifex config for next version of CS (4.1...

2016-08-13 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1619
  
Does those files actually exist? otherwise how do we generate those files?



---
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 issue #1628: updated contributing.md

2016-08-13 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1628
  
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-docs-rn pull request #29: 4.9

2016-08-13 Thread pdion891
Github user pdion891 closed the pull request at:

https://github.com/apache/cloudstack-docs-rn/pull/29


---
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 issue #872: Strongswan vpn feature

2016-08-12 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/872
  
JIRA reference: https://issues.apache.org/jira/browse/CLOUDSTACK-8682



---
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-docs-admin issue #36: OSPF: WIP formatting documentation for ospf

2016-08-11 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/36
  
@agneya2001 , any update on the merge of your code in the upstream ?
I'm keeping this PR open until the actual code get merge into master

Regards,


---
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 issue #1630: Add projectid to project details page

2016-08-08 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1630
  
I've added some more information in the project detail page as on following 
screenshot:


![prj1](https://cloud.githubusercontent.com/assets/5923502/17500440/995a4420-5da3-11e6-947e-b1fe66dabbfe.png)

![prj2](https://cloud.githubusercontent.com/assets/5923502/17500444/9de96912-5da3-11e6-80b3-ca125017beae.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 issue #1054: CLOUDSTACK-8818: Use MySQL native connector with Pyt...

2016-08-08 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1054
  
ok, thanks, I will make sure the doc reflect that. 

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 issue #1054: CLOUDSTACK-8818: Use MySQL native connector with Pyt...

2016-08-08 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1054
  
Are you saying it's not like CentOS which require to install 
mysq-connector-python from mysql repo [1], on ubuntu the package would have not 
been renamed ?

[1] 
https://github.com/cloudops/cloudstack-docs-rn/blob/4.9/source/upgrade/_mysql_connector.rst



---
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-docs-rn pull request #29: 4.9

2016-08-08 Thread pdion891
GitHub user pdion891 opened a pull request:

https://github.com/apache/cloudstack-docs-rn/pull/29

4.9

Please confirm it make sense to add a new package repo for the MySQL driver.

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

$ git pull https://github.com/cloudops/cloudstack-docs-rn 4.9

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

https://github.com/apache/cloudstack-docs-rn/pull/29.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 #29


commit 0cfe73b39ac84ea1b9ef764a2eddba7ba48e2bf5
Author: Will Stevens <williamstev...@gmail.com>
Date:   2016-08-01T20:20:49Z

initial commit of the 4.9 changes

commit c083230f81dc23a4e0fd8a947f08fe6f8519ec17
Author: Will Stevens <williamstev...@gmail.com>
Date:   2016-08-03T19:11:28Z

updated the 'about' section

commit 166cdbf2574f1a0417611f681c3d88e843d62b12
Author: Will Stevens <williamstev...@gmail.com>
Date:   2016-08-04T20:39:31Z

updated known issues

commit a13414718c182f5a935b99d8f95e5c89ec9bdca6
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2016-08-06T18:09:20Z

complete upgrade paths to 4.9




---
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 issue #1629: CLOUDSTACK-9449: Fix typo in user role description

2016-08-07 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1629
  
Hum, This is a change to a file for an upgrade SQL to a release that is 
already GA. 
I think it would be preferable to get it into the 490_to_4100 file to avoid 
divergence between the voted release and the repo.




---
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-www pull request #28: website update for 4.9 and page template ch...

2016-08-07 Thread pdion891
GitHub user pdion891 opened a pull request:

https://github.com/apache/cloudstack-www/pull/28

website update for 4.9 and page template changes

This PR update the site for the 4.9.0 release and also add change on the 
way page get generated.

versions for download and archives are now stored in an answer file 
``data/cloudstack.yml`` to simplify side update when new version of cloudstack 
is released.

commit to review: 
[d5cb84c38d319f4729f27df5bc73eed2bb12bbb7](https://github.com/apache/cloudstack-www/commit/d5cb84c38d319f4729f27df5bc73eed2bb12bbb7)

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

$ git pull https://github.com/pdion891/cloudstack-www 4.9

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

https://github.com/apache/cloudstack-www/pull/28.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 #28


commit d5cb84c38d319f4729f27df5bc73eed2bb12bbb7
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2016-08-07T15:30:58Z

pkg versions now varialbe in cloudstack.yml. update download and archive 
pages as template, upgrade version to 4.9.0

commit b01812a4171c8a3da4ac95d99b9e9693a1c4220d
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2016-08-07T15:32:33Z

update gitignore

commit 21afdd57951c351c1221fdd5aea8fee19d839ce7
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2016-08-07T15:37:56Z

add api for 4.9.0




---
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 issue #1054: CLOUDSTACK-8818: Use MySQL native connector with Pyt...

2016-08-06 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1054
  
Could it be possible the file ``debian/control`` should be updated to 
depend on mysql-connector-python  deb package instead of python-mysql.connector 
? 

I have this PR ready for the Debian side: 

https://github.com/pdion891/cloudstack/blob/b6ca215b17b7d65fd08cf567c8b584cc76aa7e52/debian/control

I'm just not sure if it's wanted to not have Debian updated to use the new 
driver?

Regards,


---
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 #:

2016-08-06 Thread pdion891
Github user pdion891 commented on the pull request:


https://github.com/apache/cloudstack/commit/3315eb5420e1b2cedf906aeb709740efc2f08cfd#commitcomment-18549608
  
Look like [building deb 
package](http://builds.apachecloudstack.net/job/package-master-deb/) fail 
because changelog is in invalid format

initial lines should look like this :
```
 cloudstack (4.10.0-SNAPSHOT) unstable; urgency=low

  * Update the version to 4.10.0

 -- the Apache CloudStack project <dev@cloudstack.apache.org>  Sat, 6 Aug 
2016 16:54:00 -0400

```

Also, while trying to build it on Debian 8.5 I'm having error about invalid 
version format, look like the line #1 should look like this:

```
cloudstack (4.10.0) unstable; urgency=low
```

So should we just use 4.10.0 instead of 4.10.0-SNAPSHOT ? this might be 
just related to Debian, I haven't test on Ubuntu 14.04.




---
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 #:

2016-08-06 Thread pdion891
Github user pdion891 commented on the pull request:


https://github.com/apache/cloudstack/commit/4347776ac6ef9ae86fb016862f4a6b2376f8319a#commitcomment-18549598
  
In setup/db/db/schema-481to490.sql:
In setup/db/db/schema-481to490.sql on line 473:
Forget about the missing field look like it work. upgrade tested from 4.7.1 
to 4.9.0.
thanks  @rhtyd !


---
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-docs-install pull request #30: update for 4.9.0

2016-08-06 Thread pdion891
GitHub user pdion891 opened a pull request:

https://github.com/apache/cloudstack-docs-install/pull/30

update for 4.9.0

* Upgrade release to 4.9.0;
* update library for building from source.
* Include PR #28 and #29. 
* fixes some confusion about nfs,  to help #27 
* CentOS 6 reference updated from 6.5 to 6.8


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

$ git pull https://github.com/pdion891/cloudstack-docs-install master

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

https://github.com/apache/cloudstack-docs-install/pull/30.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 #30


commit 1f805eaa2cef32c40867624911e3f44919defa38
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2015-12-19T21:27:16Z

update release to 4.7

commit 7c9cbc7038143a1a4d50e4a29784e16f22818f93
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2015-12-19T21:27:30Z

Merge branch 'master' of 
https://git-wip-us.apache.org/repos/asf/cloudstack-docs-install

commit bd562e1912da4cc3d5c17ae14714cc5951bbc71d
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2016-01-20T15:42:30Z

Merge branch 'master' of 
https://git-wip-us.apache.org/repos/asf/cloudstack-docs-install

commit 0b23e9d53477a29a9a2d61a0fb59df2e813f6109
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2016-02-17T01:51:03Z

update to version 4.8.0

commit c577db63ea11df2e784be7946f9c41cb0e0a7506
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2016-08-06T18:14:57Z

ignoring .pyc files

commit 3c1084775ac759dea76d6ceea62612a2a9420b8f
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2016-08-06T18:15:50Z

remove file tracking

commit 1265b05f75460828eec48ad3f8e741e012b1d703
Author: Rajani Karuturi <karut...@users.noreply.github.com>
Date:   2016-08-05T06:51:34Z

corrected -d value while building rpm packages

Its listed as centos6 while its centos63.
Also added the help section for other available options

Signed-off-by: Pierre-Luc Dion <pdion...@apache.org>

commit 834e36eeb49fe20287b5dee8c6ce234915c10a6e
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2016-08-06T21:30:09Z

update version to 4.9.0




---
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 #1630: Add projectid to project details page

2016-08-06 Thread pdion891
GitHub user pdion891 opened a pull request:

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

Add projectid to project details page

Display the project ID on the details view of a project.


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

$ git pull https://github.com/pdion891/cloudstack projectid

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

https://github.com/apache/cloudstack/pull/1630.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 #1630


commit 98228c5fb1d5fd6eecd46eff631e630e4b192d08
Author: Pierre-Luc Dion <pdion...@apache.org>
Date:   2016-08-06T18:38:20Z

add projectid to project details page




---
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-docs-install issue #27: make sure we use the same nfs paths. /pri...

2016-08-06 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-install/pull/27
  
Hi @bugoff, 

thanks for rising the source of confusion about nfs shares. Although, you 
change from ``/primary`` to ``/mnt/primary`` but quickly looking at the page 
other reference to nfs shares use ``/export/primary``.

I think it would be preferable to standardize to ``/export/primary``, make 
sense?

systemvm template path also need to be updated

Cheers,


---
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 #:

2016-08-05 Thread pdion891
Github user pdion891 commented on the pull request:


https://github.com/apache/cloudstack/commit/4347776ac6ef9ae86fb016862f4a6b2376f8319a#commitcomment-18546909
  
In setup/db/db/schema-481to490.sql:
In setup/db/db/schema-481to490.sql on line 473:
Role User description is for root, shouldn't it be changed for "Default 
User role"?

also, starting 4.9.0 fail because of a missing field:
```
ERROR 1054 (42S22): Unknown column 'account.role_id' in 'field list'
```


---
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-docs issue #18: Removed support for Citrix XenServer

2016-08-01 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs/pull/18
  
Is this for a part of a particular version of CloudStack, or it never been 
supported?


---
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-docs-admin issue #39: corrected a typo

2016-06-29 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack-docs-admin/pull/39
  
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 issue #1582: CLOUDSTACK-9408 for the move away from download.clou...

2016-06-13 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1582
  
if we are to replace the old default template centos5, would it be possible 
on the new one to support user-data, ssh keys and password reset? so it would 
help exposing cloudstack capabilities.

download.apachecloudstack{.org|.net} is already set to direct traffic on 
cloudstack.apt-get.eu. so it can be use.




---
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 issue #1435: Dockerfile4.9

2016-06-09 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1435
  
@runseb I agree, images must be smaller. for the simulator, it's kind of 
usefull to have everything in it but it's not container best practice for sure. 
 other images, we could probably use alpine linux, but still need to figure out 
how to run cloudstack as a single .war file.


---
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 issue #1435: Dockerfile4.9

2016-06-08 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1435
  
@swill, @runseb does one of you could LGTM or comment  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: Dockerfile4.9

2016-05-31 Thread pdion891
Github user pdion891 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1435#discussion_r65180940
  
--- Diff: tools/docker/Dockerfile.centos6 ---
@@ -18,14 +18,16 @@
 FROM centos:6
 
 MAINTAINER "Apache CloudStack" <dev@cloudstack.apache.org>
-LABEL Vendor="Apache.org" License="ApacheV2" Version="4.6.0"
+LABEL Vendor="Apache.org" License="ApacheV2" Version="4.9.0-SNAPSHOT"
 
-ENV 
PKG_URL=http://jenkins.buildacloud.org/job/package-rhel63-master/lastSuccessfulBuild/artifact/dist/rpmbuild/RPMS/x86_64
+ENV 
PKG_URL=http://74.121.245.247/job/package-master-rhel63/lastSuccessfulBuild/artifact/dist/rpmbuild/RPMS/x86_64
--- End diff --

@rhtyd I've create you an account on the Jenkins server. not a lot of jobs 
are there yet. 
Lets continue the discussion on slack...


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


  1   2   3   >