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.

Sign up for GitHub

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

Jump to bottom

CSI E2E test failure #60803

Closed
sbezverk opened this issue Mar 5, 2018 · 2 comments
Closed

CSI E2E test failure #60803

sbezverk opened this issue Mar 5, 2018 · 2 comments
Assignees
@sbezverk
Labels
kind/bug Categorizes issue or PR as related to a bug. kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. sig/storage Categorizes an issue or PR as relevant to SIG Storage.
Milestone
v1.10

Comments

@sbezverk
Copy link
Contributor

sbezverk commented Mar 5, 2018

After switching to CSI spec 0.2.0 extra permissions are required.
/kind bug
/assign @sbezverk
/sig storager
@k8s-ci-robot k8s-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Mar 5, 2018
@sbezverk sbezverk mentioned this issue Mar 5, 2018
Merged
@saad-ali saad-ali added priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. sig/storage Categorizes an issue or PR as relevant to SIG Storage. kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test. labels Mar 5, 2018
@saad-ali saad-ali added this to the v1.10 milestone Mar 5, 2018
@k8s-github-robot
Copy link

[MILESTONENOTIFIER] Milestone Issue: Up-to-date for process

@sbezverk

Note: This issue is marked as priority/critical-urgent, and must be updated every 1 day during code freeze.

Example update:

ACK.  In progress
ETA: DD/MM/YYYY
Risks: Complicated fix required
Issue Labels
  • sig/storage: Issue will be escalated to these SIGs if needed.
  • priority/critical-urgent: Never automatically move issue out of a release milestone; continually escalate to contributor and SIG through all available channels.
  • kind/bug: Fixes a bug discovered during the current release.
Help

k8s-github-robot pushed a commit that referenced this issue Mar 5, 2018
Merge pull request #60804 from sbezverk/e2e_csi_test
f68eb1e 
Automatic merge from submit-queue (batch tested with PRs 60679, 60804). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Fixing e2e CSI test

Closes #60803

After switching to CSI 0.2.0 spec, additional RBAC permissions are required.  This PR adds missing permissions.

```release-note
None
```
pohly added a commit to pohly/kubernetes that referenced this issue Mar 6, 2018
@pohly
Fixing e2e CSI test, II
17d9a0c 
The fix for kubernetes#60803 in commit 2ae33cc had a typo, so the "Server
rejected event" error still showed up in the external-provisioner log
of the "Sanity CSI plugin test using hostPath CSI driver" e2e test.
@pohly pohly mentioned this issue Mar 6, 2018
Merged
@pohly
Copy link
Contributor

pohly commented Mar 6, 2018

This isn't fixed (see PR #60840) and even with that PR and kubernetes-csi/external-provisioner#52 the e2e test is still failing for me.

k8s-github-robot pushed a commit that referenced this issue Mar 6, 2018
Merge pull request #60840 from pohly/e2e_csi_create
86ca3af 
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Fixing e2e CSI test, II

The fix for #60803 in commit 2ae33cc had a typo, so the "Server
rejected event" error still showed up in the external-provisioner log
of the "Sanity CSI plugin test using hostPath CSI driver" e2e test.

```release-note
None
```
jingxu97 pushed a commit to jingxu97/kubernetes that referenced this issue Mar 13, 2018
@pohly @jingxu97
Fixing e2e CSI test, II
1732e04 
The fix for kubernetes#60803 in commit 2ae33cc had a typo, so the "Server
rejected event" error still showed up in the external-provisioner log
of the "Sanity CSI plugin test using hostPath CSI driver" e2e test.
prameshj pushed a commit to prameshj/kubernetes that referenced this issue Jun 1, 2018
@pohly @prameshj
Fixing e2e CSI test, II
132bf8a 
The fix for kubernetes#60803 in commit 2ae33cc had a typo, so the "Server
rejected event" error still showed up in the external-provisioner log
of the "Sanity CSI plugin test using hostPath CSI driver" e2e test.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sbezverk sbezverk

Labels
kind/bug Categorizes issue or PR as related to a bug. kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. sig/storage Categorizes an issue or PR as relevant to SIG Storage.
Projects
None yet
Milestone
v1.10
Development

No branches or pull requests
5 participants
@pohly @saad-ali @k8s-github-robot @sbezverk @k8s-ci-robot