Merge ~smoser/cloud-init:fix/rpm-build-systemd-fsck-dropin into cloud-init:master

Proposed by Scott Moser on 2017-07-26
Status: Merged
Approved by: Scott Moser on 2017-07-26
Approved revision: 8a5296c41db45be3a172862f324ad44e732a2250
Merged at revision: 8a5296c41db45be3a172862f324ad44e732a2250
Proposed branch: ~smoser/cloud-init:fix/rpm-build-systemd-fsck-dropin
Merge into: cloud-init:master
Diff against target: 16 lines (+6/-0)
1 file modified
packages/redhat/cloud-init.spec.in (+6/-0)
Reviewer Review Type Date Requested Status
Chad Smith 2017-07-26 Approve on 2017-07-26
Server Team CI bot continuous-integration Approve on 2017-07-26
Review via email: mp+328101@code.launchpad.net

Commit Message

centos: do not package systemd-fsck drop-in.

The change here fixes the build of CentOS 7 rpm.

The systemd 'drop-in' file systemd-fsck@.service.d/cloud-init.conf is
not expected to work on CentOS at the moment. There, the
cloud-init-local.service currently runs without DefaultDependencies=no.
That would likely cause a dependency loop.

The drop-in was added for bug 1691489. It may well be needed at a later
date in CentOS also.

To post a comment you must log in.

PASSED: Continuous integration, rev:5310c7cd5662afa5e2fe040491e4c5201d696a6b
https://jenkins.ubuntu.com/server/job/cloud-init-ci/96/
Executed test runs:
    SUCCESS: Checkout
    SUCCESS: Unit & Style Tests
    SUCCESS: Ubuntu LTS: Build
    SUCCESS: Ubuntu LTS: Integration
    SUCCESS: CentOS 6 & 7: Build & Test
    IN_PROGRESS: Declarative: Post Actions

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/96/rebuild

review: Approve (continuous-integration)
Scott Moser (smoser) wrote :

spelling error in subject.
... to fix on pull
systemd

Chad Smith (chad.smith) wrote :

Looks simple/good. Builds working as you mentioned already https://jenkins.ubuntu.com/server/job/cloud-init-ci/96/console.

We can revisit this case in the future if CentOs exhibits /mnt timing issues across redeployments in Azure.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/packages/redhat/cloud-init.spec.in b/packages/redhat/cloud-init.spec.in
2index 6ab0d20..d995b85 100644
3--- a/packages/redhat/cloud-init.spec.in
4+++ b/packages/redhat/cloud-init.spec.in
5@@ -115,6 +115,12 @@ rm -rf $RPM_BUILD_ROOT%{python_sitelib}/tests
6 mkdir -p $RPM_BUILD_ROOT/%{_sharedstatedir}/cloud
7 mkdir -p $RPM_BUILD_ROOT/%{_libexecdir}/%{name}
8
9+# LP: #1691489: Remove systemd-fsck dropin (currently not expected to work)
10+%if "%{init_system}" == "systemd"
11+rm $RPM_BUILD_ROOT/usr/lib/systemd/system/systemd-fsck@.service.d/cloud-init.conf
12+%endif
13+
14+
15 %clean
16 rm -rf $RPM_BUILD_ROOT
17

Subscribers

People subscribed via source and target branches

to all changes: