Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature: build deb package for pouch #674

Merged
merged 1 commit into from
Feb 1, 2018

Conversation

zzchun
Copy link
Contributor

@zzchun zzchun commented Feb 1, 2018

Signed-off-by: Zhanchun Zhang [email protected]

1.Describe what this PR did
This pr added a way to build rpm package

2.Does this pull request fix one issue?
fixes part of #67

3.Describe how you did it

  1. Build pouch:deb image
    cd pouch
    docker build -t pouch:deb -f hack/package/deb/ubuntu-xenial/Dockerfile.x86_64 .
  2. Mount a directory which contains gpg keys, eg
    $ tree /root/deb/
    deb
    ├── keys
    │   ├── gpg
    │   └── secretkey
  3. run pouch:deb to generate deb package
    docker run -it -e VERSION="0.1.0" -v /root/deb/:/root/deb pouch:deb bash -c hack/package/deb/build.sh
  4. In this example, rpm package will be output in '/root/deb/package/' directory

4.Describe how to verify it
NONE

5.Special notes for reviews
NONE

@codecov-io
Copy link

codecov-io commented Feb 1, 2018

Codecov Report

Merging #674 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #674   +/-   ##
=======================================
  Coverage   10.16%   10.16%           
=======================================
  Files          71       71           
  Lines        3425     3425           
=======================================
  Hits          348      348           
  Misses       3038     3038           
  Partials       39       39

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 62aada5...03645b2. Read the comment docs.

Signed-off-by: Zhanchun Zhang <[email protected]>
@Letty5411
Copy link
Contributor

LGTM

@pouchrobot pouchrobot added the LGTM one maintainer or community participant agrees to merge the pull reuqest. label Feb 1, 2018
@Letty5411 Letty5411 merged commit ebd8827 into AliyunContainerService:master Feb 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature LGTM one maintainer or community participant agrees to merge the pull reuqest. size/XL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants