<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>9199</bug_id>
          
          <creation_ts>2016-03-02 14:23:41 +0000</creation_ts>
          <short_desc>[meta-swupd] Enable updating of configuration files in swupd-client</short_desc>
          <delta_ts>2016-04-18 12:46:38 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>7</classification_id>
          <classification>Build System, Metadata &amp; Runtime</classification>
          <product>Other YP Layers</product>
          <component>meta-swupd</component>
          <version>2.1</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>2.2 M1</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Joshua Lock">joshuagloe</reporter>
          <assigned_to name="Joshua Lock">joshuagloe</assigned_to>
          <cc>patrick.ohly</cc>
    
    <cc>poky.bs.watcher</cc>
    
    <cc>poky.watcher</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>59624</commentid>
    <comment_count>0</comment_count>
    <who name="Joshua Lock">joshuagloe</who>
    <bug_when>2016-03-02 14:23:41 +0000</bug_when>
    <thetext>We have a patch to ensure that swupd-server writes information about configuration files to the generated Manifests, we need a similar patch to swupd-client so that it doesn&apos;t ignore configuration files it finds in the manifests.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>59632</commentid>
    <comment_count>1</comment_count>
    <who name="Joshua Lock">joshuagloe</who>
    <bug_when>2016-03-02 17:10:56 +0000</bug_when>
    <thetext>We have various files that are generated, or modified, at first boot. i.e. in my minimal distro + systemd the following files all fail a swupd verify after initial boot:

/etc/ld.so.cache
/etc/group-
/etc/gshadow
/etc/mtab
/etc.machine-id

I wonder what the best way to enhance swupd client for the stateful use-case is.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61202</commentid>
    <comment_count>2</comment_count>
      <attachid>3125</attachid>
    <who name="Joshua Lock">joshuagloe</who>
    <bug_when>2016-04-14 13:12:37 +0000</bug_when>
    <thetext>Created attachment 3125
Bare minimum patch to have swupd-client pay attention to files in /etc

Attached is a tiny patch which changes swupd-client to process files in /etc (developed against v2.x), this fails to consider the boot-time generated/modified files.

Most likely the correct solution is to modify swupd-server to be aware of these files and not include them in the manifest.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61279</commentid>
    <comment_count>3</comment_count>
    <who name="Patrick Ohly">patrick.ohly</who>
    <bug_when>2016-04-15 10:31:58 +0000</bug_when>
    <thetext>I ported that patch to current meta-swupd and currently have everything working in Ostro OS. Test build here:

https://github.com/ostroproject/ostro-os/pull/58

The basic approach is that the rootfs must not contain anything in /etc which later gets modified on the device. IMHO this depends a lot on the distro; I wouldn&apos;t expect meta-swupd to get involved in that area.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61317</commentid>
    <comment_count>4</comment_count>
    <who name="Joshua Lock">joshuagloe</who>
    <bug_when>2016-04-17 21:23:03 +0000</bug_when>
    <thetext>(In reply to comment #3)
&gt; I ported that patch to current meta-swupd and currently have everything
&gt; working in Ostro OS. Test build here:
&gt; 
&gt; https://github.com/ostroproject/ostro-os/pull/58

Patch has been applied to meta-swupd master, thanks.
 
&gt; The basic approach is that the rootfs must not contain anything in /etc
&gt; which later gets modified on the device. IMHO this depends a lot on the
&gt; distro; I wouldn&apos;t expect meta-swupd to get involved in that area.

Agreed, it does depend a lot on the distro. I plan to at least update the Guide to discuss this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61335</commentid>
    <comment_count>5</comment_count>
    <who name="Joshua Lock">joshuagloe</who>
    <bug_when>2016-04-18 12:46:38 +0000</bug_when>
    <thetext>I&apos;ve also applied a similar patch to the 3.x version of the recipes in meta-swupd master. Marking this bug as resolved.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>3125</attachid>
            <date>2016-04-14 13:12:37 +0000</date>
            <delta_ts>2016-04-14 13:12:37 +0000</delta_ts>
            <desc>Bare minimum patch to have swupd-client pay attention to files in /etc</desc>
            <filename>0002-swupd-client-recognise-and-update-changed-config-fil.patch</filename>
            <type>text/plain</type>
            <size>4519</size>
            <attacher name="Joshua Lock">joshuagloe</attacher>
            
              <data encoding="base64">RnJvbSAwM2M2ZDA5MzcxZDhmMjc5NzZlYjliZDAzZDkzOTUzNTk4NzQ2MDY3IE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBKb3NodWEgTG9jayA8am9zaHVhLmcubG9ja0BpbnRlbC5jb20+
CkRhdGU6IFdlZCwgMiBNYXIgMjAxNiAxNDozNzo1OSArMDAwMApTdWJqZWN0OiBbUEFUQ0ggMi82
XSBzd3VwZC1jbGllbnQ6IHJlY29nbmlzZSBhbmQgdXBkYXRlIGNoYW5nZWQgY29uZmlnIGZpbGVz
Cgpzd3VwZCB1cHN0cmVhbSBleHBlY3RzIGl0cyBob3N0IE9TLCBpLmUuIENsZWFyIExpbnV4LCB0
byBiZSBzdGF0ZWxlc3MgYW5kCnRodXMgYnkgZGVmYXVsdCBpZ25vcmVzIGNvbmZpZ3VyYXRpb24g
ZmlsZSBjaGFuZ2VzIGluIHRoZSBtYW5pZmVzdHMuCgpBZGQgYSBjb25maWd1cmUgb3B0aW9uIHRv
IHN3dXBkLWNsaWVudCB0byBub3QgaWdub3JlIHRoZXNlIGZpbGVzLCBtYXRjaGluZwphIHNpbWls
YXIgY2hhbmdlIGluIHN3dXBkLXNlcnZlciB0byBlbnN1cmUgY29uZmlnIGZpbGVzIGFyZSBpbmNs
dWRlZCBpbgp0aGUgbWFuaWZlc3RzLgoKRG8gbm90IGVuYWJsZSB0aGlzIG9wdGlvbiBieSBkZWZh
dWx0IGluIG91ciByZWNpcGUgYXMgd2UgaGF2ZSBzZXZlcmFsIGZpbGVzCmluIC9ldGMgd2hpY2gg
YXJlIGdlbmVyYXRlZC9tb2RpZmllZCBvbiBmaXJzdC1ib290IGFuZCB3aWxsIGNhdXNlIGEKc3d1
cGQgdmVyaWZ5IHRvIGZhaWwuCgpbWU9DVE8gIzkxOTldCgpTaWduZWQtb2ZmLWJ5OiBKb3NodWEg
TG9jayA8am9zaHVhLmcubG9ja0BpbnRlbC5jb20+Ci0tLQogLi4ucmUtb3B0aW9uLXRvLXJlLWVu
YWJsZS11cGRhdGluZy1vZi1jb25maWcucGF0Y2ggfCA1OCArKysrKysrKysrKysrKysrKysrKysr
CiByZWNpcGVzLWNvcmUvc3d1cGQtY2xpZW50L3N3dXBkLWNsaWVudF8yLjg3LmJiICAgICB8ICA0
ICsrCiAyIGZpbGVzIGNoYW5nZWQsIDYyIGluc2VydGlvbnMoKykKIGNyZWF0ZSBtb2RlIDEwMDY0
NCByZWNpcGVzLWNvcmUvc3d1cGQtY2xpZW50L3N3dXBkLWNsaWVudC8wMDAxLUFkZC1jb25maWd1
cmUtb3B0aW9uLXRvLXJlLWVuYWJsZS11cGRhdGluZy1vZi1jb25maWcucGF0Y2gKCmRpZmYgLS1n
aXQgYS9yZWNpcGVzLWNvcmUvc3d1cGQtY2xpZW50L3N3dXBkLWNsaWVudC8wMDAxLUFkZC1jb25m
aWd1cmUtb3B0aW9uLXRvLXJlLWVuYWJsZS11cGRhdGluZy1vZi1jb25maWcucGF0Y2ggYi9yZWNp
cGVzLWNvcmUvc3d1cGQtY2xpZW50L3N3dXBkLWNsaWVudC8wMDAxLUFkZC1jb25maWd1cmUtb3B0
aW9uLXRvLXJlLWVuYWJsZS11cGRhdGluZy1vZi1jb25maWcucGF0Y2gKbmV3IGZpbGUgbW9kZSAx
MDA2NDQKaW5kZXggMDAwMDAwMC4uYWMxNmQwNgotLS0gL2Rldi9udWxsCisrKyBiL3JlY2lwZXMt
Y29yZS9zd3VwZC1jbGllbnQvc3d1cGQtY2xpZW50LzAwMDEtQWRkLWNvbmZpZ3VyZS1vcHRpb24t
dG8tcmUtZW5hYmxlLXVwZGF0aW5nLW9mLWNvbmZpZy5wYXRjaApAQCAtMCwwICsxLDU4IEBACitG
cm9tIDFmMmFkNGNkZmY2Yjk5OWZjZDAwMTkzY2RkMWRjOGEzNWU0OWMwYjIgTW9uIFNlcCAxNyAw
MDowMDowMCAyMDAxCitGcm9tOiBKb3NodWEgTG9jayA8am9zaHVhLmcubG9ja0BpbnRlbC5jb20+
CitEYXRlOiBUaHUsIDMgTWFyIDIwMTYgMTk6NTU6NDEgKzAwMDAKK1N1YmplY3Q6IFtQQVRDSF0g
QWRkIGNvbmZpZ3VyZSBvcHRpb24gdG8gcmUtZW5hYmxlIHVwZGF0aW5nIG9mIGNvbmZpZyBmaWxl
cworCitTaWduZWQtb2ZmLWJ5OiBKb3NodWEgTG9jayA8am9zaHVhLmcubG9ja0BpbnRlbC5jb20+
CisKK1Vwc3RyZWFtLVN0YXR1czogUGVuZGluZworCistLS0KKyBjb25maWd1cmUuYWMgICAgIHwg
NyArKysrKysrCisgc3JjL2hldXJpc3RpY3MuYyB8IDUgKysrLS0KKyAyIGZpbGVzIGNoYW5nZWQs
IDEwIGluc2VydGlvbnMoKyksIDIgZGVsZXRpb25zKC0pCisKK2RpZmYgLS1naXQgYS9jb25maWd1
cmUuYWMgYi9jb25maWd1cmUuYWMKK2luZGV4IGY5NGExN2QuLjAwNjM0NjMgMTAwNjQ0CistLS0g
YS9jb25maWd1cmUuYWMKKysrKyBiL2NvbmZpZ3VyZS5hYworQEAgLTc4LDYgKzc4LDEzIEBAIEFD
X0FSR19FTkFCTEUoCisgCWVuYWJsZV9saW51eF9yb290ZnNfYnVpbGQ9Im5vIgorICkKKyAKKytB
Q19BUkdfRU5BQkxFKAorKyAgICAgICAgW3N0YXRlbGVzc10sCisrICAgICAgICBBU19IRUxQX1NU
UklORyhbLS1kaXNhYmxlLXN0YXRlbGVzc10sW09TIGlzIG5vdCBzdGF0ZWxlc3MsIGRvIG5vdCBp
Z25vcmUgY29uZmlndXJhdGlvbiBmaWxlcyAoc3RhdGVsZXNzIGJ5IGRlZmF1bHQpXSksCisrICAg
ICAgICBBQ19ERUZJTkUoT1NfSVNfU1RBVEVMRVNTLDAsW09TIGlzIG5vdCBzdGF0ZWxlc3NdKSwK
KysgICAgICAgIEFDX0RFRklORShPU19JU19TVEFURUxFU1MsMSxbT1MgaXMgc3RhdGVsZXNzXSkK
KyspCisrCisgQVNfSUYoW3Rlc3QgIiRlbmFibGVfbGludXhfYnRyZnNfYnVpbGQiID0gInllcyIg
LWEgIiRlbmFibGVfYW5kcm9pZF9idWlsZCIgPSAieWVzIl0sCisgICAgICAgW0FDX01TR19FUlJP
UihbQ2Fubm90IGVuYWJsZSBtb3JlIHRoYW4gb25lIGJ1aWxkIHZhcmlhbnQuIENob29zZSBhIHNp
bmdsZSB2YXJpYW50Ll0pXSkKKyAKK2RpZmYgLS1naXQgYS9zcmMvaGV1cmlzdGljcy5jIGIvc3Jj
L2hldXJpc3RpY3MuYworaW5kZXggMTJmYjU5Yy4uOTJkYmNkYyAxMDA2NDQKKy0tLSBhL3NyYy9o
ZXVyaXN0aWNzLmMKKysrKyBiL3NyYy9oZXVyaXN0aWNzLmMKK0BAIC0yNyw2ICsyNyw3IEBACisg
I2luY2x1ZGUgPHN0cmluZy5oPgorICNpbmNsdWRlIDxhc3NlcnQuaD4KKyAKKysjaW5jbHVkZSA8
Y29uZmlnLmg+CisgI2luY2x1ZGUgPHN3dXBkLmg+CisgCisgLyogdHJhaWxpbmcgc2xhc2ggaXMg
dG8gaW5kaWNhdGUgZGlyIGl0c2VsZiBpcyBleHBlY3RlZCB0byBleGlzdCwgYnV0CitAQCAtMTEz
LDggKzExNCw4IEBAIHZvaWQgYXBwbHlfaGV1cmlzdGljcyhzdHJ1Y3QgZmlsZSAqZmlsZSkKKyAK
KyBib29sIGlnbm9yZShzdHJ1Y3QgZmlsZSAqZmlsZSkKKyB7CistCWlmICgoZmlsZS0+aXNfY29u
ZmlnKSB8fAorLQkgICAgaXNfY29uZmlnKGZpbGUtPmZpbGVuYW1lKSB8fCAvLyBpZGVhbGx5IHdl
IHRydXN0IHRoZSBtYW5pZmVzdCBidXQgc2hvcnQgdGVybSByZWFwcGx5IGNoZWNrIGhlcmUKKysJ
aWYgKChPU19JU19TVEFURUxFU1MgJiYgZmlsZS0+aXNfY29uZmlnKSB8fAorKwkgICAgKE9TX0lT
X1NUQVRFTEVTUyAmJiBpc19jb25maWcoZmlsZS0+ZmlsZW5hbWUpKSB8fCAvLyBpZGVhbGx5IHdl
IHRydXN0IHRoZSBtYW5pZmVzdCBidXQgc2hvcnQgdGVybSByZWFwcGx5IGNoZWNrIGhlcmUKKyAJ
ICAgIChmaWxlLT5pc19zdGF0ZSkgfHwKKyAJICAgIGlzX3N0YXRlKGZpbGUtPmZpbGVuYW1lKSB8
fCAvLyBpZGVhbGx5IHdlIHRydXN0IHRoZSBtYW5pZmVzdCBidXQgc2hvcnQgdGVybSByZWFwcGx5
IGNoZWNrIGhlcmUKKyAJICAgIChmaWxlLT5pc19ib290ICYmICBmaXggJiYgIGZpbGUtPmlzX2Rl
bGV0ZWQpIHx8IC8vIHNob3VsZG4ndCBoYXBwZW4KKy0tIAorMi41LjAKKwpkaWZmIC0tZ2l0IGEv
cmVjaXBlcy1jb3JlL3N3dXBkLWNsaWVudC9zd3VwZC1jbGllbnRfMi44Ny5iYiBiL3JlY2lwZXMt
Y29yZS9zd3VwZC1jbGllbnQvc3d1cGQtY2xpZW50XzIuODcuYmIKaW5kZXggODIyNjYyNC4uNDQz
MDViYyAxMDA2NDQKLS0tIGEvcmVjaXBlcy1jb3JlL3N3dXBkLWNsaWVudC9zd3VwZC1jbGllbnRf
Mi44Ny5iYgorKysgYi9yZWNpcGVzLWNvcmUvc3d1cGQtY2xpZW50L3N3dXBkLWNsaWVudF8yLjg3
LmJiCkBAIC05LDYgKzksNyBAQCBTUkNfVVJJID0gIlwKICAgICBmaWxlOi8vUmlnaHQtdXNhZ2Ut
b2YtQUNfQVJHX0VOQUJMRS1vbi1iemlwMi5wYXRjaCBcCiAgICAgZmlsZTovL0NoYW5nZS1zeXN0
ZW1jdGwtcGF0aC10by1PRS1zeXN0ZW1jdGwtcGF0aC5wYXRjaCBcCiAgICAgZmlsZTovLzAwMDEt
VG9sZXJhdGUtcXVvdGVzLWluLW9zLXJlbGVhc2UtZmlsZXMucGF0Y2ggXAorICAgIGZpbGU6Ly8w
MDAxLUFkZC1jb25maWd1cmUtb3B0aW9uLXRvLXJlLWVuYWJsZS11cGRhdGluZy1vZi1jb25maWcu
cGF0Y2ggXAogIgogCiBTUkNfVVJJW21kNXN1bV0gPSAiNWQyNzJjNjJlZGI4YTljNTc2MDA1YWM1
ZTExODJlYTMiCkBAIC0yNCw2ICsyNSw5IEBAIGluaGVyaXQgcGtnY29uZmlnIGF1dG90b29scy1i
cm9rZW5zZXAgc3lzdGVtZAogCiBFWFRSQV9PRUNPTkYgPSAiLS13aXRoLXN5c3RlbWRzeXN0ZW11
bml0ZGlyPSR7c3lzdGVtZF9zeXN0ZW1fdW5pdGRpcn0iCiAKK1BBQ0tBR0VDT05GSUcgPz89ICJz
dGF0ZWxlc3MiCitQQUNLQUdFQ09ORklHW3N0YXRlbGVzc10gPSAiLC0tZGlzYWJsZS1zdGF0ZWxl
c3MiCisKICNUT0RPOiBjcmVhdGUgYW5kIGluc3RhbGwgL3Zhci9saWIvc3d1cGQve2RlbHRhLHN0
YWdlZC9kb3dubG9hZH0KIGRvX2luc3RhbGxfYXBwZW5kICgpIHsKICAgICAjIHN3dXBkLWNsaWVu
dCAyLjg3IGRvZXNuJ3QgKHN1Y2Nlc2Z1bGx5KSBjcmVhdGUgdGhlc2UgYW5kIGZhaWxzIHRvIHVw
ZGF0ZQotLSAKMS45LjEKCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>