You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Rajani Karuturi <ra...@apache.org> on 2017/06/20 05:45:05 UTC

[VOTE] Apache CloudStack 4.10.0.0 RC5

Hi All,

I've created 4.10.0.0 release with the following artifacts up for a vote:

Git Branch and Commit SH:
https://github.com/apache/cloudstack/commit/058e34224c0555396c043c6473ac07ab3e8424cd
Commit:058e34224c0555396c043c6473ac07ab3e8424cd
Branch: https://github.com/apache/cloudstack/tree/4.10.0.0-RC20170620T1023

Source release (checksums and signatures are available at the same
location):
https://dist.apache.org/repos/dist/dev/cloudstack/4.10.0.0/

SystemVm Templates: http://download.cloudstack.org/systemvm/4.10/RC5/

New PRs merged since RC4: #2150 and #2089

PGP release keys (signed using CBB44821):
https://dist.apache.org/repos/dist/release/cloudstack/KEYS

Vote will be open for 72 hours.

For sanity in tallying the vote, can PMC members please be sure to indicate
"(binding)" with their vote?

[ ] +1  approve
[ ] +0  no opinion
[ ] -1  disapprove (and reason why)


Thanks,
~Rajani
http://cloudplatform.accelerite.com/

Re: [VOTE] Apache CloudStack 4.10.0.0 RC5

Posted by Milamber <mi...@apache.org>.
Hello,

I've tested this version on Ubuntu 16.04 / KVM with xenial deb packages 
(generated with docker build).

1/ One (major?) issue : On the management node, the 
cloudstack-management service don't start correctly. The error message 
from systemctl/jsvc is:
  jsvc.exec[9299]: 2017-06-20 16:29:35 9297 jsvc.exec error: Service 
killed by signal 11

(signal 11 is segmentation fault)

I can start the management service directly with the java command (with 
openjdk8 or oracle jdk8)


2/ One minor issue: during the first startup of the management service, 
the DB update process have this error [see below]

3/ A lot of this message in the log:
Tue Jun 20 18:10:18 WET 2017 WARN: Establishing SSL connection without 
server's identity verification is not recommended. According to MySQL 
5.5.45+, 5.6.26+ and 5.7.6+ requirements SSL connection must be 
established by default if explicit option isn't set. For compliance with 
existing applications not using SSL the verifyServerCertificate property 
is set to 'false'. You need either to explicitly disable SSL by setting 
useSSL=false, or set useSSL=true and provide truststore for server 
certificate verification.

4/ Always this error using cloudmonkey
(xenial-mgr1) >  update configuration name="vm.password.length" value="8"
Error 431: Please enter a value greater than 6 for the configuration 
parameter:vm.password.length
cserrorcode = 4350
errorcode = 431
errortext = Please enter a value greater than 6 for the configuration 
parameter:vm.password.length

Fix by this PR: https://github.com/apache/cloudstack/pull/1902 
(CLOUDSTACK-9736)


Milamber

=======

INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) DB version = 4.0.0 Code Version = 4.10.0.0
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Database upgrade must be performed from 4.0.0 to 4.10.0.0
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade40to41 to upgrade from 4.0.0-4.1.0 to 4.1.0
ERROR [c.c.u.d.Upgrade410to420] (localhost-startStop-1:null) (logid:) 
migrateDatafromIsoIdInVolumesTable:Exception:Unknown column 'iso_id1' in 
'field list'
com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: Unknown 
column 'iso_id1' in 'field list'
     at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native 
Method)
     at 
sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
     at 
sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
     at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
     at com.mysql.jdbc.Util.handleNewInstance(Util.java:404)
     at com.mysql.jdbc.Util.getInstance(Util.java:387)
     at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:939)
     at com.mysql.jdbc.MysqlIO.checkErrorPacket(MysqlIO.java:3878)
     at com.mysql.jdbc.MysqlIO.checkErrorPacket(MysqlIO.java:3814)
     at com.mysql.jdbc.MysqlIO.sendCommand(MysqlIO.java:2478)
     at com.mysql.jdbc.MysqlIO.sqlQueryDirect(MysqlIO.java:2625)
     at com.mysql.jdbc.ConnectionImpl.execSQL(ConnectionImpl.java:2551)
     at 
com.mysql.jdbc.PreparedStatement.executeInternal(PreparedStatement.java:1861)
     at 
com.mysql.jdbc.PreparedStatement.executeQuery(PreparedStatement.java:1962)
     at 
org.apache.commons.dbcp.DelegatingPreparedStatement.executeQuery(DelegatingPreparedStatement.java:96)
     at 
org.apache.commons.dbcp.DelegatingPreparedStatement.executeQuery(DelegatingPreparedStatement.java:96)
     at 
com.cloud.upgrade.dao.Upgrade410to420.migrateDatafromIsoIdInVolumesTable(Upgrade410to420.java:2381)
     at 
com.cloud.upgrade.dao.Upgrade410to420.performDataMigration(Upgrade410to420.java:110)
     at 
com.cloud.upgrade.DatabaseUpgradeChecker.upgrade(DatabaseUpgradeChecker.java:429)
     at 
com.cloud.upgrade.DatabaseUpgradeChecker.check(DatabaseUpgradeChecker.java:509)
     at 
org.apache.cloudstack.spring.lifecycle.CloudStackExtendedLifeCycle.checkIntegrity(CloudStackExtendedLifeCycle.java:65)
     at 
org.apache.cloudstack.spring.lifecycle.CloudStackExtendedLifeCycle.start(CloudStackExtendedLifeCycle.java:55)
     at 
org.springframework.context.support.DefaultLifecycleProcessor.doStart(DefaultLifecycleProcessor.java:173)
     at 
org.springframework.context.support.DefaultLifecycleProcessor.access$200(DefaultLifecycleProcessor.java:51)
     at 
org.springframework.context.support.DefaultLifecycleProcessor$LifecycleGroup.start(DefaultLifecycleProcessor.java:346)
     at 
org.springframework.context.support.DefaultLifecycleProcessor.startBeans(DefaultLifecycleProcessor.java:149)
     at 
org.springframework.context.support.DefaultLifecycleProcessor.onRefresh(DefaultLifecycleProcessor.java:112)
     at 
org.springframework.context.support.AbstractApplicationContext.finishRefresh(AbstractApplicationContext.java:879)
     at 
org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:545)
     at 
org.apache.cloudstack.spring.module.model.impl.DefaultModuleDefinitionSet.loadContext(DefaultModuleDefinitionSet.java:145)
     at 
org.apache.cloudstack.spring.module.model.impl.DefaultModuleDefinitionSet$2.with(DefaultModuleDefinitionSet.java:122)
     at 
org.apache.cloudstack.spring.module.model.impl.DefaultModuleDefinitionSet.withModule(DefaultModuleDefinitionSet.java:245)
     at 
org.apache.cloudstack.spring.module.model.impl.DefaultModuleDefinitionSet.withModule(DefaultModuleDefinitionSet.java:250)
     at 
org.apache.cloudstack.spring.module.model.impl.DefaultModuleDefinitionSet.withModule(DefaultModuleDefinitionSet.java:233)
     at 
org.apache.cloudstack.spring.module.model.impl.DefaultModuleDefinitionSet.loadContexts(DefaultModuleDefinitionSet.java:117)
     at 
org.apache.cloudstack.spring.module.model.impl.DefaultModuleDefinitionSet.load(DefaultModuleDefinitionSet.java:79)
     at 
org.apache.cloudstack.spring.module.factory.ModuleBasedContextFactory.loadModules(ModuleBasedContextFactory.java:37)
     at 
org.apache.cloudstack.spring.module.factory.CloudStackSpringContext.init(CloudStackSpringContext.java:71)
     at 
org.apache.cloudstack.spring.module.factory.CloudStackSpringContext.<init>(CloudStackSpringContext.java:58)
     at 
org.apache.cloudstack.spring.module.factory.CloudStackSpringContext.<init>(CloudStackSpringContext.java:62)
     at 
org.apache.cloudstack.spring.module.web.CloudStackContextLoaderListener.contextInitialized(CloudStackContextLoaderListener.java:52)
     at 
org.apache.catalina.core.StandardContext.listenerStart(StandardContext.java:5068)
     at 
org.apache.catalina.core.StandardContext.startInternal(StandardContext.java:5584)
     at org.apache.catalina.util.LifecycleBase.start(LifecycleBase.java:147)
     at 
org.apache.catalina.core.ContainerBase.addChildInternal(ContainerBase.java:899)
     at 
org.apache.catalina.core.ContainerBase.addChild(ContainerBase.java:875)
     at 
org.apache.catalina.core.StandardHost.addChild(StandardHost.java:652)
     at 
org.apache.catalina.startup.HostConfig.deployDirectory(HostConfig.java:1259)
     at 
org.apache.catalina.startup.HostConfig$DeployDirectory.run(HostConfig.java:1998)
     at 
java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
     at java.util.concurrent.FutureTask.run(FutureTask.java:266)
     at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
     at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
     at java.lang.Thread.run(Thread.java:748)
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade410to420 to upgrade from 4.1.0-4.2.0 to 
4.2.0
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade420to421 to upgrade from 4.2.0-4.2.1 to 
4.2.1
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade421to430 to upgrade from 4.2.1-4.3.0 to 
4.3.0
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade430to440 to upgrade from 4.3.0-4.4.0 to 
4.4.0
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade440to441 to upgrade from 4.4.0-4.4.1 to 
4.4.1
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade441to442 to upgrade from 4.4.1-4.4.2 to 
4.4.2
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade442to450 to upgrade from 4.4.2-4.5.0 to 
4.5.0
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade450to451 to upgrade from 4.5.0-4.5.1 to 
4.5.1
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade451to452 to upgrade from 4.5.1-4.5.2 to 
4.5.2
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade452to460 to upgrade from 4.5.2-4.6.0 to 
4.6.0
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade460to461 to upgrade from 4.6.0-4.6.1 to 
4.6.1
INFO  [c.c.u.d.Upgrade461to470] (localhost-startStop-1:null) (logid:) 
Altered cloud_usage.cloud_usage table and added column quota_calculated
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade461to470 to upgrade from 4.6.1-4.7.0 to 
4.7.0
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade470to471 to upgrade from 4.7.0-4.7.1 to 
4.7.1
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade471to480 to upgrade from 4.7.1-4.8.0 to 
4.8.0
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade480to481 to upgrade from 4.8.0-4.8.1 to 
4.8.1
ERROR [c.c.u.PropertiesUtil] (localhost-startStop-1:null) (logid:) 
Unable to find properties file: commands.properties
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade481to490 to upgrade from 4.8.1-4.9.0 to 
4.9.0
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade490to4910 to upgrade from 4.9.0-4.9.1.0 
to 4.9.1.0
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade4910to4920 to upgrade from 
4.9.1.0-4.9.2.0 to 4.9.2.0
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade4920to4930 to upgrade from 
4.9.2.0-4.9.3.0 to 4.9.3.0
WARN  [c.c.u.d.Upgrade4930to41000] (localhost-startStop-1:null) (logid:) 
4.10.0.0XenServer SystemVm template not found. XenServer hypervisor is 
not used, so not failing upgrade
WARN  [c.c.u.d.Upgrade4930to41000] (localhost-startStop-1:null) (logid:) 
4.10.0.0Ovm3 SystemVm template not found. Ovm3 hypervisor is not used, 
so not failing upgrade
WARN  [c.c.u.d.Upgrade4930to41000] (localhost-startStop-1:null) (logid:) 
4.10.0.0KVM SystemVm template not found. KVM hypervisor is not used, so 
not failing upgrade
WARN  [c.c.u.d.Upgrade4930to41000] (localhost-startStop-1:null) (logid:) 
4.10.0.0LXC SystemVm template not found. LXC hypervisor is not used, so 
not failing upgrade
WARN  [c.c.u.d.Upgrade4930to41000] (localhost-startStop-1:null) (logid:) 
4.10.0.0VMware SystemVm template not found. VMware hypervisor is not 
used, so not failing upgrade
WARN  [c.c.u.d.Upgrade4930to41000] (localhost-startStop-1:null) (logid:) 
4.10.0.0Hyperv SystemVm template not found. Hyperv hypervisor is not 
used, so not failing upgrade
INFO  [c.c.u.DatabaseUpgradeChecker] (localhost-startStop-1:null) 
(logid:) Cleanup upgrade Upgrade4930to41000 to upgrade from 
4.9.3.0-4.10.0.0 to 4.10.0.0
INFO  [o.a.c.s.l.CloudStackExtendedLifeCycle] 
(localhost-startStop-1:null) (logid:) Configuring CloudStack Components
INFO  [o.a.c.s.l.CloudStackExtendedLifeCycle] 
(localhost-startStop-1:null) (logid:) Done Configuring CloudStack Components



On 20/06/2017 05:45, Rajani Karuturi wrote:
> Hi All,
>
> I've created 4.10.0.0 release with the following artifacts up for a vote:
>
> Git Branch and Commit SH:
> https://github.com/apache/cloudstack/commit/058e34224c0555396c043c6473ac07ab3e8424cd
> Commit:058e34224c0555396c043c6473ac07ab3e8424cd
> Branch: https://github.com/apache/cloudstack/tree/4.10.0.0-RC20170620T1023
>
> Source release (checksums and signatures are available at the same
> location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.10.0.0/
>
> SystemVm Templates: http://download.cloudstack.org/systemvm/4.10/RC5/
>
> New PRs merged since RC4: #2150 and #2089
>
> PGP release keys (signed using CBB44821):
> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>
> Vote will be open for 72 hours.
>
> For sanity in tallying the vote, can PMC members please be sure to indicate
> "(binding)" with their vote?
>
> [ ] +1  approve
> [ ] +0  no opinion
> [ ] -1  disapprove (and reason why)
>
>
> Thanks,
> ~Rajani
> http://cloudplatform.accelerite.com/
>


Re: [VOTE] Apache CloudStack 4.10.0.0 RC5

Posted by "Tutkowski, Mike" <Mi...@netapp.com>.
I should also point out that I performed my tests on a fresh system (I did not upgrade from a prior system).

On 6/20/17, 9:24 AM, "Tutkowski, Mike" <Mi...@netapp.com> wrote:

    +1 (binding)
    
    I ran all of the managed-storage automated regression tests against the prior RC. The results can be located here: https://github.com/apache/cloudstack/pull/2141.
    
    As noted in https://github.com/apache/cloudstack/pull/2150, I submitted fixes to resolve two issues I had discovered.
    
    I verified this new code is present in the new RC. I also examined the new commits to this new RC that were not related to my PR and I am comfortable that it should not adversely impact managed storage.
    
    In addition to the automated tests noted above (which are related to XenServer), I performed a lot of manual testing using managed storage with vSphere and KVM and it all passed.
    
    Thanks!
    Mike
    
    On 6/19/17, 11:45 PM, "Rajani Karuturi" <ra...@apache.org> wrote:
    
        Hi All,
        
        I've created 4.10.0.0 release with the following artifacts up for a vote:
        
        Git Branch and Commit SH:
        https://github.com/apache/cloudstack/commit/058e34224c0555396c043c6473ac07ab3e8424cd
        Commit:058e34224c0555396c043c6473ac07ab3e8424cd
        Branch: https://github.com/apache/cloudstack/tree/4.10.0.0-RC20170620T1023
        
        Source release (checksums and signatures are available at the same
        location):
        https://dist.apache.org/repos/dist/dev/cloudstack/4.10.0.0/
        
        SystemVm Templates: http://download.cloudstack.org/systemvm/4.10/RC5/
        
        New PRs merged since RC4: #2150 and #2089
        
        PGP release keys (signed using CBB44821):
        https://dist.apache.org/repos/dist/release/cloudstack/KEYS
        
        Vote will be open for 72 hours.
        
        For sanity in tallying the vote, can PMC members please be sure to indicate
        "(binding)" with their vote?
        
        [ ] +1  approve
        [ ] +0  no opinion
        [ ] -1  disapprove (and reason why)
        
        
        Thanks,
        ~Rajani
        http://cloudplatform.accelerite.com/
        
    
    


Re: [VOTE] Apache CloudStack 4.10.0.0 RC5

Posted by "Tutkowski, Mike" <Mi...@netapp.com>.
+1 (binding)

I ran all of the managed-storage automated regression tests against the prior RC. The results can be located here: https://github.com/apache/cloudstack/pull/2141.

As noted in https://github.com/apache/cloudstack/pull/2150, I submitted fixes to resolve two issues I had discovered.

I verified this new code is present in the new RC. I also examined the new commits to this new RC that were not related to my PR and I am comfortable that it should not adversely impact managed storage.

In addition to the automated tests noted above (which are related to XenServer), I performed a lot of manual testing using managed storage with vSphere and KVM and it all passed.

Thanks!
Mike

On 6/19/17, 11:45 PM, "Rajani Karuturi" <ra...@apache.org> wrote:

    Hi All,
    
    I've created 4.10.0.0 release with the following artifacts up for a vote:
    
    Git Branch and Commit SH:
    https://github.com/apache/cloudstack/commit/058e34224c0555396c043c6473ac07ab3e8424cd
    Commit:058e34224c0555396c043c6473ac07ab3e8424cd
    Branch: https://github.com/apache/cloudstack/tree/4.10.0.0-RC20170620T1023
    
    Source release (checksums and signatures are available at the same
    location):
    https://dist.apache.org/repos/dist/dev/cloudstack/4.10.0.0/
    
    SystemVm Templates: http://download.cloudstack.org/systemvm/4.10/RC5/
    
    New PRs merged since RC4: #2150 and #2089
    
    PGP release keys (signed using CBB44821):
    https://dist.apache.org/repos/dist/release/cloudstack/KEYS
    
    Vote will be open for 72 hours.
    
    For sanity in tallying the vote, can PMC members please be sure to indicate
    "(binding)" with their vote?
    
    [ ] +1  approve
    [ ] +0  no opinion
    [ ] -1  disapprove (and reason why)
    
    
    Thanks,
    ~Rajani
    http://cloudplatform.accelerite.com/
    


Re: [VOTE] Apache CloudStack 4.10.0.0 RC5

Posted by Rajani Karuturi <ra...@apache.org>.
Agree. Will create rc6 soon.

~Rajani

Sent from phone.

On 23 Jun 2017 3:17 p.m., "Wido den Hollander" <wi...@widodh.nl> wrote:

>
> > Op 22 juni 2017 om 21:38 schreef Kris Sterckx <
> kris.sterckx@nuagenetworks.net>:
> >
> >
> > Sorry , i meant  PR/2084 <https://github.com/apache/cloudstack/pull/2084
> >
> >
>
> Ok, so revert and RC6 then?
>
> Wido
>
> > On 22 June 2017 at 21:25, Kris Sterckx <kr...@nuagenetworks.net>
> > wrote:
> >
> > > Hi all
> > >
> > >
> > > Thanks to Daan for driving the work at PR/2155
> > > <https://github.com/apache/cloudstack/pull/2155> , but i believe it
> needs
> > > more soak time.
> > >
> > > We found today this PR is breaking VR-provided Guest DNS for SDN
> providers
> > > also.
> > >
> > > As Daan also suggested, i would recommend taking it out 4.10 and give
> it
> > > some more cycles in 4.11.
> > >
> > > We will support qualifying it.
> > >
> > > thanks,
> > >
> > > Kris
> > >
> > > - Nuage Networks
> > >
> > > On 21 June 2017 at 16:45, Kris Sterckx <kris.sterckx@nuagenetworks.net
> >
> > > wrote:
> > >
> > >> Hi  Wido, Daan, all
> > >>
> > >>
> > >> We have just pushed a workaround
> > >>
> > >> https://github.com/apache/cloudstack/pull/2155
> > >>
> > >> This works for us.
> > >>
> > >> The CsPassword class seems like not needed at all ?  The important
> part
> > >> for us is that  iptables_change = True  is set also in case of
> password
> > >> handling.  That was missing.
> > >>
> > >> Daan, we left a TODO for you to look into deeper though. We worked at
> > >> getting the databag handling fixed inside the class but as we don't
> have
> > >> full background, we kept hitting issues. Until we then saw that
> commenting
> > >> out the lines which we did actually works as well.
> > >>
> > >> I suggest the /2155 is reviewed by the involved stakeholders and
> based on
> > >> that we decide ?
> > >>
> > >>
> > >> thanks and toi-toi :-)
> > >>
> > >>
> > >> Kris
> > >>
> > >>
> > >>
> > >>
> > >
> > >
>

Re: [VOTE] Apache CloudStack 4.10.0.0 RC5

Posted by Wido den Hollander <wi...@widodh.nl>.
> Op 22 juni 2017 om 21:38 schreef Kris Sterckx <kr...@nuagenetworks.net>:
> 
> 
> Sorry , i meant  PR/2084 <https://github.com/apache/cloudstack/pull/2084>
> 

Ok, so revert and RC6 then?

Wido

> On 22 June 2017 at 21:25, Kris Sterckx <kr...@nuagenetworks.net>
> wrote:
> 
> > Hi all
> >
> >
> > Thanks to Daan for driving the work at PR/2155
> > <https://github.com/apache/cloudstack/pull/2155> , but i believe it needs
> > more soak time.
> >
> > We found today this PR is breaking VR-provided Guest DNS for SDN providers
> > also.
> >
> > As Daan also suggested, i would recommend taking it out 4.10 and give it
> > some more cycles in 4.11.
> >
> > We will support qualifying it.
> >
> > thanks,
> >
> > Kris
> >
> > - Nuage Networks
> >
> > On 21 June 2017 at 16:45, Kris Sterckx <kr...@nuagenetworks.net>
> > wrote:
> >
> >> Hi  Wido, Daan, all
> >>
> >>
> >> We have just pushed a workaround
> >>
> >> https://github.com/apache/cloudstack/pull/2155
> >>
> >> This works for us.
> >>
> >> The CsPassword class seems like not needed at all ?  The important part
> >> for us is that  iptables_change = True  is set also in case of password
> >> handling.  That was missing.
> >>
> >> Daan, we left a TODO for you to look into deeper though. We worked at
> >> getting the databag handling fixed inside the class but as we don't have
> >> full background, we kept hitting issues. Until we then saw that commenting
> >> out the lines which we did actually works as well.
> >>
> >> I suggest the /2155 is reviewed by the involved stakeholders and based on
> >> that we decide ?
> >>
> >>
> >> thanks and toi-toi :-)
> >>
> >>
> >> Kris
> >>
> >>
> >>
> >>
> >
> >

Re: [VOTE] Apache CloudStack 4.10.0.0 RC5

Posted by Kris Sterckx <kr...@nuagenetworks.net>.
Sorry , i meant  PR/2084 <https://github.com/apache/cloudstack/pull/2084>

On 22 June 2017 at 21:25, Kris Sterckx <kr...@nuagenetworks.net>
wrote:

> Hi all
>
>
> Thanks to Daan for driving the work at PR/2155
> <https://github.com/apache/cloudstack/pull/2155> , but i believe it needs
> more soak time.
>
> We found today this PR is breaking VR-provided Guest DNS for SDN providers
> also.
>
> As Daan also suggested, i would recommend taking it out 4.10 and give it
> some more cycles in 4.11.
>
> We will support qualifying it.
>
> thanks,
>
> Kris
>
> - Nuage Networks
>
> On 21 June 2017 at 16:45, Kris Sterckx <kr...@nuagenetworks.net>
> wrote:
>
>> Hi  Wido, Daan, all
>>
>>
>> We have just pushed a workaround
>>
>> https://github.com/apache/cloudstack/pull/2155
>>
>> This works for us.
>>
>> The CsPassword class seems like not needed at all ?  The important part
>> for us is that  iptables_change = True  is set also in case of password
>> handling.  That was missing.
>>
>> Daan, we left a TODO for you to look into deeper though. We worked at
>> getting the databag handling fixed inside the class but as we don't have
>> full background, we kept hitting issues. Until we then saw that commenting
>> out the lines which we did actually works as well.
>>
>> I suggest the /2155 is reviewed by the involved stakeholders and based on
>> that we decide ?
>>
>>
>> thanks and toi-toi :-)
>>
>>
>> Kris
>>
>>
>>
>>
>
>

Re: [VOTE] Apache CloudStack 4.10.0.0 RC5

Posted by Kris Sterckx <kr...@nuagenetworks.net>.
Hi all


Thanks to Daan for driving the work at PR/2155
<https://github.com/apache/cloudstack/pull/2155> , but i believe it needs
more soak time.

We found today this PR is breaking VR-provided Guest DNS for SDN providers
also.

As Daan also suggested, i would recommend taking it out 4.10 and give it
some more cycles in 4.11.

We will support qualifying it.

thanks,

Kris

- Nuage Networks

On 21 June 2017 at 16:45, Kris Sterckx <kr...@nuagenetworks.net>
wrote:

> Hi  Wido, Daan, all
>
>
> We have just pushed a workaround
>
> https://github.com/apache/cloudstack/pull/2155
>
> This works for us.
>
> The CsPassword class seems like not needed at all ?  The important part
> for us is that  iptables_change = True  is set also in case of password
> handling.  That was missing.
>
> Daan, we left a TODO for you to look into deeper though. We worked at
> getting the databag handling fixed inside the class but as we don't have
> full background, we kept hitting issues. Until we then saw that commenting
> out the lines which we did actually works as well.
>
> I suggest the /2155 is reviewed by the involved stakeholders and based on
> that we decide ?
>
>
> thanks and toi-toi :-)
>
>
> Kris
>
>
>
>

Re: [VOTE] Apache CloudStack 4.10.0.0 RC5

Posted by Kris Sterckx <kr...@nuagenetworks.net>.
Hi  Wido, Daan, all


We have just pushed a workaround

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

This works for us.

The CsPassword class seems like not needed at all ?  The important part for
us is that  iptables_change = True  is set also in case of password
handling.  That was missing.

Daan, we left a TODO for you to look into deeper though. We worked at
getting the databag handling fixed inside the class but as we don't have
full background, we kept hitting issues. Until we then saw that commenting
out the lines which we did actually works as well.

I suggest the /2155 is reviewed by the involved stakeholders and based on
that we decide ?


thanks and toi-toi :-)


Kris

Re: [VOTE] Apache CloudStack 4.10.0.0 RC5

Posted by Daan Hoogland <da...@shapeblue.com>.
It was an optimization that got merged and if someone is hindered we should back out and proceed, I’d say. 

On 21/06/17 16:00, "Wido den Hollander" <wi...@widodh.nl> wrote:

    
    > Op 20 juni 2017 om 21:35 schreef Kris Sterckx <kr...@nuagenetworks.net>:
    > 
    > 
    > I am sorry, I need to vote -1
    
    :(
    
    Anybody else have an idea on how to proceed? We keep having issues popping up with 4.10
    
    This should be fixable right for 4.10?
    
    We have so many work piling up for 4.11 right now, we should really get that going.
    
    Wido
    
    > 
    > 
    > https://github.com/apache/cloudstack/pull/2084  is causing password reset
    > issues for us (Nuage) (i know in native tests pass though...).
    > 
    > We are trying to provide a fix but the following modified VR script is
    > having severe issues :
    > 
    > https://github.com/apache/cloudstack/blob/master/systemvm/patches/debian/config/opt/cloud/bin/configure.py
    >  :
    > 
    > - import of  CsPassword  is missing
    > 
    > - line 910 : this invocation of the constructor is invalid  (ctor is  def
    > __init__(self, dbag)  )
    > 
    > Also i believe for SDN provided networks there is something overlooked as
    > we don't see the ip tables set for the VR to respond to password reset
    > requests.  We believe the script is not setting iptables_change = True in
    > case of networks for which the VR is not in the dataplane.
    > 
    > 
    > I suggest this PR is backed out for now it this is not an urgent add to the
    > 4.10 release, or we spend time at fixing it.
    > 
    > We definitely want to help in making that happen.
    > 
    > 
    > Thanks,
    > 
    > Kris
    > 
    > -- Nuage Networks
    > 
    
daan.hoogland@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue
  
 

> On 20 June 2017 at 07:45, Rajani Karuturi <ra...@apache.org> wrote:
    > 
    > > Hi All,
    > >
    > > I've created 4.10.0.0 release with the following artifacts up for a vote:
    > >
    > > Git Branch and Commit SH:
    > > https://github.com/apache/cloudstack/commit/058e34224c0555396c043c6473ac07
    > > ab3e8424cd
    > > Commit:058e34224c0555396c043c6473ac07ab3e8424cd
    > > Branch: https://github.com/apache/cloudstack/tree/4.10.0.0-RC20170620T1023
    > >
    > > Source release (checksums and signatures are available at the same
    > > location):
    > > https://dist.apache.org/repos/dist/dev/cloudstack/4.10.0.0/
    > >
    > > SystemVm Templates: http://download.cloudstack.org/systemvm/4.10/RC5/
    > >
    > > New PRs merged since RC4: #2150 and #2089
    > >
    > > PGP release keys (signed using CBB44821):
    > > https://dist.apache.org/repos/dist/release/cloudstack/KEYS
    > >
    > > Vote will be open for 72 hours.
    > >
    > > For sanity in tallying the vote, can PMC members please be sure to indicate
    > > "(binding)" with their vote?
    > >
    > > [ ] +1  approve
    > > [ ] +0  no opinion
    > > [ ] -1  disapprove (and reason why)
    > >
    > >
    > > Thanks,
    > > ~Rajani
    > > http://cloudplatform.accelerite.com/
    > >
    


Re: [VOTE] Apache CloudStack 4.10.0.0 RC5

Posted by Wido den Hollander <wi...@widodh.nl>.
> Op 20 juni 2017 om 21:35 schreef Kris Sterckx <kr...@nuagenetworks.net>:
> 
> 
> I am sorry, I need to vote -1

:(

Anybody else have an idea on how to proceed? We keep having issues popping up with 4.10

This should be fixable right for 4.10?

We have so many work piling up for 4.11 right now, we should really get that going.

Wido

> 
> 
> https://github.com/apache/cloudstack/pull/2084  is causing password reset
> issues for us (Nuage) (i know in native tests pass though...).
> 
> We are trying to provide a fix but the following modified VR script is
> having severe issues :
> 
> https://github.com/apache/cloudstack/blob/master/systemvm/patches/debian/config/opt/cloud/bin/configure.py
>  :
> 
> - import of  CsPassword  is missing
> 
> - line 910 : this invocation of the constructor is invalid  (ctor is  def
> __init__(self, dbag)  )
> 
> Also i believe for SDN provided networks there is something overlooked as
> we don't see the ip tables set for the VR to respond to password reset
> requests.  We believe the script is not setting iptables_change = True in
> case of networks for which the VR is not in the dataplane.
> 
> 
> I suggest this PR is backed out for now it this is not an urgent add to the
> 4.10 release, or we spend time at fixing it.
> 
> We definitely want to help in making that happen.
> 
> 
> Thanks,
> 
> Kris
> 
> -- Nuage Networks
> 
> On 20 June 2017 at 07:45, Rajani Karuturi <ra...@apache.org> wrote:
> 
> > Hi All,
> >
> > I've created 4.10.0.0 release with the following artifacts up for a vote:
> >
> > Git Branch and Commit SH:
> > https://github.com/apache/cloudstack/commit/058e34224c0555396c043c6473ac07
> > ab3e8424cd
> > Commit:058e34224c0555396c043c6473ac07ab3e8424cd
> > Branch: https://github.com/apache/cloudstack/tree/4.10.0.0-RC20170620T1023
> >
> > Source release (checksums and signatures are available at the same
> > location):
> > https://dist.apache.org/repos/dist/dev/cloudstack/4.10.0.0/
> >
> > SystemVm Templates: http://download.cloudstack.org/systemvm/4.10/RC5/
> >
> > New PRs merged since RC4: #2150 and #2089
> >
> > PGP release keys (signed using CBB44821):
> > https://dist.apache.org/repos/dist/release/cloudstack/KEYS
> >
> > Vote will be open for 72 hours.
> >
> > For sanity in tallying the vote, can PMC members please be sure to indicate
> > "(binding)" with their vote?
> >
> > [ ] +1  approve
> > [ ] +0  no opinion
> > [ ] -1  disapprove (and reason why)
> >
> >
> > Thanks,
> > ~Rajani
> > http://cloudplatform.accelerite.com/
> >

Re:RE: [VOTE] Apache CloudStack 4.10.0.0 RC5

Posted by Haijiao <18...@163.com>.
I saw Pierre-Luc Dion is working on a PR upon this,  but it's not included in 4.10 yet,  maybe 4.11


https://issues.apache.org/jira/browse/CLOUDSTACK-9839


在2017年06月21 10时24分, "Marty Godsey"<ma...@gonsource.com>写道:

Is XenServer 7.1 support being added in this release?

Regards,
Marty Godsey
Principal Engineer
nSource Solutions, LLC

-----Original Message-----
From: Kris Sterckx [mailto:kris.sterckx@nuagenetworks.net]
Sent: Tuesday, June 20, 2017 3:35 PM
To: dev@cloudstack.apache.org
Subject: Re: [VOTE] Apache CloudStack 4.10.0.0 RC5

I am sorry, I need to vote -1


https://github.com/apache/cloudstack/pull/2084  is causing password reset issues for us (Nuage) (i know in native tests pass though...).

We are trying to provide a fix but the following modified VR script is having severe issues :

https://github.com/apache/cloudstack/blob/master/systemvm/patches/debian/config/opt/cloud/bin/configure.py
:

- import of  CsPassword  is missing

- line 910 : this invocation of the constructor is invalid  (ctor is  def __init__(self, dbag)  )

Also i believe for SDN provided networks there is something overlooked as we don't see the ip tables set for the VR to respond to password reset requests.  We believe the script is not setting iptables_change = True in case of networks for which the VR is not in the dataplane.


I suggest this PR is backed out for now it this is not an urgent add to the
4.10 release, or we spend time at fixing it.

We definitely want to help in making that happen.


Thanks,

Kris

-- Nuage Networks

On 20 June 2017 at 07:45, Rajani Karuturi <ra...@apache.org> wrote:

> Hi All,
>
> I've created 4.10.0.0 release with the following artifacts up for a vote:
>
> Git Branch and Commit SH:
> https://github.com/apache/cloudstack/commit/058e34224c0555396c043c6473
> ac07
> ab3e8424cd
> Commit:058e34224c0555396c043c6473ac07ab3e8424cd
> Branch:
> https://github.com/apache/cloudstack/tree/4.10.0.0-RC20170620T1023
>
> Source release (checksums and signatures are available at the same
> location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.10.0.0/
>
> SystemVm Templates: http://download.cloudstack.org/systemvm/4.10/RC5/
>
> New PRs merged since RC4: #2150 and #2089
>
> PGP release keys (signed using CBB44821):
> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>
> Vote will be open for 72 hours.
>
> For sanity in tallying the vote, can PMC members please be sure to
> indicate "(binding)" with their vote?
>
> [ ] +1  approve
> [ ] +0  no opinion
> [ ] -1  disapprove (and reason why)
>
>
> Thanks,
> ~Rajani
> http://cloudplatform.accelerite.com/
>

RE: [VOTE] Apache CloudStack 4.10.0.0 RC5

Posted by Marty Godsey <ma...@gonsource.com>.
Is XenServer 7.1 support being added in this release?

Regards,
Marty Godsey
Principal Engineer
nSource Solutions, LLC

-----Original Message-----
From: Kris Sterckx [mailto:kris.sterckx@nuagenetworks.net] 
Sent: Tuesday, June 20, 2017 3:35 PM
To: dev@cloudstack.apache.org
Subject: Re: [VOTE] Apache CloudStack 4.10.0.0 RC5

I am sorry, I need to vote -1


https://github.com/apache/cloudstack/pull/2084  is causing password reset issues for us (Nuage) (i know in native tests pass though...).

We are trying to provide a fix but the following modified VR script is having severe issues :

https://github.com/apache/cloudstack/blob/master/systemvm/patches/debian/config/opt/cloud/bin/configure.py
 :

- import of  CsPassword  is missing

- line 910 : this invocation of the constructor is invalid  (ctor is  def __init__(self, dbag)  )

Also i believe for SDN provided networks there is something overlooked as we don't see the ip tables set for the VR to respond to password reset requests.  We believe the script is not setting iptables_change = True in case of networks for which the VR is not in the dataplane.


I suggest this PR is backed out for now it this is not an urgent add to the
4.10 release, or we spend time at fixing it.

We definitely want to help in making that happen.


Thanks,

Kris

-- Nuage Networks

On 20 June 2017 at 07:45, Rajani Karuturi <ra...@apache.org> wrote:

> Hi All,
>
> I've created 4.10.0.0 release with the following artifacts up for a vote:
>
> Git Branch and Commit SH:
> https://github.com/apache/cloudstack/commit/058e34224c0555396c043c6473
> ac07
> ab3e8424cd
> Commit:058e34224c0555396c043c6473ac07ab3e8424cd
> Branch: 
> https://github.com/apache/cloudstack/tree/4.10.0.0-RC20170620T1023
>
> Source release (checksums and signatures are available at the same
> location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.10.0.0/
>
> SystemVm Templates: http://download.cloudstack.org/systemvm/4.10/RC5/
>
> New PRs merged since RC4: #2150 and #2089
>
> PGP release keys (signed using CBB44821):
> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>
> Vote will be open for 72 hours.
>
> For sanity in tallying the vote, can PMC members please be sure to 
> indicate "(binding)" with their vote?
>
> [ ] +1  approve
> [ ] +0  no opinion
> [ ] -1  disapprove (and reason why)
>
>
> Thanks,
> ~Rajani
> http://cloudplatform.accelerite.com/
>

Re: [VOTE] Apache CloudStack 4.10.0.0 RC5

Posted by Kris Sterckx <kr...@nuagenetworks.net>.
I am sorry, I need to vote -1


https://github.com/apache/cloudstack/pull/2084  is causing password reset
issues for us (Nuage) (i know in native tests pass though...).

We are trying to provide a fix but the following modified VR script is
having severe issues :

https://github.com/apache/cloudstack/blob/master/systemvm/patches/debian/config/opt/cloud/bin/configure.py
 :

- import of  CsPassword  is missing

- line 910 : this invocation of the constructor is invalid  (ctor is  def
__init__(self, dbag)  )

Also i believe for SDN provided networks there is something overlooked as
we don't see the ip tables set for the VR to respond to password reset
requests.  We believe the script is not setting iptables_change = True in
case of networks for which the VR is not in the dataplane.


I suggest this PR is backed out for now it this is not an urgent add to the
4.10 release, or we spend time at fixing it.

We definitely want to help in making that happen.


Thanks,

Kris

-- Nuage Networks

On 20 June 2017 at 07:45, Rajani Karuturi <ra...@apache.org> wrote:

> Hi All,
>
> I've created 4.10.0.0 release with the following artifacts up for a vote:
>
> Git Branch and Commit SH:
> https://github.com/apache/cloudstack/commit/058e34224c0555396c043c6473ac07
> ab3e8424cd
> Commit:058e34224c0555396c043c6473ac07ab3e8424cd
> Branch: https://github.com/apache/cloudstack/tree/4.10.0.0-RC20170620T1023
>
> Source release (checksums and signatures are available at the same
> location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.10.0.0/
>
> SystemVm Templates: http://download.cloudstack.org/systemvm/4.10/RC5/
>
> New PRs merged since RC4: #2150 and #2089
>
> PGP release keys (signed using CBB44821):
> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>
> Vote will be open for 72 hours.
>
> For sanity in tallying the vote, can PMC members please be sure to indicate
> "(binding)" with their vote?
>
> [ ] +1  approve
> [ ] +0  no opinion
> [ ] -1  disapprove (and reason why)
>
>
> Thanks,
> ~Rajani
> http://cloudplatform.accelerite.com/
>