Re: [Qemu-devel] [PULL v4 00/38] Test and build patches

2017-09-15 Thread Fam Zheng
On Fri, 09/15 11:47, Philippe Mathieu-Daudé wrote: > why not generate 1 key in > tests/vm/ (or clever ~/.cache/qemu-vm/ already used by those scripts) once > when the make vm-test rule is called, that would be 1 key per repository > clone (or 1 per user using ~/.cache). Like I explained

Re: [Qemu-devel] [PULL v4 00/38] Test and build patches

2017-09-15 Thread Philippe Mathieu-Daudé
Hi Daniel, On 09/15/2017 08:40 AM, Daniel P. Berrange wrote: On Fri, Sep 15, 2017 at 11:55:44AM +0100, Peter Maydell wrote: [...] So, before I commit an ssh private key to our git repo, can you explain why it's ok that this is public? The commit message for the relevant patch doesn't really

Re: [Qemu-devel] [PULL v4 00/38] Test and build patches

2017-09-15 Thread Fam Zheng
On Fri, 09/15 13:31, Daniel P. Berrange wrote: > On Fri, Sep 15, 2017 at 08:21:53PM +0800, Fam Zheng wrote: > > On Fri, 09/15 12:40, Daniel P. Berrange wrote: > > > On Fri, Sep 15, 2017 at 11:55:44AM +0100, Peter Maydell wrote: > > > > On 15 September 2017 at 10:02, Fam Zheng

Re: [Qemu-devel] [PULL v4 00/38] Test and build patches

2017-09-15 Thread Daniel P. Berrange
On Fri, Sep 15, 2017 at 08:21:53PM +0800, Fam Zheng wrote: > On Fri, 09/15 12:40, Daniel P. Berrange wrote: > > On Fri, Sep 15, 2017 at 11:55:44AM +0100, Peter Maydell wrote: > > > On 15 September 2017 at 10:02, Fam Zheng wrote: > > > > The following changes since commit > > > >

Re: [Qemu-devel] [PULL v4 00/38] Test and build patches

2017-09-15 Thread Fam Zheng
On Fri, 09/15 12:40, Daniel P. Berrange wrote: > On Fri, Sep 15, 2017 at 11:55:44AM +0100, Peter Maydell wrote: > > On 15 September 2017 at 10:02, Fam Zheng wrote: > > > The following changes since commit > > > 04ef33052c205170c92df21ca0b4be4f3b102188: > > > > > > tcg/tci: do

Re: [Qemu-devel] [PULL v4 00/38] Test and build patches

2017-09-15 Thread Daniel P. Berrange
On Fri, Sep 15, 2017 at 01:03:54PM +0100, Peter Maydell wrote: > On 15 September 2017 at 12:40, Daniel P. Berrange wrote: > > IIUC, the public part of the key gets exposed to the guest images via > > cloud-init metadata. During boot the guest read this metadata and add > >

Re: [Qemu-devel] [PULL v4 00/38] Test and build patches

2017-09-15 Thread Peter Maydell
On 15 September 2017 at 12:40, Daniel P. Berrange wrote: > IIUC, the public part of the key gets exposed to the guest images via > cloud-init metadata. During boot the guest read this metadata and add > the public key to authorized_keys. The private key is used by the test >

Re: [Qemu-devel] [PULL v4 00/38] Test and build patches

2017-09-15 Thread Daniel P. Berrange
On Fri, Sep 15, 2017 at 11:55:44AM +0100, Peter Maydell wrote: > On 15 September 2017 at 10:02, Fam Zheng wrote: > > The following changes since commit 04ef33052c205170c92df21ca0b4be4f3b102188: > > > > tcg/tci: do not use ldst label (never implemented) (2017-09-11 19:24:05 > >

Re: [Qemu-devel] [PULL v4 00/38] Test and build patches

2017-09-15 Thread Fam Zheng
On Fri, 09/15 11:55, Peter Maydell wrote: > On 15 September 2017 at 10:02, Fam Zheng wrote: > > The following changes since commit 04ef33052c205170c92df21ca0b4be4f3b102188: > > > > tcg/tci: do not use ldst label (never implemented) (2017-09-11 19:24:05 > > +0100) > > > > are

Re: [Qemu-devel] [PULL v4 00/38] Test and build patches

2017-09-15 Thread Peter Maydell
On 15 September 2017 at 10:02, Fam Zheng wrote: > The following changes since commit 04ef33052c205170c92df21ca0b4be4f3b102188: > > tcg/tci: do not use ldst label (never implemented) (2017-09-11 19:24:05 > +0100) > > are available in the git repository at: > >

[Qemu-devel] [PULL v4 00/38] Test and build patches

2017-09-15 Thread Fam Zheng
The following changes since commit 04ef33052c205170c92df21ca0b4be4f3b102188: tcg/tci: do not use ldst label (never implemented) (2017-09-11 19:24:05 +0100) are available in the git repository at: git://github.com/famz/qemu.git tags/test-and-build-pull-request for you to fetch changes up to