<?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>6132</bug_id>
          
          <creation_ts>2014-04-08 15:36:58 +0000</creation_ts>
          <short_desc>Have persistent log data (/var/log)</short_desc>
          <delta_ts>2017-08-21 14:20:51 +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>OE-Core</product>
          <component>core</component>
          <version>1.7</version>
          <rep_platform>All</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard>06 Jan 2017: IN PROGRESS REVIEW</status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>2.4 M3</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Mark Hatle">mark.hatle</reporter>
          <assigned_to name="Chen Qi">Qi.Chen</assigned_to>
          <cc>bluelightning</cc>
    
    <cc>jeff.polk</cc>
    
    <cc>kristi</cc>
    
    <cc>meta.mr.watcher</cc>
    
    <cc>meta.watcher</cc>
    
    <cc>milinevskyy</cc>
    
    <cc>net147</cc>
    
    <cc>Qi.Chen</cc>
    
    <cc>richard.purdie</cc>
    
    <cc>srifenbark</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Done (doc changes complete)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>42396</commentid>
    <comment_count>0</comment_count>
    <who name="Mark Hatle">mark.hatle</who>
    <bug_when>2014-04-08 15:36:58 +0000</bug_when>
    <thetext>There are some systems that require persistent log data.  Currently /var/log is defaulted to a tmpfs mount via the populate-volatiles subsystem.

In order to change this, it&apos;s required to modify the fs-perms, base-files and initscripts packages.  There should be a distribution switch to do this.

It may also be appropriate to ensure logrotate behavior is appropriate for disk (persistent) storage vs tmpfs based storage.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>47176</commentid>
    <comment_count>1</comment_count>
    <who name="Chen Qi">Qi.Chen</who>
    <bug_when>2014-11-25 06:03:29 +0000</bug_when>
    <thetext>I&apos;ve seen emails about persistent /var/log on Yocto mailing list.
I think this feature is important and needs to be done in 1.8.

//Chen Qi</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>49719</commentid>
    <comment_count>2</comment_count>
    <who name="Chen Qi">Qi.Chen</who>
    <bug_when>2015-03-25 01:58:20 +0000</bug_when>
    <thetext>Hi Mark,

Could you please help review the patches?

git://git.openembedded.org/openembedded-core-contrib ChenQi/persistent-var-log
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=ChenQi/persistent-var-log

Thanks,
Chen Qi</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>51435</commentid>
    <comment_count>3</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2015-06-02 20:28:07 +0000</bug_when>
    <thetext>Hi, 

It looks like the documentation for this bug would be to provide a glossary description for VOLATILE_LOG_DIR variable, which we currently do not document.  Aside from documenting this variable and perhaps adding some information else where in the YP doc set showing its use, would there be any other end-user documentation points for this issue?

Thanks, 
Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>68315</commentid>
    <comment_count>4</comment_count>
    <who name="Chen Qi">Qi.Chen</who>
    <bug_when>2016-11-15 06:39:50 +0000</bug_when>
    <thetext>I&apos;ll rebase the patches and send them out again.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>68473</commentid>
    <comment_count>5</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2016-11-22 00:00:05 +0000</bug_when>
    <thetext>Hey, 

Seems weird that we use &quot;&quot; to override &quot;log&quot; and make it persistent and not some more meaningful value for VOLATILE_LOG_DIR.  Anyway... to make sure I understand how VOLATILE_LOG_DIR works it basically defaults to &quot;log&quot;, which makes the /var/log volatile.  If the user wants to save this log and make it persistent, they simply set VOLATILE_LOG_DIR to &quot;&quot;.  

Are there any other user steps they need to take to get this done?

Thanks, 
Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>68930</commentid>
    <comment_count>6</comment_count>
    <who name="Dmytro Milinevskyi">milinevskyy</who>
    <bug_when>2016-12-08 14:35:43 +0000</bug_when>
    <thetext>Hello Chen, 
could you please bump the revisions of the recipes you&apos;ve modified?
It&apos;s needed to have &quot;smart&quot; updates to work.
Thanks</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>68932</commentid>
    <comment_count>7</comment_count>
    <who name="Mark Hatle">mark.hatle</who>
    <bug_when>2016-12-08 15:19:55 +0000</bug_when>
    <thetext>Are you using the PR server?  (You should be when doing target package upgrades.)

Community policy is not to bump PR numbers manually, except for a few extreme cases.

If you are using the PR server, and the PR did not update, please let us know (in a new bug) what the conditions are, as it may be a bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>68940</commentid>
    <comment_count>8</comment_count>
    <who name="Dmytro Milinevskyi">milinevskyy</who>
    <bug_when>2016-12-08 15:41:47 +0000</bug_when>
    <thetext>Mark, thanks for your reply. We are not using PR service as we can&apos;t be 100% sure that the build history is always persisted. I see your point and will bump the PRs of the packages manually in the private branch.
Thanks</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69011</commentid>
    <comment_count>9</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2016-12-10 00:03:10 +0000</bug_when>
    <thetext>Any feedback on Comment 3 and Comment 5?  For documentation?

thanks, 
Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69024</commentid>
    <comment_count>10</comment_count>
    <who name="Chen Qi">Qi.Chen</who>
    <bug_when>2016-12-12 01:55:01 +0000</bug_when>
    <thetext>(In reply to comment #9)
&gt; Any feedback on Comment 3 and Comment 5?  For documentation?
&gt; 
&gt; thanks, 
&gt; Scott

Sorry I didn&apos;t notice it.
I&apos;ve changed VOLATILE_LOG_DIR to default to &quot;yes&quot;, and we use &apos;no&apos; to override the default value to make /var/log on target persistent.
For documentation, apart from glossary description for VOLATILE_LOG_DIR, a section about &apos;making /var/log persistent&apos; might be needed. I&apos;ve seen some users on yocto mailing list requiring this feature, so making it clear in docs might be better.
The only problem here is that the patches are not accepted yet. When they are merged I&apos;ll change the state of this bug.

Regards,
Chen Qi</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69412</commentid>
    <comment_count>11</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2016-12-30 18:24:56 +0000</bug_when>
    <thetext>Hi, 

I updated a couple spots in the ref-manual to deal with this issue:

 1 - in the &quot;Image Features&quot; section there is a list of features the user can
     apply.  One being &quot;post-install-logging&quot;, which talks about the /var/log
     directory.  I have created a note in that entry mentioning the 
     VOLATILE_LOG_DIR variable as the way to control the persistence of 
     this target directory.  See http://www.yoctoproject.org/docs/2.3/ref-manual/ref-manual.html#ref-features-image

 2 - In the reference manual, I created a new glossary entry for the 
     VOLATILE_LOG_DIR variable.  See http://www.yoctoproject.org/docs/2.3/ref-manual/ref-manual.html#var-VOLATILE_LOG_DIR

Take a look and let me know if these are okay. 

Thanks, 
Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69578</commentid>
    <comment_count>12</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2017-01-06 15:29:38 +0000</bug_when>
    <thetext>I updated the &quot;Whiteboard&quot; information.  Please let me know if the the doc changes I have made are acceptable.

Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69906</commentid>
    <comment_count>13</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2017-01-18 21:47:44 +0000</bug_when>
    <thetext>So you changed the target... does that mean we are good?  If so, let me know and I can update the status to RESOLVED and put the doc flag to &quot;done.&quot;

Thanks, 
Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69920</commentid>
    <comment_count>14</comment_count>
    <who name="Chen Qi">Qi.Chen</who>
    <bug_when>2017-01-19 01:50:05 +0000</bug_when>
    <thetext>(In reply to comment #13)
&gt; So you changed the target... does that mean we are good?  If so, let me know
&gt; and I can update the status to RESOLVED and put the doc flag to &quot;done.&quot;
&gt; 
&gt; Thanks, 
&gt; Scott

Got it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69949</commentid>
    <comment_count>15</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2017-01-19 23:53:43 +0000</bug_when>
    <thetext>Thanks, 

Setting to RESOLVED and putting the doc flag to &quot;done.&quot;

Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75886</commentid>
    <comment_count>16</comment_count>
    <who name="Chen Qi">Qi.Chen</who>
    <bug_when>2017-08-16 11:54:51 +0000</bug_when>
    <thetext>I&apos;ve no idea why I replied &apos;got it&apos; ...
And this answer led to communication mistake, resulting in this bug changed into a wrong state.
I&apos;ve sent out rebased patches to the mailing list, waiting for review.
Sorry for my mistake.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75916</commentid>
    <comment_count>17</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2017-08-16 18:42:05 +0000</bug_when>
    <thetext>Ok - So I am not sure where we are on the doc changes.  You sent out rebased patches for the fix.  Do the documentation changes I made in Comment 11 still hold true?

Thanks,
Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75932</commentid>
    <comment_count>18</comment_count>
    <who name="Chen Qi">Qi.Chen</who>
    <bug_when>2017-08-17 01:34:14 +0000</bug_when>
    <thetext>(In reply to comment #17)
&gt; Ok - So I am not sure where we are on the doc changes.  You sent out rebased
&gt; patches for the fix.  Do the documentation changes I made in Comment 11
&gt; still hold true?
&gt; 
&gt; Thanks,
&gt; Scott

It&apos;s true for my patches. But my patches haven&apos;t been merged.
If they are merged, then there&apos;s no need to change the doc.

Regards,
Chen Qi</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>76015</commentid>
    <comment_count>19</comment_count>
    <who name="Chen Qi">Qi.Chen</who>
    <bug_when>2017-08-21 02:19:52 +0000</bug_when>
    <thetext>Merged in master. No need for changing the current doc.
Sorry for my mistake.

Regards,
Chen Qi</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>