-
Notifications
You must be signed in to change notification settings - Fork 35
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
PWX-31842: PKS mounts-leak fix #1081
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #1081 +/- ##
==========================================
+ Coverage 77.47% 77.49% +0.02%
==========================================
Files 62 62
Lines 17086 17083 -3
==========================================
+ Hits 13237 13239 +2
+ Misses 2922 2919 -3
+ Partials 927 925 -2
☔ View full report in Codecov by Sentry. |
327fcd9
to
f3843e8
Compare
@@ -51,7 +51,7 @@ spec: | |||
- name: "PX_TEMPLATE_VERSION" | |||
value: "v4" | |||
- name: "PRE-EXEC" | |||
value: "if [ ! -x /bin/systemctl ]; then apt-get update; apt-get install -y systemd; fi" | |||
value: "rm -fr /var/lib/osd/driver" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, we don't need the old PRE-EXEC command?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's correct -- it was added initially to support Ubuntu 14 PKS stemcells, which did not have the systemd
package installed. These are "long gone" now -- standard support for Ubuntu 14 LTS version ended beginning of 2019.
ccd52c9
to
762abc6
Compare
* mounting changing OSD-mount to `-v /var/lib/osd:/var/vcap/store/lib/osd:shared` * removing offending "pxlogs" (/var/vcap/store/lib/osd/log) mount * changing `PRE-EXEC` to remove the /var/lib/osd/driver directory * fixing UTs Signed-off-by: Zoran Rajic <zrajic@purestorage.com>
Thanks Piyush -- merging this PR, cherry-picking for |
* mounting changing OSD-mount to `-v /var/lib/osd:/var/vcap/store/lib/osd:shared` * removing offending "pxlogs" (/var/vcap/store/lib/osd/log) mount * changing `PRE-EXEC` to remove the /var/lib/osd/driver directory * fixing UTs Signed-off-by: Zoran Rajic <zrajic@purestorage.com>
-v /var/lib/osd:/var/vcap/store/lib/osd:shared
PRE-EXEC
to remove the /var/lib/osd/driver directoryWhat this PR does / why we need it:
ISSUE: the
/var/lib/osd
+/var/lib/osd/log
mounts were causing the "mounts leak" issue on PKS platformAs a fix, we are:
/var/vcap/store/lib/osd
/var/lib/osd/log
mountPRE-EXEC
to remove /var/lib/osd/driver directory, as thepwx.sock
file inside causes 2-minute delays for everypxctl
commandWhich issue(s) this PR fixes (optional)
Closes # PWX-31842
Special notes for your reviewer: