Project

Profile

Help

Issue #991

pulp-dev.py uses /sbin/systemctl which does not exist in F21

Added by rbarlow over 6 years ago. Updated over 2 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Assignee:
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
3. High
Version:
Platform Release:
2.6.2
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix, Pulp 2
Sprint:
Quarter:

Description

A recent commit to pulp-dev.py added a call to /sbin/systemctl[0]. In F21, systemctl is located in /usr/bin/ instead of /sbin. Due to this change, pulp-dev.py fails to complete successfully which also causes our Vagrant provisioning to fail.

[0] https://github.com/pulp/pulp/commit/087d85085f99daf04e4f0929bed2f2a5c75bc2a1


Related issues

Related to Pulp - Issue #980: Systemctl daemon reload should occur using macrosCLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision c15a2dfe View on GitHub
Added by rbarlow over 6 years ago

Don't use full path to systemctl in pulp-dev.py.

Not all distributions put the systemctl executable in the same location. pulp-dev.py had been calling it as /sbin/systemctl, but Fedora 21 keeps that executable in /usr/bin/systemctl. This commit calls the executable as just systemctl and allows the PATH variable to be used to locate it.

https://pulp.plan.io/issues/991

fixes #991

Revision c15a2dfe View on GitHub
Added by rbarlow over 6 years ago

Don't use full path to systemctl in pulp-dev.py.

Not all distributions put the systemctl executable in the same location. pulp-dev.py had been calling it as /sbin/systemctl, but Fedora 21 keeps that executable in /usr/bin/systemctl. This commit calls the executable as just systemctl and allows the PATH variable to be used to locate it.

https://pulp.plan.io/issues/991

fixes #991

History

#1 Updated by rbarlow over 6 years ago

  • Related to Issue #980: Systemctl daemon reload should occur using macros added

#2 Updated by rbarlow over 6 years ago

  • Status changed from NEW to POST
  • Assignee set to rbarlow
  • Platform Release set to 2.6.2
  • Tags Easy Fix added

#3 Updated by mhrivnak over 6 years ago

  • Triaged changed from No to Yes

#4 Updated by rbarlow over 6 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#5 Updated by dkliban@redhat.com over 6 years ago

  • Status changed from MODIFIED to 6

#6 Updated by dkliban@redhat.com about 6 years ago

  • Status changed from 6 to CLOSED - CURRENTRELEASE

#8 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF