Skip to content

testcases: OpTestKernelDump.py Add testcase for timezone change with kdump#945

Open
shirishaganta1 wants to merge 1 commit intoopen-power:masterfrom
shirishaganta1:change-timezone
Open

testcases: OpTestKernelDump.py Add testcase for timezone change with kdump#945
shirishaganta1 wants to merge 1 commit intoopen-power:masterfrom
shirishaganta1:change-timezone

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.

…kdump

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

timezone-log.txt

@abdhaleegit abdhaleegit self-requested a review April 16, 2026 09:31
@abdhaleegit abdhaleegit self-assigned this Apr 16, 2026
Copy link
Copy Markdown
Collaborator

@abdhaleegit abdhaleegit left a comment

Choose a reason for hiding this comment

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

LGTM


def get_file_timestamps(self, crash_dir):
cmd = f"ls -l --time-style=full-iso {crash_dir}"
return self.c.run_command(cmd)
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.

all above 4 methods can go to OpTestUtils.py

raise Exception("No files found in crash directory")

# Optional strict validation: check offset
# Example: +05:30, -04:00 etc.
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.

remove all commented one lines

# Step 5: Trigger crash
self.kernel_crash()

# Step 6: Validate crash directory timestamps
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.

no one line comments..

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