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

feat(docker-network): docker.network now supports any value #9390

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

idsulik
Copy link
Contributor

@idsulik idsulik commented Apr 17, 2024

Fixes: #9389

Description
Removed docker network validation, now you can pass any network

User facing changes (remove if N/A)
before: docker.network used to support only predefined values.
before: docker.network supports any value.

@idsulik idsulik requested a review from a team as a code owner April 17, 2024 07:48
@idsulik idsulik requested a review from JeromeJu April 17, 2024 07:48
@@ -1,3 +1,5 @@
module github.com/hashicorp/hcl

go 1.22
Copy link
Contributor Author

Choose a reason for hiding this comment

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

to fix this issue #9352 (comment)

@ericzzzzzzz
Copy link
Contributor

Hi @idsulik , thank you for the pr, I think we might not need a new schema version.

@idsulik
Copy link
Contributor Author

idsulik commented Apr 17, 2024

@ericzzzzzzz , agree. replaced the commit(it was easy to replace rather than update the current)

@ericzzzzzzz ericzzzzzzz added the kokoro:run runs the kokoro jobs on a PR label Apr 17, 2024
@kokoro-team kokoro-team removed the kokoro:run runs the kokoro jobs on a PR label Apr 17, 2024
@ericzzzzzzz ericzzzzzzz merged commit f69774e into GoogleContainerTools:main Apr 17, 2024
14 checks passed
@idsulik
Copy link
Contributor Author

idsulik commented Apr 24, 2024

@ericzzzzzzz hi! When the release is scheduled?

@ericzzzzzzz
Copy link
Contributor

Hi, @idsulik We haven't set a specific release date yet, but we expect it to be available sometime next month.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow any network to be provided to docker builder
3 participants