Skip to content

[tests] Add testltp program to enable testing with newer LTP versions - #2118

Draft
efu39 wants to merge 1 commit into
gramineproject:masterfrom
efu39:erica/testltp
Draft

[tests] Add testltp program to enable testing with newer LTP versions#2118
efu39 wants to merge 1 commit into
gramineproject:masterfrom
efu39:erica/testltp

Conversation

@efu39

@efu39 efu39 commented Mar 28, 2025

Copy link
Copy Markdown
Contributor

Description of the changes

This is a draft PR, to get input if there is interest to merge it.

With the LTP 1909cd5675 change, the command gramine-{direct|sgx} <TEST_BINARY> does not work with LTP releases after 20220527, failing with “TFAIL: Main test process might have exit”
The commit introduces testltp program, allowing newer versions of LTP to be tested using:

gramine-{direct|sgx} testltp <TEST_BINARY>  

Additionally, testing with pytest can be done by setting the LTP_ENV variable, for example:

 LTP_ENV=testltp python3 -m pytest -v -k chmod01
 LTP_ENV=testltp make [SGX=1] regression

The change enables testing with newer LTP versions for development purposes only. The LTP version in use remains unchanged.

How to test this PR?

For the current LTP version, LTP test behave as before and can be run with:make [SGX=1] regression
Tested with LTP version 20240930 using: LTP_ENV=testltp make [SGX=1] regression


This change is Reviewable

@kailun-qin kailun-qin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@efu39: Can we discuss this in the next core Gramine meeting?

Reviewable status: 0 of 4 files reviewed, all discussions resolved, not enough approvals from maintainers (2 more required), not enough approvals from different teams (2 more required, approved so far: )

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