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 Windows support for toolbox image #27

Merged

Conversation

antoninbas
Copy link
Collaborator

We add Windows images to the antrea/toolbox manifest (for both the 1809 and ltsc2022 Windows nanoserver versions).

The Windows image is based directly on K8s pause image. It includes wincat, pause, and of course everything included by default in the nanoserver image (such as ping).

In the future, we may want to install additional utilities in the Windows image.

Because we share the same versioning between Linux and Windows, the toolbox version is bumped up to 1.3.

Fixes #26

We add Windows images to the antrea/toolbox manifest (for both the 1809
and ltsc2022 Windows nanoserver versions).

The Windows image is based directly on K8s pause image. It includes
wincat, pause, and of course everything included by default in the
nanoserver image (such as ping).

In the future, we may want to install additional utilities in the
Windows image.

Because we share the same versioning between Linux and Windows, the
toolbox version is bumped up to 1.3.

Fixes antrea-io#26

Signed-off-by: Antonin Bas <[email protected]>
@antoninbas antoninbas assigned tnqn and unassigned tnqn Jan 26, 2024
@antoninbas antoninbas requested a review from tnqn January 29, 2024 18:34
Copy link
Member

@tnqn tnqn left a comment

Choose a reason for hiding this comment

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

LGTM

echo "docker_tag=${version}-${rev}" >> $GITHUB_OUTPUT
outputs:
version: ${{ steps.get-outputs.outputs.version }}
rev: ${{ steps.get-outputs.outputs.rev }}
Copy link
Member

Choose a reason for hiding this comment

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

this seems not used

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yes, it's true but I didn't feel the need to remove it

Comment on lines +26 to +27
# We do not add any tools to the image at the moment.
# The image comes with pause + wincat + anything included by default in the nanoserver base image.
Copy link
Member

Choose a reason for hiding this comment

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

Does it have ping?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, I checked. It's included by default in the nanoserver image

@antoninbas antoninbas merged commit c164cf8 into antrea-io:main Jan 30, 2024
5 checks passed
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.

e2e: Windows e2e tests failed because toolbox has no windows versions
2 participants