8000 Plugin integration test by dmcgowan · Pull Request #2085 · distribution/distribution · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Plugin integration test #2085

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

Merged
merged 4 commits into from
Jan 10, 2017
Merged

Conversation

dmcgowan
Copy link
Collaborator
  • Add plugin push and pull tests. Tests that images pushed as plugins cannot be pulled as images and vice versa.
  • Additional base image used for creating plugin rootfs
  • Updates login to allow supply the email flag in less than docker 1.11. Needed to support 1.13 which removes the email flag.

@codecov-io
Copy link
codecov-io commented Nov 29, 2016

Current coverage is 51.20% (diff: 100%)

Merging #2085 into master will decrease coverage by 10.02%

@@             master      #2085   diff @@
==========================================
  Files           129        129           
  Lines         11416      11416           
  Methods           0          0           
  Messages          0          0           
  Branches          0          0           
==========================================
- Hits           6990       5846   -1144   
- Misses         3535       4821   +1286   
+ Partials        891        749    -142   

Powered by Codecov. Last update 2d50093...65a2b50

@aaronlehmann
Copy link
Contributor

LGTM

@dmcgowan
Copy link
Collaborator Author
dmcgowan commented Dec 6, 2016

Going to wait until docker rc4 to merge this so I can update the test to run against 1.13 before merging.

@thaJeztah
Copy link
Member

ping @dmcgowan what's the status on this one?

@dmcgowan
Copy link
Collaborator Author
dmcgowan commented Jan 5, 2017

Still one failure and waiting for the rc5 docker image to come out. Once that is updated and janky passes will merge.

@dmcgowan dmcgowan force-pushed the plugin-integration-test branch from e0be2bd to 7f4ccf0 Compare January 10, 2017 00:07
Signed-off-by: Derek McGowan <derek@mcgstyle.net> (github: dmcgowan)
Signed-off-by: Derek McGowan <derek@mcgstyle.net> (github: dmcgowan)
Use rc4 for support with plugins change

Signed-off-by: Derek McGowan <derek@mcgstyle.net> (github: dmcgowan)
Signed-off-by: Derek McGowan <derek@mcgstyle.net> (github: dmcgowan)
@dmcgowan dmcgowan force-pushed the plugin-integration-test branch from 7f4ccf0 to fd13a2e Compare January 10, 2017 00:44
@dmcgowan dmcgowan merged commit 1140605 into distribution:master Jan 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0