Thanks for the PR Jerry,
Can you try Rohit's way. He seems to provide a way, making this not a
blocker.

On Mon, Jan 4, 2021 at 7:22 AM li jerry <div...@hotmail.com> wrote:

> I'm creating PR to fix this.
>
> I think we should block, because it will cause the RBD primary storage to
> be unable to be added.
>
> -----邮件原件-----
> 发件人: Daan Hoogland <daan.hoogl...@gmail.com>
> 发送时间: 2021年1月4日 14:57
> 收件人: users <us...@cloudstack.apache.org>
> 抄送: dev <dev@cloudstack.apache.org>
> 主题: Re: [VOTE] Apache Cloudstack 4.15.0.0 and UI [RC3]
>
> looks good Jerry,
> Are you making a PR? It seems to me that this would not be a blocker and
> should go in future releases. Please argue against me if you disagree.
>
> On Mon, Jan 4, 2021 at 6:48 AM li jerry <div...@hotmail.com> wrote:
>
> > - Is this a setup that does work with a prior version?
> > - Did you fresh install or upgrade?
> >
> > No, This is a new deployment, there are no upgrades
> >
> > I have changed two methods. At present, RBD storage is running
> >
> >
> >
> /cloud-plugin-hypervisor-kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtStoragePoolXMLParser.java
> > //                String _xmlPort = Integer.parseInt(getAttrValue("host",
> > "port", source));
> >                 int port = 0;
> >                 String _xmlPort = getAttrValue("host", "port", source);
> >                 if ( ! _xmlPort.isEmpty()) {
> >                 port = Integer.parseInt(_xmlPort);
> >                 }
> >
> >
> >
> >
> >
> /cloud-plugin-hypervisor-kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtDomainXMLParser.java
> > //                    int port = Integer.parseInt(getAttrValue("host",
> > "port", disk));
> >                     int port = 0;
> >                     String _xmlPort = getAttrValue("host", "port", disk);
> >                     if ( ! _xmlPort.isEmpty()) {
> >                         port = Integer.parseInt(_xmlPort);
> >                     }
> >
> >
> > -Jerry
> >
> > 发件人: Daan Hoogland<mailto:daan.hoogl...@gmail.com>
> > 发送时间: 2021年1月4日 14:41
> > 收件人: users<mailto:us...@cloudstack.apache.org>
> > 抄送: dev<mailto:dev@cloudstack.apache.org>
> > 主题: Re: [VOTE] Apache Cloudstack 4.15.0.0 and UI [RC3]
> >
> > Jerry,
> > - Is this a setup that does work with a prior version?
> > - Did you fresh install or upgrade?
> > @list is there any RDB user that can advise on the missing
> > configuration causing the NumberFormatException, please?
> >
> >
> > On Sun, Jan 3, 2021 at 1:25 PM li jerry <div...@hotmail.com> wrote:
> >
> > > Happy New Year to all.
> > >
> > >
> > > Sorry, I can't add RBD primary storage when I deploy with 4.15 RC3
> > >
> > > CloudStack: 4.15 RC3
> > >
> > > OS : Ubuntu 20.04.01
> > >
> > > DB: MYSQL 8.0.22
> > >
> > > CEPH: 15.2.8
> > >
> > > libvirt:6.0.0
> > > hypervisor: QEMU 4.2.1
> > >
> > >
> > > Add main memory to report the following error:
> > >
> > > 2021-01-03 13:15:32,605 DEBUG [cloud.agent.Agent]
> > > (agentRequest-Handler-2:null) (logid:0fd66f6e) Seq
> 1-2968153629413867529:
> > > { Ans: , MgmtId: 182719176, via: 1, Ver: v1, Flags: 10,
> > >
> > [{"com.cloud.agent.api.Answer":{"result":"true","details":"success","w
> > ait":"0"}}]
> > > }
> > > 2021-01-03 13:15:32,631 DEBUG [cloud.agent.Agent]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) Request:Seq
> > > 1-2968153629413867530:  { Cmd , MgmtId: 182719176, via: 1, Ver: v1,
> > Flags:
> > > 100011,
> > >
> > [{"com.cloud.agent.api.ModifyStoragePoolCommand":{"add":"true","pool":
> > {"id":"3","uuid":"d9b976cb-bcaf-320a-94e6-b337e65dd4f5","host":"10.100
> > .250.14","path":"rbd","userInfo":"hyperx:AQAywfFf8jCiIxAAbnDBjX1QQAO9S
> > j22kUBh7g==","port":"0","type":"RBD"},"localPath":"/mnt//5472031c-3588
> > -3e2c-b106-74c8d9f4ca83","wait":"0"}}]
> > > }
> > > 2021-01-03 13:15:32,631 DEBUG [cloud.agent.Agent]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) Processing command:
> > > com.cloud.agent.api.ModifyStoragePoolCommand
> > > 2021-01-03 13:15:32,632 INFO  [kvm.storage.LibvirtStorageAdaptor]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) Attempting to create
> > storage
> > > pool d9b976cb-bcaf-320a-94e6-b337e65dd4f5 (RBD) in libvirt
> > > 2021-01-03 13:15:32,632 DEBUG [kvm.resource.LibvirtConnection]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) Looking for libvirtd
> > > connection at: qemu:///system
> > > 2021-01-03 13:15:32,654 WARN  [kvm.storage.LibvirtStorageAdaptor]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) Storage pool
> > > d9b976cb-bcaf-320a-94e6-b337e65dd4f5 was not found running in libvirt.
> > Need
> > > to create it.
> > > 2021-01-03 13:15:32,655 INFO  [kvm.storage.LibvirtStorageAdaptor]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) Didn't find an
> > > existing storage pool d9b976cb-bcaf-320a-94e6-b337e65dd4f5 by UUID,
> > > checking for pools with duplicate paths
> > > 2021-01-03 13:15:32,657 DEBUG [kvm.storage.LibvirtStorageAdaptor]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) Checking path of
> > > existing pool root against pool we want to create
> > > 2021-01-03 13:15:32,667 DEBUG [kvm.storage.LibvirtStorageAdaptor]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) Checking path of
> > > existing pool 1739fc06-2a31-4af1-b8cb-871a27989f37 against pool we
> > > want to create
> > > 2021-01-03 13:15:32,672 DEBUG [kvm.storage.LibvirtStorageAdaptor]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) Attempting to create
> > storage
> > > pool d9b976cb-bcaf-320a-94e6-b337e65dd4f5
> > > 2021-01-03 13:15:32,686 DEBUG [kvm.storage.LibvirtStorageAdaptor]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) <secret ephemeral='no'
> > > private='no'>
> > > <uuid>d9b976cb-bcaf-320a-94e6-b337e65dd4f5</uuid>
> > > <usage type='ceph'>
> > > <name>hyperx@10.100.250.14:0/rbd</name>
> > > </usage>
> > > </secret>
> > >
> > > 2021-01-03 13:15:32,706 DEBUG [kvm.storage.LibvirtStorageAdaptor]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) <pool type='rbd'>
> > > <name>d9b976cb-bcaf-320a-94e6-b337e65dd4f5</name>
> > > <uuid>d9b976cb-bcaf-320a-94e6-b337e65dd4f5</uuid>
> > > <source>
> > > <host name='10.100.250.14'/>
> > > <name>rbd</name>
> > > <auth username='hyperx' type='ceph'> <secret
> > > uuid='d9b976cb-bcaf-320a-94e6-b337e65dd4f5'/>
> > > </auth>
> > > </source>
> > > </pool>
> > >
> > > 2021-01-03 13:15:32,759 INFO  [kvm.storage.LibvirtStorageAdaptor]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) Trying to fetch
> > > storage
> > pool
> > > d9b976cb-bcaf-320a-94e6-b337e65dd4f5 from libvirt
> > > 2021-01-03 13:15:32,760 DEBUG [kvm.resource.LibvirtConnection]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) Looking for libvirtd
> > > connection at: qemu:///system
> > > 2021-01-03 13:15:32,769 WARN  [cloud.agent.Agent]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) Caught:
> > > java.lang.NumberFormatException: For input string: ""
> > >         at
> > >
> > java.base/java.lang.NumberFormatException.forInputString(NumberFormatE
> > xception.java:65)
> > >         at java.base/java.lang.Integer.parseInt(Integer.java:662)
> > >         at java.base/java.lang.Integer.parseInt(Integer.java:770)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.resource.LibvirtStoragePoolXMLParser.parseSto
> > ragePoolXML(LibvirtStoragePoolXMLParser.java:58)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.storage.LibvirtStorageAdaptor.getStoragePoolD
> > ef(LibvirtStorageAdaptor.java:413)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.storage.LibvirtStorageAdaptor.getStoragePool(
> > LibvirtStorageAdaptor.java:439)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.storage.LibvirtStorageAdaptor.getStoragePool(
> > LibvirtStorageAdaptor.java:424)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.storage.LibvirtStorageAdaptor.createStoragePo
> > ol(LibvirtStorageAdaptor.java:654)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.createStoragePo
> > ol(KVMStoragePoolManager.java:329)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.createStoragePo
> > ol(KVMStoragePoolManager.java:323)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.resource.wrapper.LibvirtModifyStoragePoolComm
> > andWrapper.execute(LibvirtModifyStoragePoolCommandWrapper.java:42)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.resource.wrapper.LibvirtModifyStoragePoolComm
> > andWrapper.execute(LibvirtModifyStoragePoolCommandWrapper.java:35)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.resource.wrapper.LibvirtRequestWrapper.execut
> > e(LibvirtRequestWrapper.java:78)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.executeRequ
> > est(LibvirtComputingResource.java:1643)
> > >         at com.cloud.agent.Agent.processRequest(Agent.java:661)
> > >         at
> > > com.cloud.agent.Agent$AgentRequestHandler.doTask(Agent.java:1079)
> > >         at com.cloud.utils.nio.Task.call(Task.java:83)
> > >         at com.cloud.utils.nio.Task.call(Task.java:29)
> > >         at
> > > java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
> > >         at
> > >
> > java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPool
> > Executor.java:1128)
> > >         at
> > >
> > java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoo
> > lExecutor.java:628)
> > >         at java.base/java.lang.Thread.run(Thread.java:834)
> > > 2021-01-03 13:15:32,778 DEBUG [cloud.agent.Agent]
> > > (agentRequest-Handler-3:null) (logid:0fd66f6e) Seq
> 1-2968153629413867530:
> > > { Ans: , MgmtId: 182719176, via: 1, Ver: v1, Flags: 10,
> > >
> >
> [{"com.cloud.agent.api.Answer":{"result":"false","details":"java.lang.NumberFormatException:
> > > For input string: ""
> > >         at
> > >
> > java.base/java.lang.NumberFormatException.forInputString(NumberFormatE
> > xception.java:65)
> > >         at java.base/java.lang.Integer.parseInt(Integer.java:662)
> > >         at java.base/java.lang.Integer.parseInt(Integer.java:770)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.resource.LibvirtStoragePoolXMLParser.parseSto
> > ragePoolXML(LibvirtStoragePoolXMLParser.java:58)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.storage.LibvirtStorageAdaptor.getStoragePoolD
> > ef(LibvirtStorageAdaptor.java:413)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.storage.LibvirtStorageAdaptor.getStoragePool(
> > LibvirtStorageAdaptor.java:439)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.storage.LibvirtStorageAdaptor.getStoragePool(
> > LibvirtStorageAdaptor.java:424)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.storage.LibvirtStorageAdaptor.createStoragePo
> > ol(LibvirtStorageAdaptor.java:654)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.createStoragePo
> > ol(KVMStoragePoolManager.java:329)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.createStoragePo
> > ol(KVMStoragePoolManager.java:323)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.resource.wrapper.LibvirtModifyStoragePoolComm
> > andWrapper.execute(LibvirtModifyStoragePoolCommandWrapper.java:42)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.resource.wrapper.LibvirtModifyStoragePoolComm
> > andWrapper.execute(LibvirtModifyStoragePoolCommandWrapper.java:35)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.resource.wrapper.LibvirtRequestWrapper.execut
> > e(LibvirtRequestWrapper.java:78)
> > >         at
> > >
> > com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.executeRequ
> > est(LibvirtComputingResource.java:1643)
> > >         at com.cloud.agent.Agent.processRequest(Agent.java:661)
> > >         at
> > > com.cloud.agent.Agent$AgentRequestHandler.doTask(Agent.java:1079)
> > >         at com.cloud.utils.nio.Task.call(Task.java:83)
> > >         at com.cloud.utils.nio.Task.call(Task.java:29)
> > >         at
> > > java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
> > >         at
> > >
> > java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPool
> > Executor.java:1128)
> > >         at
> > >
> > java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoo
> > lExecutor.java:628)
> > >         at java.base/java.lang.Thread.run(Thread.java:834)
> > > ","wait":"0"}}] }
> > > 2021-01-03 13:15:43,241 DEBUG
> > > [kvm.resource.LibvirtComputingResource]
> > > (UgentTask-2:null) (logid:) Executing:
> > > /usr/share/cloudstack-common/scripts/vm/network/security_group.py
> > > get_rule_logs_for_vms
> > >
> > >
> > >
> > >
> > >
> > > After the failure, I checked the pool through virsh and found that
> > > it had been added successfully.
> > > Here is the virsh output information:
> > >
> > >
> > > root@noded:/etc/cloudstack/agent# virsh pool-list
> > >  Name                                   State    Autostart
> > > ------------------------------------------------------------
> > >  1739fc06-2a31-4af1-b8cb-871a27989f37   active   no
> > >  d9b976cb-bcaf-320a-94e6-b337e65dd4f5   active   no
> > >  root                                   active   yes
> > >
> > > root@noded:/etc/cloudstack/agent# virsh pool-dumpxml
> > > d9b976cb-bcaf-320a-94e6-b337e65dd4f5
> > > <pool type='rbd'>
> > >   <name>d9b976cb-bcaf-320a-94e6-b337e65dd4f5</name>
> > >   <uuid>d9b976cb-bcaf-320a-94e6-b337e65dd4f5</uuid>
> > >   <capacity unit='bytes'>12122373201920</capacity>
> > >   <allocation unit='bytes'>912457728</allocation>
> > >   <available unit='bytes'>11998204379136</available>
> > >   <source>
> > >     <host name='10.100.250.14'/>
> > >     <name>rbd</name>
> > >     <auth type='ceph' username='hyperx'>
> > >       <secret uuid='d9b976cb-bcaf-320a-94e6-b337e65dd4f5'/>
> > >     </auth>
> > >   </source>
> > > </pool>
> > >
> > >
> > >
> > >
> > >
> > > -----邮件原件-----
> > > 发件人: Daan Hoogland <daan.hoogl...@gmail.com>
> > > 发送时间: 2021年1月1日 16:55
> > > 收件人: users <us...@cloudstack.apache.org>
> > > 抄送: dev <dev@cloudstack.apache.org>
> > > 主题: Re: [VOTE] Apache Cloudstack 4.15.0.0 and UI [RC3]
> > >
> > > Happy New Year to all, I think we have a release but I'll wait to
> > > tally votes until Monday. Enjoy your weekend and the coming year.
> > >
> > > On Thu, 31 Dec 2020, 15:10 Boris Stoyanov,
> > > <boris.stoya...@shapeblue.com
> > >
> > > wrote:
> > >
> > > > +1 (binding)
> > > >
> > > > In shared effort with Vladimir Petrov, we've done upgrade testing
> > > > from latest of:
> > > > 4.11
> > > > 4.13
> > > > 4.14
> > > >
> > > > Also did a basic lifecycle operations of:
> > > > VMs, Networks, Storage, Infra(pod, cluster, zone, hosts).
> > > >
> > > > And we couldn't find any stopping issues with this RC.
> > > >
> > > > Thanks,
> > > > Bobby.
> > > >
> > > > On 24.12.20, 5:14, "Rohit Yadav" <rohit.ya...@shapeblue.com> wrote:
> > > >
> > > >
> > > >     All,
> > > >
> > > >     Here are the convenience packages build from 4.15.0.0-RC3 if
> > > > you don't want to build CloudStack from the source artifacts:
> > > >
> > > >     Packages: (Debian, CentOS7, and CentOS8)
> > > >     http://download.cloudstack.org/testing/4.15.0.0-rc3/
> > > >
> > > >     4.15 systemvmtemplate:
> > > >     http://download.cloudstack.org/systemvm/4.15/
> > > >
> > > >     Build from the master branch of
> > > > https://github.com/apache/cloudstack-documentation (if/after
> > > > voting passes, we'll update and publish the docs):
> > > >     http://docs.cloudstack.apache.org/en/master/upgrading/
> > > >
> > > >     Additional notes:
> > > >       *   The new UI is bundled within the cloudstack-management
> > package
> > > > and is shipped as the default UI served at <host:8080>/client, old
> > > > UI will be served via <host:8080>/client/legacy. Most users don't
> > > > need to do any separate installation or perform an installation step.
> > > >       *   We've added support for CentOS8 with 4.15 but CentOS8 will
> > EOL
> > > > in Dec 2021 (https://wiki.centos.org/About/Product).
> > > >
> > > >
> > > >     Regards.
> > > >
> > > >     ________________________________
> > > >     From: Daan Hoogland <daan.hoogl...@gmail.com>
> > > >     Sent: Wednesday, December 23, 2020 23:13
> > > >     To: users <us...@cloudstack.apache.org>; dev <
> > > > dev@cloudstack.apache.org>
> > > >     Subject: [VOTE] Apache Cloudstack 4.15.0.0 and UI [RC3]
> > > >
> > > >     LS,
> > > >     After fixing another few blockers, we have an RC3, The changes
> > > > (other than
> > > >     bundling) are mostly interesting for those working with
> > > > templates and on
> > > >     vmware.
> > > >
> > > >     We are voting for the new UI and the main code.
> > > >
> > > >     The candidate release branch is 4.15.0.0-RC20201223T1632. The
> > > > UI is still
> > > >     separate but as agreed upon before this will be merged in
> > > > coming releases,
> > > >     at least from a version management point of view.
> > > >     I've created a 4.15.0.0 release candidate, with the following
> > > > artifacts up
> > > >     for a vote:Git Branches:
> > > >     main code:
> > > >
> > > >
> > >
> > https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=
> > refs/heads/4.15.0.0-RC20201223T1632
> > > >     <
> > > > https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlo
> > > > g;h=
> > > > refs/heads/4.15.0.0-RC20201214T1124
> > > > >
> > > >     ui code:
> > > >     <
> > > > https://git-wip-us.apache.org/repos/asf?p=cloudstack-primate.git;a
> > > > =sho
> > > > rtlog;h=refs/tags/1.0
> > > > >
> > > >
> > > >
> > >
> > https://git-wip-us.apache.org/repos/asf?p=cloudstack-primate.git;a=tag
> > ;h=refs/tags/1.0
> > > >     and Commit SH:
> > > >     main code: 01b3e361c7bb81fd1ea822faddd6594e52bb00c1
> > > >     ui code: 0593302dd53ac3203d3ab43b62d890605910f3e1
> > > >
> > > >     Source release (checksums and signatures are available at the
> same
> > > >     location):
> > > >     https://dist.apache.org/repos/dist/dev/cloudstack/4.15.0.0/
> (rev.
> > > > 45059)
> > > >     PGP release keys (signed using 7975062401944786):
> > > >     https://dist.apache.org/repos/dist/release/cloudstack/KEYSVote
> > > > will be open
> > > >     for (at least) 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)
> > > >
> > > >     I will work with community members to provide convenience
> > > > packaging over
> > > >     the next few days.
> > > >     The documentation repo will be updated as we move along.
> > > >
> > > >
> > > >
> > > >     --
> > > >     Daan
> > > >
> > > >     rohit.ya...@shapeblue.com
> > > >     www.shapeblue.com<http://www.shapeblue.com>
> > > >     3 London Bridge Street,  3rd floor, News Building, London  SE1
> > 9SGUK
> > > >     @shapeblue
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > boris.stoya...@shapeblue.com
> > > > www.shapeblue.com<http://www.shapeblue.com>
> > > > 3 London Bridge Street,  3rd floor, News Building, London  SE1
> > > > 9SGUK @shapeblue
> > > >
> > > >
> > > >
> > > >
> > >
> >
> >
> > --
> > Daan
> >
> >
>
> --
> Daan
>


-- 
Daan

Reply via email to