Skip to content

[Bookworm] [TZ] Fix timezone propagation to containers#118

Closed
vivekrnv wants to merge 2 commits intomasterfrom
fix_tz
Closed

[Bookworm] [TZ] Fix timezone propagation to containers#118
vivekrnv wants to merge 2 commits intomasterfrom
fix_tz

Conversation

@vivekrnv
Copy link
Owner

@vivekrnv vivekrnv commented Dec 5, 2024

Why I did it

Fix https://github.com/sonic-net/sonic-buildimage/issues/20925

How I did it

Mount /etc/localtime on containers and remove /etc/timezone dependency

How to verify it

root@sonic:/home/admin# zdump /etc/localtime
/etc/localtime  Fri Dec  6 23:24:03 2024 IST

root@sonic:/home/admin# docker exec swss zdump /etc/localtime
/etc/localtime  Fri Dec  6 23:24:12 2024 IST

Verify swss.rec/sairedis.rec and syslog are following same time

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Vivek <vkarri@nvidia.com>
@vivekrnv vivekrnv closed this Dec 6, 2024
@vivekrnv vivekrnv deleted the fix_tz branch December 19, 2024 06:15
vivekrnv pushed a commit that referenced this pull request Nov 12, 2025
…ically (sonic-net#24435)

#### Why I did it
src/sonic-dash-ha
```
* 149774b - (HEAD -> master, origin/master, origin/HEAD) Add a PR template (#129) (3 days ago) [yue-fred-gao]
* 711a0be - Sort the output of show actor command (#128) (3 days ago) [yue-fred-gao]
* 15fbd5c - Fix format for protobuf fields in show command. (#127) (3 days ago) [dypet]
* f2d47b9 - Add wait for loopback script. (#126) (3 days ago) [dypet]
* 570db05 - Fix issue #118: use hostname to build service path (#122) (3 days ago) [yue-fred-gao]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants