[v2] Deploy script fixes for importer

[Patchew-devel] [PATCH v2 0/2] Deploy script fixes for importer

Fam Zheng posted 2 patches 6 years, 1 month ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/patchew-ci tags/patchew/20180321031858.22975-1-famz@redhat.com

scripts/dockerfiles/importer.docker       |  6 +++++-
scripts/playbooks/deploy-importers.yml    |  1 +
scripts/playbooks/tasks/docker-deploy.yml | 12 +++++++++---
3 files changed, 15 insertions(+), 4 deletions(-)

Expand all Fold all

[Patchew-devel] [PATCH v2 0/2] Deploy script fixes for importer

Posted by Fam Zheng 6 years, 1 month ago


Fam Zheng (2):
  deploy: Allow empty importer_identity
  deploy: Install Python3 pip

 scripts/dockerfiles/importer.docker       |  6 +++++-
 scripts/playbooks/deploy-importers.yml    |  1 +
 scripts/playbooks/tasks/docker-deploy.yml | 12 +++++++++---
 3 files changed, 15 insertions(+), 4 deletions(-)

-- 
2.14.3

_______________________________________________
Patchew-devel mailing list
Patchew-devel@redhat.com
https://www.redhat.com/mailman/listinfo/patchew-devel

Re: [Patchew-devel] [PATCH v2 0/2] Deploy script fixes for importer

Posted by Paolo Bonzini 6 years, 1 month ago

On 21/03/2018 04:18, Fam Zheng wrote:
> Fam Zheng (2):
>   deploy: Allow empty importer_identity
>   deploy: Install Python3 pip
> 
>  scripts/dockerfiles/importer.docker       |  6 +++++-
>  scripts/playbooks/deploy-importers.yml    |  1 +
>  scripts/playbooks/tasks/docker-deploy.yml | 12 +++++++++---
>  3 files changed, 15 insertions(+), 4 deletions(-)
> 

LGTM, thanks for the fixes!

Paolo

_______________________________________________
Patchew-devel mailing list
Patchew-devel@redhat.com
https://www.redhat.com/mailman/listinfo/patchew-devel

Re: [Patchew-devel] [PATCH v2 0/2] Deploy script fixes for importer

Posted by Fam Zheng 6 years, 1 month ago

On Wed, 03/21 06:50, Paolo Bonzini wrote:
> On 21/03/2018 04:18, Fam Zheng wrote:
> > Fam Zheng (2):
> >   deploy: Allow empty importer_identity
> >   deploy: Install Python3 pip
> > 
> >  scripts/dockerfiles/importer.docker       |  6 +++++-
> >  scripts/playbooks/deploy-importers.yml    |  1 +
> >  scripts/playbooks/tasks/docker-deploy.yml | 12 +++++++++---
> >  3 files changed, 15 insertions(+), 4 deletions(-)
> > 
> 
> LGTM, thanks for the fixes!

Picking up "LGTM" tags should be part of a github plugin of patchew. (joking)

Thanks, pushed.

Fam

_______________________________________________
Patchew-devel mailing list
Patchew-devel@redhat.com
https://www.redhat.com/mailman/listinfo/patchew-devel