Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix rsyslog_plugin UT with timestamp formatter #13241

Merged
merged 5 commits into from
Jan 4, 2023

Conversation

zbud-msft
Copy link
Contributor

@zbud-msft zbud-msft commented Jan 3, 2023

Why I did it

Timestamp formatter inside UT was failing due to new year change

How I did it

Use a const stored year that will used as expected value

How to verify it

Run UT

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

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

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

@zbud-msft zbud-msft marked this pull request as ready for review January 3, 2023 17:16
@zbud-msft zbud-msft requested a review from lguohan as a code owner January 3, 2023 17:16
@zbud-msft zbud-msft added Request for 202111 Branch For PRs being requested for 202111 branch Unit Tests labels Jan 3, 2023
Copy link
Contributor

@renukamanavalan renukamanavalan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a comment, on why we need this explicit declaration ?
It helps anyone new reading the code.

@qiluo-msft qiluo-msft enabled auto-merge (squash) January 3, 2023 22:25
@zbud-msft
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 13241 in repo sonic-net/sonic-buildimage

@qiluo-msft qiluo-msft disabled auto-merge January 4, 2023 22:55
@qiluo-msft qiluo-msft merged commit 90950d9 into sonic-net:master Jan 4, 2023
xumia pushed a commit to xumia/sonic-buildimage-1 that referenced this pull request Jan 5, 2023
#### Why I did it

Timestamp formatter inside UT was failing due to new year change

#### How I did it

Use a const stored year that will used as expected value

#### How to verify it

Run UT
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Jan 5, 2023
#### Why I did it

Timestamp formatter inside UT was failing due to new year change

#### How I did it

Use a const stored year that will used as expected value

#### How to verify it

Run UT
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202211: #13263

mssonicbld pushed a commit that referenced this pull request Jan 5, 2023
#### Why I did it

Timestamp formatter inside UT was failing due to new year change

#### How I did it

Use a const stored year that will used as expected value

#### How to verify it

Run UT
DavidZagury pushed a commit to DavidZagury/sonic-buildimage that referenced this pull request Jan 12, 2023
#### Why I did it

Timestamp formatter inside UT was failing due to new year change

#### How I did it

Use a const stored year that will used as expected value

#### How to verify it

Run UT
liorghub pushed a commit to liorghub/sonic-buildimage that referenced this pull request Jan 23, 2023
#### Why I did it

Timestamp formatter inside UT was failing due to new year change

#### How I did it

Use a const stored year that will used as expected value

#### How to verify it

Run UT
@zbud-msft zbud-msft deleted the fixTimestampFormatterYear branch March 8, 2023 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants