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

Fix podman/docker config calling ddtool #1294

Merged
merged 2 commits into from
Dec 13, 2024
Merged

Conversation

CelianR
Copy link
Contributor

@CelianR CelianR commented Dec 13, 2024

What does this PR do?

When executing inv localpodman.create-vm, it was possible to obtain a timeout on vault with ddtool hook.
Replaced the config by a custom empty config.

Note

Do we want to be able to customize the config? Maybe it is better to provide an argument to replace by a custom config

Which scenarios this will impact?

Motivation

Additional Notes

@CelianR CelianR self-assigned this Dec 13, 2024
@CelianR CelianR requested a review from a team as a code owner December 13, 2024 09:16
resources/local/podman/vm.go Show resolved Hide resolved
@CelianR
Copy link
Contributor Author

CelianR commented Dec 13, 2024

/merge

@dd-devflow
Copy link

dd-devflow bot commented Dec 13, 2024

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-12-13 10:05:40 UTC ℹ️ MergeQueue: pull request added to the queue

The median merge time in main is 2m.


2024-12-13 10:11:39 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue dd-mergequeue bot merged commit 28fc639 into main Dec 13, 2024
9 checks passed
@dd-mergequeue dd-mergequeue bot deleted the celian/fix-podman-config branch December 13, 2024 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants