Skip to content

OpTestKernelDump.py Add testcase for timezone change with kdump#968

Open
shirishaganta1 wants to merge 1 commit into
open-power:masterfrom
shirishaganta1:timezone-change-kdump-check
Open

OpTestKernelDump.py Add testcase for timezone change with kdump#968
shirishaganta1 wants to merge 1 commit into
open-power:masterfrom
shirishaganta1:timezone-change-kdump-check

Conversation

@shirishaganta1

Copy link
Copy Markdown
Contributor

Introduce a testcase that modifies the system timezone, triggers a crash dump, and verifies that the crash directory and generated files reflect the updated timezone setting.

Introduce a testcase that modifies the system timezone,
triggers a crash dump, and verifies that the crash directory
and generated files reflect the updated timezone setting.

Signed-off-by: Shirisha G <shirisha@linux.ibm.com>
@shirishaganta1

Copy link
Copy Markdown
Contributor Author

new-timezone-log.txt

s.addTest(PstoreCheck())
s.addTest(MeasureMakedumpTime())

s.addTest(TestTimezoneKdump())

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

can you pleae add this in op-test kdump suite as well

self.kernel_crash()
crash_dir = self.op_test_util.get_latest_crash_dir()
if not crash_dir:
raise Exception("No crash directory found")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

is this not a optesterror or fail ?

tz_line = [l for l in timedatectl_output if "Time zone" in l][0]
log.info(f"Post-crash timezone:{tz_line}")
if not timestamps:
raise Exception("No files found in crash directory")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

here as well we need to fail test

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