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

Update test for milestone 2.18 #4237

Merged
merged 2 commits into from
Jun 26, 2024
Merged

Conversation

cidrblock
Copy link
Contributor

@cidrblock cidrblock commented Jun 25, 2024

For unknown reason, the behaviour of milestone differs such that the PWD environment variable is not updated to the directory of the process.

Because we are not running with a shell and it's the shell's responsibility to keep it updated, we'll switch instead to a lookup pipe to get the pwd and not rely on the env var.

After several hours of digging without success, this should accomplish what the test intended to do with a dependency on the shell.

@cidrblock cidrblock requested a review from a team as a code owner June 25, 2024 22:12
@github-actions github-actions bot added the bug label Jun 25, 2024
@cidrblock cidrblock changed the title Fix milestone Update test for milestone 2.18 Jun 25, 2024
@cidrblock
Copy link
Contributor Author

cidrblock commented Jun 25, 2024

pkg failure: #4236 (comment)

@ssbarnea
Copy link
Member

pkg job failing due to new twine release, see pypa/twine#977

@ssbarnea ssbarnea merged commit d5173cb into ansible:main Jun 26, 2024
16 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants