Skip to content
This repository has been archived by the owner on Oct 5, 2022. It is now read-only.

Update to latest sandbox version and remove some workarounds #23

Merged
merged 3 commits into from
Jan 22, 2021
Merged

Update to latest sandbox version and remove some workarounds #23

merged 3 commits into from
Jan 22, 2021

Conversation

invidian
Copy link
Contributor

Draft as there might be some workarounds which can be now removed.

Signed-off-by: Mateusz Gozdek <[email protected]>
This should no longer be happening with new version of Tinkerbell, so we
should return error when this occurs.

Signed-off-by: Mateusz Gozdek <[email protected]>
It seems they do not occur when creating anymore.

Signed-off-by: Mateusz Gozdek <[email protected]>
@invidian invidian changed the title Update to latest sandbox version Update to latest sandbox version and remove some workarounds Jan 15, 2021
@invidian invidian marked this pull request as ready for review January 15, 2021 16:30
@invidian invidian requested a review from displague January 15, 2021 16:30
@invidian
Copy link
Contributor Author

Once this is merged, I would like to make a release.

@invidian invidian added this to the v0.1.1 milestone Jan 15, 2021
Copy link
Member

@displague displague left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

I don't understand how go.mod got bigger when we've effectively removed code. Did the updated tink bring all of this in?

@invidian
Copy link
Contributor Author

I don't understand how go.mod got bigger when we've effectively removed code. Did the updated tink bring all of this in?

Yeah, they are most likely pulled together with tink.

@invidian invidian merged commit 2a7b83e into tinkerbell:master Jan 22, 2021
@invidian invidian deleted the invidian/update-to-latest-sandbox-version branch January 22, 2021 22:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants