Merge lp://staging/~cpe-sa/orange-box/staging into lp://staging/orange-box

Proposed by Nicolas Thomas
Status: Needs review
Proposed branch: lp://staging/~cpe-sa/orange-box/staging
Merge into: lp://staging/orange-box
Diff against target: 1907 lines (+176/-779)
21 files modified
debian/changelog (+69/-11)
debian/control (+3/-1)
debian/install (+3/-0)
debian/postinst (+0/-310)
debian/postrm (+0/-23)
debian/preinst (+0/-136)
debian/source/format (+1/-1)
etc/apt/orange-box-mirror.list (+1/-1)
usr/bin/orange-box-add-node (+7/-4)
usr/bin/orange-box-add-physical-nodes (+7/-2)
usr/bin/orange-box-add-virtual-nodes (+0/-93)
usr/bin/orange-box-amt-monitor (+3/-1)
usr/bin/orange-box-apm-power-fix (+0/-38)
usr/bin/orange-box-bootstrap-juju (+0/-31)
usr/bin/orange-box-create-package-mirror (+14/-1)
usr/bin/orange-box-demo-cleanup (+3/-1)
usr/bin/orange-box-hacks (+32/-0)
usr/bin/orange-box-purge-everything (+1/-0)
usr/bin/orange-box-run (+21/-18)
usr/bin/orange-box-setup-nat (+0/-99)
usr/bin/orange-box-status-all-nodes (+11/-8)
To merge this branch: bzr merge lp://staging/~cpe-sa/orange-box/staging
Reviewer Review Type Date Requested Status
Orange Box Pending
Review via email: mp+265504@code.staging.launchpad.net

Description of the change

This is the test by SAs last revision of the OB code.

Wish to make it official

To post a comment you must log in.
Revision history for this message
Zhanglei Mao (zhanglei-mao) wrote :

Nicolas,

How can I do tests use your new merged package. I supposed that I can just follow OrangeBox V2 manual and it will use automatically. Is it right?

I install it step by step as manual before your merge time and stop at:
+ sudo -u maas virsh -c qemu+ssh://ubuntu@172.27.8.1/system list
error: failed to connect to the hypervisor
error: Cannot recv data: Permission denied (publickey).: Connection reset by pee

After I see your merge mail, I purge, reboot and rerun (apt-get install orange-box) again, it shows error as:
+ orange-box-add-virtual-nodes
+ virsh net-info default
error: failed to get network 'default'
error: Network not found: no network with matching name 'default'

I doubt both of them are caused by not using of new scripts. I will try it again from beginning again to verify it.

thanks,
Mao

552. By Brad Erhart <email address hidden>

* debian/control: Added qemu-system. Removed biosdevname
* debian/postinst: Disabled VM creation during install. Added network config for maas-br0
* debian/preinst: Removed biosdevname due to issues

Revision history for this message
Zhanglei Mao (zhanglei-mao) wrote :

hi Brad,

I test it again. It is still stop at the errors as yesterday. I follow the manual, only exception are area use local Hongkong and I use Wifi for internet.

# 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
+ sudo -u maas tee -a /var/lib/maas/.ssh/known_hosts
+ sudo -u maas ssh-keyscan 172.27.8.1
# 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
# 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
+ sudo -u maas virsh -c qemu+ssh://ubuntu@172.27.8.1/system list
error: failed to connect to the hypervisor
error: Cannot recv data: Permission denied (publickey).: Connection reset by peer
+ sed -i '/ maas@/d' /home/ubuntu/.ssh/authorized_keys
sed: can't read /home/ubuntu/.ssh/authorized_keys: No such file or directory
dpkg: error processing package orange-box (--configure):
 subprocess installed post-installation script returned error exit status 2
Setting up libwww-perl (6.05-2) ...
Setting up libxml-parser-perl (2.41-1build3) ...
Setting up libsoap-lite-perl (1.09-1) ...
Setting up libxmlrpc-lite-perl (0.717-1) ...
Processing triggers for libc-bin (2.19-0ubuntu6.6) ...
Processing triggers for initramfs-tools (0.103ubuntu4.2) ...
update-initramfs: Generating /boot/initrd.img-3.16.0-44-generic
Errors were encountered while processing:
 orange-box
E: Sub-process /usr/bin/dpkg returned an error code (1)
ubuntu@OrangeBox8:~$

thanks,
Mao

Revision history for this message
Nicolas Thomas (thomnico) wrote :

Brad this should not append with my last modification merged on the code???

Mao please send orange-box-add-virtual-nodes script ..

On Thu, Jul 23, 2015 at 1:06 PM, Zhanglei Mao
<email address hidden> wrote:
> hi Brad,
>
> I test it again. It is still stop at the errors as yesterday. I follow the manual, only exception are area use local Hongkong and I use Wifi for internet.
>
> # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
> + sudo -u maas tee -a /var/lib/maas/.ssh/known_hosts
> + sudo -u maas ssh-keyscan 172.27.8.1
> # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
> # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
> + sudo -u maas virsh -c qemu+ssh://ubuntu@172.27.8.1/system list
> error: failed to connect to the hypervisor
> error: Cannot recv data: Permission denied (publickey).: Connection reset by peer
> + sed -i '/ maas@/d' /home/ubuntu/.ssh/authorized_keys
> sed: can't read /home/ubuntu/.ssh/authorized_keys: No such file or directory
> dpkg: error processing package orange-box (--configure):
> subprocess installed post-installation script returned error exit status 2
> Setting up libwww-perl (6.05-2) ...
> Setting up libxml-parser-perl (2.41-1build3) ...
> Setting up libsoap-lite-perl (1.09-1) ...
> Setting up libxmlrpc-lite-perl (0.717-1) ...
> Processing triggers for libc-bin (2.19-0ubuntu6.6) ...
> Processing triggers for initramfs-tools (0.103ubuntu4.2) ...
> update-initramfs: Generating /boot/initrd.img-3.16.0-44-generic
> Errors were encountered while processing:
> orange-box
> E: Sub-process /usr/bin/dpkg returned an error code (1)
> ubuntu@OrangeBox8:~$
>
> thanks,
> Mao
> --
> https://code.launchpad.net/~cpe-sa/orange-box/staging/+merge/265504
> You proposed lp:~cpe-sa/orange-box/staging for merging.

--
Best Regards,
       Nicolas Thomas
http://insights.ubuntu.com/?p=889
EMEA Solution Architect Canonical
GPG FPR: D592 4185 F099 9031 6590 6292 492F C740 F03A 7EB9

Revision history for this message
Zhanglei Mao (zhanglei-mao) wrote :

My orange-box-add-virtual-nodes script was enclosed

On Thu, Jul 23, 2015 at 7:21 PM, Nicolas Thomas <
<email address hidden>> wrote:

> Brad this should not append with my last modification merged on the code???
>
> Mao please send orange-box-add-virtual-nodes script ..
>
> On Thu, Jul 23, 2015 at 1:06 PM, Zhanglei Mao
> <email address hidden> wrote:
> > hi Brad,
> >
> > I test it again. It is still stop at the errors as yesterday. I follow
> the manual, only exception are area use local Hongkong and I use Wifi for
> internet.
> >
> > # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
> > + sudo -u maas tee -a /var/lib/maas/.ssh/known_hosts
> > + sudo -u maas ssh-keyscan 172.27.8.1
> > # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
> > # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
> > + sudo -u maas virsh -c qemu+ssh://ubuntu@172.27.8.1/system list
> > error: failed to connect to the hypervisor
> > error: Cannot recv data: Permission denied (publickey).: Connection
> reset by peer
> > + sed -i '/ maas@/d' /home/ubuntu/.ssh/authorized_keys
> > sed: can't read /home/ubuntu/.ssh/authorized_keys: No such file or
> directory
> > dpkg: error processing package orange-box (--configure):
> > subprocess installed post-installation script returned error exit
> status 2
> > Setting up libwww-perl (6.05-2) ...
> > Setting up libxml-parser-perl (2.41-1build3) ...
> > Setting up libsoap-lite-perl (1.09-1) ...
> > Setting up libxmlrpc-lite-perl (0.717-1) ...
> > Processing triggers for libc-bin (2.19-0ubuntu6.6) ...
> > Processing triggers for initramfs-tools (0.103ubuntu4.2) ...
> > update-initramfs: Generating /boot/initrd.img-3.16.0-44-generic
> > Errors were encountered while processing:
> > orange-box
> > E: Sub-process /usr/bin/dpkg returned an error code (1)
> > ubuntu@OrangeBox8:~$
> >
> > thanks,
> > Mao
> > --
> > https://code.launchpad.net/~cpe-sa/orange-box/staging/+merge/265504
> > You proposed lp:~cpe-sa/orange-box/staging for merging.
>
>
>
> --
> Best Regards,
> Nicolas Thomas
> http://insights.ubuntu.com/?p=889
> EMEA Solution Architect Canonical
> GPG FPR: D592 4185 F099 9031 6590 6292 492F C740 F03A 7EB9
>
> https://code.launchpad.net/~cpe-sa/orange-box/staging/+merge/265504
> Your team Orange Box is requested to review the proposed merge of
> lp:~cpe-sa/orange-box/staging into lp:orange-box.
>

--
Zhanglei Mao
Solutions Architect, Sales and Business Development
Canonical Group Ltd.
<email address hidden>
+86-13625010929 (m)
+852-6700 6026 (m)
www.ubuntu.com
www.canonical.com

Revision history for this message
Nicolas Thomas (thomnico) wrote :
Download full text (3.2 KiB)

I see Brad change the all behavior instead of fixing bugs . can't
afford the time to play that game.. refer to Brad fro now on..
Changing behavior /doc and way of operating when seeing a bug is a
receipt for never releasing/delivering ..

On Thu, Jul 23, 2015 at 1:30 PM, Zhanglei Mao
<email address hidden> wrote:
> My orange-box-add-virtual-nodes script was enclosed
>
> On Thu, Jul 23, 2015 at 7:21 PM, Nicolas Thomas <
> <email address hidden>> wrote:
>
>> Brad this should not append with my last modification merged on the code???
>>
>> Mao please send orange-box-add-virtual-nodes script ..
>>
>> On Thu, Jul 23, 2015 at 1:06 PM, Zhanglei Mao
>> <email address hidden> wrote:
>> > hi Brad,
>> >
>> > I test it again. It is still stop at the errors as yesterday. I follow
>> the manual, only exception are area use local Hongkong and I use Wifi for
>> internet.
>> >
>> > # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
>> > + sudo -u maas tee -a /var/lib/maas/.ssh/known_hosts
>> > + sudo -u maas ssh-keyscan 172.27.8.1
>> > # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
>> > # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
>> > + sudo -u maas virsh -c qemu+ssh://ubuntu@172.27.8.1/system list
>> > error: failed to connect to the hypervisor
>> > error: Cannot recv data: Permission denied (publickey).: Connection
>> reset by peer
>> > + sed -i '/ maas@/d' /home/ubuntu/.ssh/authorized_keys
>> > sed: can't read /home/ubuntu/.ssh/authorized_keys: No such file or
>> directory
>> > dpkg: error processing package orange-box (--configure):
>> > subprocess installed post-installation script returned error exit
>> status 2
>> > Setting up libwww-perl (6.05-2) ...
>> > Setting up libxml-parser-perl (2.41-1build3) ...
>> > Setting up libsoap-lite-perl (1.09-1) ...
>> > Setting up libxmlrpc-lite-perl (0.717-1) ...
>> > Processing triggers for libc-bin (2.19-0ubuntu6.6) ...
>> > Processing triggers for initramfs-tools (0.103ubuntu4.2) ...
>> > update-initramfs: Generating /boot/initrd.img-3.16.0-44-generic
>> > Errors were encountered while processing:
>> > orange-box
>> > E: Sub-process /usr/bin/dpkg returned an error code (1)
>> > ubuntu@OrangeBox8:~$
>> >
>> > thanks,
>> > Mao
>> > --
>> > https://code.launchpad.net/~cpe-sa/orange-box/staging/+merge/265504
>> > You proposed lp:~cpe-sa/orange-box/staging for merging.
>>
>>
>>
>> --
>> Best Regards,
>> Nicolas Thomas
>> http://insights.ubuntu.com/?p=889
>> EMEA Solution Architect Canonical
>> GPG FPR: D592 4185 F099 9031 6590 6292 492F C740 F03A 7EB9
>>
>> https://code.launchpad.net/~cpe-sa/orange-box/staging/+merge/265504
>> Your team Orange Box is requested to review the proposed merge of
>> lp:~cpe-sa/orange-box/staging into lp:orange-box.
>>
>
>
>
> --
> Zhanglei Mao
> Solutions Architect, Sales and Business Development
> Canonical Group Ltd.
> <email address hidden>
> +86-13625010929 (m)
> +852-6700 6026 (m)
> www.ubuntu.com
> www.canonical.com
>
> https://code.launchpad.net/~cpe-sa/orange-box/staging/+merge/265504
> You proposed lp:~cpe-sa/orange-box/staging for merging.

--
Best Regards,
       Nicolas Thomas
http://insights.ubuntu.com/?p=889
E...

Read more...

Revision history for this message
vio0au0d (chei6ied-deactivatedaccount) wrote :
Download full text (3.8 KiB)

> I see Brad change the all behavior instead of fixing bugs . can't
> afford the time to play that game.. refer to Brad fro now on..
> Changing behavior /doc and way of operating when seeing a bug is a
> receipt for never releasing/delivering ..
I have no clue what that means Nicolas. It does not translate very well. Yes I did have to revert some changes that you made because they were breaking the install. I'd be more than happy to help find solutions, but I don't have time to not have a working package. Please review the revision history and you'll see that they were merely commented out and can be re-added once they've been tested by multiple people.

>
>
>
>
> On Thu, Jul 23, 2015 at 1:30 PM, Zhanglei Mao
> <email address hidden> wrote:
> > My orange-box-add-virtual-nodes script was enclosed
> >
> > On Thu, Jul 23, 2015 at 7:21 PM, Nicolas Thomas <
> > <email address hidden>> wrote:
> >
> >> Brad this should not append with my last modification merged on the code???
> >>
> >> Mao please send orange-box-add-virtual-nodes script ..
> >>
> >> On Thu, Jul 23, 2015 at 1:06 PM, Zhanglei Mao
> >> <email address hidden> wrote:
> >> > hi Brad,
> >> >
> >> > I test it again. It is still stop at the errors as yesterday. I follow
> >> the manual, only exception are area use local Hongkong and I use Wifi for
> >> internet.
> >> >
> >> > # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
> >> > + sudo -u maas tee -a /var/lib/maas/.ssh/known_hosts
> >> > + sudo -u maas ssh-keyscan 172.27.8.1
> >> > # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
> >> > # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
> >> > + sudo -u maas virsh -c qemu+ssh://ubuntu@172.27.8.1/system list
> >> > error: failed to connect to the hypervisor
> >> > error: Cannot recv data: Permission denied (publickey).: Connection
> >> reset by peer
> >> > + sed -i '/ maas@/d' /home/ubuntu/.ssh/authorized_keys
> >> > sed: can't read /home/ubuntu/.ssh/authorized_keys: No such file or
> >> directory
> >> > dpkg: error processing package orange-box (--configure):
> >> > subprocess installed post-installation script returned error exit
> >> status 2
> >> > Setting up libwww-perl (6.05-2) ...
> >> > Setting up libxml-parser-perl (2.41-1build3) ...
> >> > Setting up libsoap-lite-perl (1.09-1) ...
> >> > Setting up libxmlrpc-lite-perl (0.717-1) ...
> >> > Processing triggers for libc-bin (2.19-0ubuntu6.6) ...
> >> > Processing triggers for initramfs-tools (0.103ubuntu4.2) ...
> >> > update-initramfs: Generating /boot/initrd.img-3.16.0-44-generic
> >> > Errors were encountered while processing:
> >> > orange-box
> >> > E: Sub-process /usr/bin/dpkg returned an error code (1)
> >> > ubuntu@OrangeBox8:~$
> >> >
> >> > thanks,
> >> > Mao
> >> > --
> >> > https://code.launchpad.net/~cpe-sa/orange-box/staging/+merge/265504
> >> > You proposed lp:~cpe-sa/orange-box/staging for merging.
> >>
> >>
> >>
> >> --
> >> Best Regards,
> >> Nicolas Thomas
> >> http://insights.ubuntu.com/?p=889
> >> EMEA Solution Architect Canonical
> >> GPG FPR: D592 4185 F099 9031 6590 6292 492F C740 F03A 7EB9
> >>
> >> https://code.launchpad.net/~cpe-sa/orange-box/staging/+m...

Read more...

Revision history for this message
mahmoh (mahmoh) wrote :
Download full text (4.5 KiB)

On 07/23/2015 11:14 AM, Brad wrote:
>> I see Brad change the all behavior instead of fixing bugs . can't
>> afford the time to play that game.. refer to Brad fro now on..
>> Changing behavior /doc and way of operating when seeing a bug is a
>> receipt for never releasing/delivering ..
> I have no clue what that means Nicolas. It does not translate very well. Yes I did have to revert some changes that you made because they were breaking the install. I'd be more than happy to help find solutions, but I don't have time to not have a working package. Please review the revision history and you'll see that they were merely commented out and can be re-added once they've been tested by multiple people.

I think what Nicolas means is, for example, instead of documenting a
"reboot" in the manual to fix a problem in the package/scripts we should
be fixing/adjusting the package/scripts to solve the problem instead and
if possible.

I understand and appreciate your deadline of getting a working package
for tomorrow & next week but I also understand and agree with Nicolas'
frustration that it doesn't excuse us from properly fixing problems.

My suggestion at this point is get a working package for tomorrow's
deadline but schedule a mini-sprint ASAP so we can work this out. We
could use next week's team time to discuss this if anyone's interested?

>
>>
>>
>>
>>
>> On Thu, Jul 23, 2015 at 1:30 PM, Zhanglei Mao
>> <email address hidden> wrote:
>>> My orange-box-add-virtual-nodes script was enclosed
>>>
>>> On Thu, Jul 23, 2015 at 7:21 PM, Nicolas Thomas <
>>> <email address hidden>> wrote:
>>>
>>>> Brad this should not append with my last modification merged on the code???
>>>>
>>>> Mao please send orange-box-add-virtual-nodes script ..
>>>>
>>>> On Thu, Jul 23, 2015 at 1:06 PM, Zhanglei Mao
>>>> <email address hidden> wrote:
>>>>> hi Brad,
>>>>>
>>>>> I test it again. It is still stop at the errors as yesterday. I follow
>>>> the manual, only exception are area use local Hongkong and I use Wifi for
>>>> internet.
>>>>>
>>>>> # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
>>>>> + sudo -u maas tee -a /var/lib/maas/.ssh/known_hosts
>>>>> + sudo -u maas ssh-keyscan 172.27.8.1
>>>>> # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
>>>>> # 172.27.8.1 SSH-2.0-OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
>>>>> + sudo -u maas virsh -c qemu+ssh://ubuntu@172.27.8.1/system list
>>>>> error: failed to connect to the hypervisor
>>>>> error: Cannot recv data: Permission denied (publickey).: Connection
>>>> reset by peer
>>>>> + sed -i '/ maas@/d' /home/ubuntu/.ssh/authorized_keys
>>>>> sed: can't read /home/ubuntu/.ssh/authorized_keys: No such file or
>>>> directory
>>>>> dpkg: error processing package orange-box (--configure):
>>>>> subprocess installed post-installation script returned error exit
>>>> status 2
>>>>> Setting up libwww-perl (6.05-2) ...
>>>>> Setting up libxml-parser-perl (2.41-1build3) ...
>>>>> Setting up libsoap-lite-perl (1.09-1) ...
>>>>> Setting up libxmlrpc-lite-perl (0.717-1) ...
>>>>> Processing triggers for libc-bin (2.19-0ubuntu6.6) ...
>>>>> Processing triggers for initramfs-tools (0.103ubuntu4.2) ...
>>>>> update-initramfs: ...

Read more...

553. By Brad Erhart <email address hidden>

* usr/bin/orange-box-add-virtual-nodes: Removed as this is now part of orange-box-examples
* usr/bin/orange-box-bootstrap-juju: Removed as this is now part of orange-box-examples
* usr/bin/orange-box-status-all-nodes: Updated per new VM naming schema

Unmerged revisions

553. By Brad Erhart <email address hidden>

* usr/bin/orange-box-add-virtual-nodes: Removed as this is now part of orange-box-examples
* usr/bin/orange-box-bootstrap-juju: Removed as this is now part of orange-box-examples
* usr/bin/orange-box-status-all-nodes: Updated per new VM naming schema

552. By Brad Erhart <email address hidden>

* debian/control: Added qemu-system. Removed biosdevname
* debian/postinst: Disabled VM creation during install. Added network config for maas-br0
* debian/preinst: Removed biosdevname due to issues

551. By Brad Erhart <email address hidden>

More comments

550. By Brad Erhart <email address hidden>

Merged changes from newnet

549. By Brad Erhart <email address hidden>

Added additional comments

modified:
  usr/bin/orange-box-add-virtual-nodes

548. By Brad Erhart <email address hidden>

Cleanup and comments

547. By Nicolas Thomas

fix default to em1 for internal_interface

546. By Nicolas Thomas

Fix zone issue
Fix ssh-keyscan for maas asking user intervention

545. By Nicolas Thomas

updates on ssh maas to ubuntu automation

modified:
  debian/changelog
  usr/bin/orange-box-add-virtual-nodes

544. By Nicolas Thomas

releasing 2.9~ppa2

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches