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

Fixes for upcoming issues during demo preparation + improvements discovered during execution of the demo flow #78

Merged
merged 18 commits into from
Sep 7, 2022

Conversation

mandelsoft
Copy link
Contributor

What this PR does / why we need it:

Which issue(s) this PR fixes:
Fixes #74

Special notes for your reviewer:

Release note:

@jensh007 jensh007 marked this pull request as ready for review August 31, 2022 06:00
@jensh007 jensh007 requested a review from a team as a code owner August 31, 2022 06:00
jensh007
jensh007 previously approved these changes Aug 31, 2022
@jschicktanz jschicktanz self-requested a review August 31, 2022 07:30
jschicktanz
jschicktanz previously approved these changes Aug 31, 2022
@gardener-robot gardener-robot added size/m Medium size/l Large and removed size/s Small size/m Medium labels Aug 31, 2022
@Skarlso
Copy link
Contributor

Skarlso commented Sep 1, 2022

Do I see it correctly that this has some unrelated changes to the actual fix of he dockerdaemon connection?

@gardener-robot gardener-robot added size/xl and removed size/l Large labels Sep 1, 2022
@Skarlso
Copy link
Contributor

Skarlso commented Sep 1, 2022

@mandelsoft
fix default compressed dir input mime type

How does this commit relate to the docker bug fix? If it's not related, would you mind please not polluting this PR further with unrelated changes? :) Thanks. :)

@mandelsoft mandelsoft changed the title Accessing the dockerdaemon on MAc using colima does not work Fixes for upcoming issues during demo preparation + improvements discovered during execution of the demo flow Sep 1, 2022
@yitsushi yitsushi dismissed stale reviews from jschicktanz and jensh007 via 22951ed September 1, 2022 14:42
@mandelsoft
Copy link
Contributor Author

mandelsoft commented Sep 2, 2022

So, the demo is prepared, we will not add further content here, and it is ready for review.

@mandelsoft mandelsoft dismissed stale reviews from yitsushi and Skarlso via ed8ded5 September 7, 2022 15:19
@mandelsoft
Copy link
Contributor Author

mandelsoft commented Sep 7, 2022

rebased and linted, no content change besides reordering the imports.

I would prefer to merge this one, before rebasing the localize PR, to avoid this additional rebase effort.

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.

OCM lib: Connection to docker-deamon on Mac does not work
6 participants