Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

revert: "virtcontainers: support pre-add storage for frakti"#530

Merged
egernst merged 1 commit intomasterfrom
revert-301-pre-addstorage-based-devmanager
Jul 27, 2018
Merged

revert: "virtcontainers: support pre-add storage for frakti"#530
egernst merged 1 commit intomasterfrom
revert-301-pre-addstorage-based-devmanager

Conversation

@sboeuf
Copy link

@sboeuf sboeuf commented Jul 27, 2018

This PR got merged while it had some issues with some shim processes
being left behind after k8s testing. And because those issues were
real issues introduced by this PR (not some random failures), now
the master branch is broken and new pull requests cannot get the
CI passing. That's the reason why this commit revert the changes
introduced by this PR so that we can fix the master branch.

Fixes #529

Signed-off-by: Sebastien Boeuf sebastien.boeuf@intel.com

@jodh-intel
Copy link

jodh-intel commented Jul 27, 2018

lgtm

Approved with PullApprove

@sboeuf sboeuf force-pushed the revert-301-pre-addstorage-based-devmanager branch from 9cb8059 to 67a9d7d Compare July 27, 2018 16:34
This PR got merged while it had some issues with some shim processes
being left behind after k8s testing. And because those issues were
real issues introduced by this PR (not some random failures), now
the master branch is broken and new pull requests cannot get the
CI passing. That's the reason why this commit revert the changes
introduced by this PR so that we can fix the master branch.

Fixes #529

Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>
@sboeuf sboeuf force-pushed the revert-301-pre-addstorage-based-devmanager branch from 67a9d7d to 927487c Compare July 27, 2018 16:40
@sboeuf sboeuf changed the title Revert "virtcontainers: support pre-add storage for frakti" revert: "virtcontainers: support pre-add storage for frakti" Jul 27, 2018
Copy link

@devimc devimc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@opendev-zuul
Copy link

opendev-zuul bot commented Jul 27, 2018

Build failed (third-party-check pipeline) integration testing with
OpenStack. For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

@sboeuf
Copy link
Author

sboeuf commented Jul 27, 2018

/recheck

@sboeuf
Copy link
Author

sboeuf commented Jul 27, 2018

@egernst @GabyCT ubuntu 16.04 is passing

@GabyCT
Copy link
Contributor

GabyCT commented Jul 27, 2018

lgtm

@katacontainersbot
Copy link
Contributor

PSS Measurement:
Qemu: 175859 KB
Proxy: 5730 KB
Shim: 8872 KB

Memory inside container:
Total Memory: 2043480 KB
Free Memory: 2003828 KB

@sboeuf
Copy link
Author

sboeuf commented Jul 27, 2018

@egernst please merge if you're fine with this.

@egernst egernst merged commit f4a7712 into master Jul 27, 2018
@WeiZhang555
Copy link
Member

Sorry for late response, I will investigate more and thank @sboeuf for fixing this!

@sboeuf
Copy link
Author

sboeuf commented Jul 30, 2018

@WeiZhang555 no problem, I hope you get this fixed quickly so that we can get your PR back ;)

@egernst egernst deleted the revert-301-pre-addstorage-based-devmanager branch August 16, 2018 18:19
@egernst egernst mentioned this pull request Aug 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

After running K8s tests processes are left behind

7 participants