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 image and tag parameter for Dockerfile.Windows #108

Merged
merged 1 commit into from
Sep 1, 2020

Conversation

jingxu97
Copy link
Contributor

@jingxu97 jingxu97 commented Aug 31, 2020

Dockerfile.Windows has two container images, servercore and nanoserver,
use seperate parameter for them.

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:

/kind api-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

Update image and tag names for Windows to have separate parameters for nanoserver and servercore

Dockerfile.Windows has two container images, servercore and nanoserver,
use seperate parameter for them.
@k8s-ci-robot k8s-ci-robot added do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Aug 31, 2020
@k8s-ci-robot k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Aug 31, 2020
@msau42
Copy link
Collaborator

msau42 commented Aug 31, 2020

/lgtm
/approve
/hold
can you add a release note?

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 31, 2020
@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 31, 2020
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jingxu97, msau42

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Aug 31, 2020
@jingxu97
Copy link
Contributor Author

jingxu97 commented Sep 1, 2020

/lgtm
/approve
/hold
can you add a release note?

I added one. PTAL

@msau42
Copy link
Collaborator

msau42 commented Sep 1, 2020

lgtm, are you waiting on someone from the windows team to take a look? Otherwise you can cancel the hold if it's ready

@jingxu97
Copy link
Contributor Author

jingxu97 commented Sep 1, 2020

@ddebroy PTAL

@jingxu97
Copy link
Contributor Author

jingxu97 commented Sep 1, 2020

/cancel hold

@jingxu97
Copy link
Contributor Author

jingxu97 commented Sep 1, 2020

/hold cancel

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 1, 2020
@k8s-ci-robot k8s-ci-robot merged commit 19baf87 into kubernetes-csi:master Sep 1, 2020
@marosset
Copy link

marosset commented Sep 1, 2020

Looks good to me.
We should consider building this (and other windows csi-related images) for multiple Windows OS versions at some point too.

pohly added a commit to pohly/node-driver-registrar that referenced this pull request Oct 27, 2020
22c0395c Merge pull request kubernetes-csi#108 from bnrjee/master
b5b447b5 Add go ldflags using LDFLAGS at the time of compilation
16f4afbd Merge pull request kubernetes-csi#107 from pohly/kind-update
7bcee13d prow.sh: update to kind 0.9, support Kubernetes 1.19
df518fbd prow.sh: usage of Bazel optional
c3afd427 Merge pull request kubernetes-csi#104 from xing-yang/snapshot
dde93b22 Update to snapshot-controller v3.0.0

git-subtree-dir: release-tools
git-subtree-split: 22c0395c9cb16a65d47197b405a8cc3cc82fced9
pohly added a commit to pohly/node-driver-registrar that referenced this pull request Oct 27, 2020
4aff857d Merge pull request kubernetes-csi#109 from pohly/alpha-test-defaults
0427289d Merge pull request kubernetes-csi#110 from pohly/kind-0.9-bazel-build-workaround
9a370ab9 prow.sh: work around "kind build node-image" failure
522361ec prow.sh: only run alpha tests for latest Kubernetes release
22c0395c Merge pull request kubernetes-csi#108 from bnrjee/master
b5b447b5 Add go ldflags using LDFLAGS at the time of compilation
16f4afbd Merge pull request kubernetes-csi#107 from pohly/kind-update
7bcee13d prow.sh: update to kind 0.9, support Kubernetes 1.19
df518fbd prow.sh: usage of Bazel optional
c3afd427 Merge pull request kubernetes-csi#104 from xing-yang/snapshot
dde93b22 Update to snapshot-controller v3.0.0

git-subtree-dir: release-tools
git-subtree-split: 4aff857d88149e07951fcd1322f462f765401a86
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants