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

Add ITVersion for helping management the versions in IT #12895

Merged
merged 4 commits into from
Dec 23, 2024

Conversation

mrproliu
Copy link
Contributor

@mrproliu mrproliu commented Dec 23, 2024

Add a new IT module to help manage the versions in the IT. When some modules are needed to do IT, this module can be introduced to obtain the version. By default, it will first determine whether there is a configuration with an INTEGRATION_ prefix, if so, it will use that. Otherwise, it will use the default configuration.

  • If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
  • Update the CHANGES log.

@mrproliu mrproliu added the enhancement Enhancement on performance or codes label Dec 23, 2024
@mrproliu mrproliu added this to the 10.2.0 milestone Dec 23, 2024
wankai123
wankai123 previously approved these changes Dec 23, 2024
*/
public class ITVersions {

static Map<String, String> env = readingEnv("test", "e2e-v2", "script", "env");
Copy link
Member

Choose a reason for hiding this comment

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

Is this work in sub folder? This seems a path to project root, but we could run this IT in maven sub module path, or separately in ide.
Have you verified these cases?

Copy link
Contributor Author

@mrproliu mrproliu Dec 23, 2024

Choose a reason for hiding this comment

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

ITVersions will read the user.dir to get the current working directory, then find the directory where mvnw is located.

@mrproliu mrproliu merged commit 3601ba1 into apache:master Dec 23, 2024
167 checks passed
@mrproliu mrproliu deleted the integration-env-reader branch December 23, 2024 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement on performance or codes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants