From Fedora Project Wiki
(update release note tracker)
 
(13 intermediate revisions by 3 users not shown)
Line 1: Line 1:
{{admon/important | Comments and Explanations | The page source contains comments providing guidance to fill out each section. They are invisible when viewing this page. To read it, choose the "view source" link.<br/> '''Copy the source to a ''new page'' before making changes!  DO NOT EDIT THIS TEMPLATE FOR YOUR CHANGE PROPOSAL.'''}}
{{admon/tip | Guidance | For details on how to fill out this form, see the [https://docs.fedoraproject.org/en-US/program_management/changes_guide/ documentation].}}
{{admon/tip | Report issues | To report an issue with this template, file an issue in the [https://pagure.io/fedora-pgm/pgm_docs pgm_docs repo].}}


<!-- The actual name of your proposed change page should look something like: Changes/Your_Change_Proposal_Name.  This keeps all change proposals in the same namespace -->
<!-- The actual name of your proposed change page should look something like: Changes/Your_Change_Proposal_Name.  This keeps all change proposals in the same namespace -->
Line 9: Line 4:
= Allow Removal of tzdata <!-- The name of your change proposal --> =
= Allow Removal of tzdata <!-- The name of your change proposal --> =


{{Change_Proposal_Banner}}


== Summary ==
== Summary ==
Line 28: Line 22:


== Current status ==
== Current status ==
[[Category:ChangePageIncomplete]]
[[Category:ChangeAcceptedF39]]
<!-- When your change proposal page is completed and ready for review and announcement -->
<!-- When your change proposal page is completed and ready for review and announcement -->
<!-- remove Category:ChangePageIncomplete and change it to Category:ChangeReadyForWrangler -->
<!-- remove Category:ChangePageIncomplete and change it to Category:ChangeReadyForWrangler -->
Line 45: Line 39:
ON_QA -> change is fully code complete
ON_QA -> change is fully code complete
-->
-->
* [<will be assigned by the Wrangler> devel thread]
* [https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/thread/CRUWIEHISRLYQ5ZLPFSAFJPRTDQQLBO2/ devel thread]
* FESCo issue: <will be assigned by the Wrangler>
* FESCo issue: [https://pagure.io/fesco/issue/3031 #3031]
* Tracker bug: <will be assigned by the Wrangler>
* Tracker bug: [https://bugzilla.redhat.com/show_bug.cgi?id=2233281 #2233281]
* Release notes tracker: <will be assigned by the Wrangler>
* Release notes tracker: [https://pagure.io/fedora-docs/release-notes/issue/1010 #1010]


== Detailed Description ==
== Detailed Description ==
This change will allow the removal of tzdata.  When tzdata is removed, the system will default to UTC. In order to reduce overhead, many container installations now remove the data associated with tzdata but cannot fuly remove the package due to dependencies by other packages.  This results in confusion regarding the expected timezone info.
This change will allow the removal of tzdata.  When tzdata is removed, the system will default to UTC. In order to reduce overhead, many container installations now remove the data associated with tzdata but cannot fuly remove the package due to dependencies by other packages.  This results in confusion regarding the expected timezone info.


In order for this to work, we need packages that use tzdata at run time to switch from Require'ing tzdata to Recommend'ing tzdata.  These packages should also gracefully handle instances where tzdata has been resolved.  For example, this would require recognizing that tzdata is not present and providing an appropriate error, such as recommending that the user install tzdata.
In order for this to work, we need packages that use tzdata at run time to switch from Require'ing tzdata to Recommend'ing tzdata.  These packages should also gracefully handle instances where tzdata has been removed.  For example, this would require recognizing that tzdata is not present and providing an appropriate error, such as recommending that the user install tzdata.


== Feedback ==
== Feedback ==
Line 97: Line 91:
* glibc (glibc-common)
* glibc (glibc-common)
* gcc (libstdc++)
* gcc (libstdc++)
List of packages which would be beneficial to be changed:
* python3.XX (3.9, 3.10, 3.11, 3.12)
* python3.XX (3.9, 3.10, 3.11, 3.12)
List of packages which would be beneficial to be changed:
* python3-dateutil
* python3-dateutil
* python3-pytz
* python3-pytz
* libical
* libical
Upon acceptance of the change request we will file bugs to fix each of these packages for Fedora 39.
Upon acceptance of the change request we will file bugs to fix each of these packages for Fedora 39.
<!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
<!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
<!-- What work do other developers have to accomplish to complete the feature in time for release?  Is it a large change affecting many parts of the distribution or is it a very isolated change? What are those changes?-->
<!-- What work do other developers have to accomplish to complete the feature in time for release?  Is it a large change affecting many parts of the distribution or is it a very isolated change? What are those changes?-->


* Release engineering: No changes needed.
* Release engineering: No changes needed.
[https://pagure.io/releng/issues #Releng issue number] <!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
<!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
<!-- Does this feature require coordination with release engineering (e.g. changes to installer image generation or update package delivery)?  Is a mass rebuild required?  include a link to the releng issue.  
<!-- Does this feature require coordination with release engineering (e.g. changes to installer image generation or update package delivery)?  Is a mass rebuild required?  include a link to the releng issue.  
The issue is required to be filed prior to feature submission, to ensure that someone is on board to do any process development work and testing and that all changes make it into the pipeline; a bullet point in a change is not sufficient communication -->
The issue is required to be filed prior to feature submission, to ensure that someone is on board to do any process development work and testing and that all changes make it into the pipeline; a bullet point in a change is not sufficient communication -->
Line 197: Line 191:


<!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
<!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
N/A (not a System Wide Change)
Document that the tzdata package may be removed if needed to reduce space on containers.  With tzdata removed, the system will default to UTC.


== Release Notes ==
== Release Notes ==

Latest revision as of 23:26, 28 September 2023


Allow Removal of tzdata

Summary

Allow the removal of tzdata especially on containers in order to minimize size.

Owner

  • Name: Patsy Griffin (Franklin)
  • Email: patsy@redhat.com


Current status

Detailed Description

This change will allow the removal of tzdata. When tzdata is removed, the system will default to UTC. In order to reduce overhead, many container installations now remove the data associated with tzdata but cannot fuly remove the package due to dependencies by other packages. This results in confusion regarding the expected timezone info.

In order for this to work, we need packages that use tzdata at run time to switch from Require'ing tzdata to Recommend'ing tzdata. These packages should also gracefully handle instances where tzdata has been removed. For example, this would require recognizing that tzdata is not present and providing an appropriate error, such as recommending that the user install tzdata.

Feedback

In June of 2021, we proposed creating a new tzdata sub-package that would only provide the UTC timezone. As part of the discussion around this proposal, it was recommended that we completely remove tzdata. We appreciated this input and welcome additional feedback.

Benefit to Fedora

This change will allow tzdata to be removed from containers without leaving inconsistent package remnants.

Scope

  • Proposal owners: No changes are needed to tzdata.
  • Other developers: Some packages need to change their spec files from Requires: tzdata to Recommends: tzdata. It would be beneficial if all packages switched in this way, but it is not required. Supporting optional tzdata installation for as many workloads as possible allows those workloads to minimize their container image size.

List of packages which need to be changed:

  • glibc (glibc-common)
  • gcc (libstdc++)

List of packages which would be beneficial to be changed:

  • python3.XX (3.9, 3.10, 3.11, 3.12)
  • python3-dateutil
  • python3-pytz
  • libical

Upon acceptance of the change request we will file bugs to fix each of these packages for Fedora 39.

  • Release engineering: No changes needed.
  • Policies and guidelines: N/A (not needed for this Change)
  • Trademark approval: N/A (not needed for this Change)
  • Alignment with Community Initiatives:

Upgrade/compatibility impact

If tzdata is already installed, then it continues to be installed on the system.

Following the Fedora Weak Dependencies Policy dnf will treat the Recommends on tzdata as if it were a Requires and tzdata will always be installed in a default system (https://docs.fedoraproject.org/en-US/packaging-guidelines/WeakDependencies/)

However, the recommends will allow tzdata to be correctly uninstalled in a container build file rather than having to use ‘rm -rf’ to delete the zone files to recover space.


How To Test

Language runtimes were installed and A/B tests carried out with tzdata present and tzdata removed. The intent of these tests was to ensure that the system can use the language frameworks without tzdata present and that when the data was required that meaningful errors were presented to the user.

Packages tested were C (glibc), C++ (libstdc++), Python (Python 3.11), using their time and date APIs. This testing led to the correction of the libstdc++ implementation as noted here: https://gcc.gnu.org/cgit/gcc/commit/?id=4abd5bc600193e821fbc41995a0b8d9ea42b42c3

Developers can test this by installing Rawhide and uninstalling tzdata and verifying their package operates as expected.

If tzdata cannot be uninstalled then we recommend filling a bug against the package that Requires: tzdata and having a discussion with the maintainer to make tzdata optional e.g. Recommends: tzdata.


User Experience

The user can remove tzdata to minimize the container size.

With tzdata removed the system is UTC only.


Dependencies

Fixing glibc, gcc, and python3.xx at a minimum to make tzdata Recommends instead of Requires.


Contingency Plan

  • Contingency mechanism: Back out the change. Don’t do it.
  • Contingency deadline: Can be backed out at the last minute since we are only dealing with conversions from Requires to Recommends.
  • Blocks release? No

Documentation

Document that the tzdata package may be removed if needed to reduce space on containers. With tzdata removed, the system will default to UTC.

Release Notes

Retrieved from "https://fedoraproject.org/w/index.php?title=Changes/AllowRemovalOfTzdata&oldid=690543"