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

driver: remove unnecessary waitActionTimeout (v2.x) #430

Closed
wants to merge 7 commits into from

Conversation

waynr
Copy link
Contributor

@waynr waynr commented May 12, 2022

No description provided.

gottwald and others added 7 commits May 12, 2022 18:23
Newer versions of Docker seem to fail when using the default platform
since Darwin on amd64 has become a dedicated image architecture with
Docker:

  e2e_test.go:284: end-to-end tests failed: failed to create container: Error response from daemon: image with reference docker.io/digitalocean/k8s-e2e-test-runner:latest was found but does not match the specified platform: wanted darwin/amd64, actual: linux/amd64

Not setting the platform ensures the Linux image is requested.
@waynr waynr force-pushed the wwarren/2.x/handle-get-actions-404-responses branch from ba0b654 to 0e42076 Compare May 12, 2022 23:28
@waynr
Copy link
Contributor Author

waynr commented May 12, 2022

Note -- I also pulled in some CI and other harmless-looking changes from master branch.

@timoreimann timoreimann changed the title driver: remove unnecessary waitActionTimeout driver: remove unnecessary waitActionTimeout (v2.x) May 17, 2022
@timoreimann
Copy link
Contributor

We may end up not needing this after all; let's keep it open for now.

@timoreimann
Copy link
Contributor

The improvements will only go into v3+ of the driver. Closing this PR.

@timoreimann timoreimann closed this Jul 7, 2022
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.

4 participants