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

cheking UID's in windows, follow on (quick pr) #96512

Open
jayunit100 opened this issue Nov 12, 2020 · 10 comments · May be fixed by #96616
Open

cheking UID's in windows, follow on (quick pr) #96512

jayunit100 opened this issue Nov 12, 2020 · 10 comments · May be fixed by #96616

Comments

@jayunit100
Copy link
Member

@jayunit100 jayunit100 commented Nov 12, 2020

What happened:

follow on to #96172

Need to comment and avoid checking UID on windows. requires some research into what we should be checking so that we can add appropriate TODO.

What you expected to happen:

Lets also check this at the call site,

if err := checkPermissions(); err != nil {

Anything else we need to know?:

Environment:

  • Kubernetes version (use kubectl version): master
  • OS (e.g: cat /etc/os-release): windows
@k8s-ci-robot
Copy link
Contributor

@k8s-ci-robot k8s-ci-robot commented Nov 12, 2020

@jayunit100: This issue is currently awaiting triage.

If a SIG or subproject determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@perithompson
Copy link

@perithompson perithompson commented Nov 12, 2020

/assign

@jayunit100
Copy link
Member Author

@jayunit100 jayunit100 commented Nov 12, 2020

/assign @perithompson

@jayunit100
Copy link
Member Author

@jayunit100 jayunit100 commented Nov 12, 2020

/good-first-issue

@k8s-ci-robot
Copy link
Contributor

@k8s-ci-robot k8s-ci-robot commented Nov 12, 2020

@jayunit100:
This request has been marked as suitable for new contributors.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-good-first-issue command.

In response to this:

/good-first-issue

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@neolit123
Copy link
Member

@neolit123 neolit123 commented Nov 13, 2020

/sig windows

@k8s-ci-robot k8s-ci-robot added sig/windows and removed needs-sig labels Nov 13, 2020
@jlsong01
Copy link

@jlsong01 jlsong01 commented Nov 15, 2020

Hi @perithompson are you working on this issue already? If not I'd like to tackle this issue :)

@perithompson
Copy link

@perithompson perithompson commented Nov 16, 2020

Hi @jlsong01 yeah I am already working on this :)

@jayunit100
Copy link
Member Author

@jayunit100 jayunit100 commented Nov 16, 2020

thanks @perithompson

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

Successfully merging a pull request may close this issue.

5 participants
You can’t perform that action at this time.