Skip to content
This repository has been archived by the owner on Feb 24, 2020. It is now read-only.

build: use rkt-builder ACI #2923

Merged
merged 1 commit into from
Jul 14, 2016
Merged

build: use rkt-builder ACI #2923

merged 1 commit into from
Jul 14, 2016

Conversation

s-urbaniak
Copy link
Contributor

Fixes #2435

Once https://github.com/coreos/rkt-builder v1.0.1 is released, this is mergeable.

@s-urbaniak
Copy link
Contributor Author

@tmrts as discussed OOB, do you mind to do a rkt-builder 1.0.1 release?

@tmrts tmrts self-assigned this Jul 13, 2016
@tmrts
Copy link
Contributor

tmrts commented Jul 13, 2016

It's in my queue for today, if not tomorrow at worst 👍

@s-urbaniak
Copy link
Contributor Author

@tmrts Don't sweat it, I'll do a release tomorrow morning.

@s-urbaniak s-urbaniak assigned s-urbaniak and unassigned tmrts Jul 13, 2016
@s-urbaniak s-urbaniak changed the title [WIP] build: use rkt-builder ACI build: use rkt-builder ACI Jul 14, 2016
@s-urbaniak
Copy link
Contributor Author

rkt-builder 1.0.1 landed, @squeed mind to have a look?

@s-urbaniak s-urbaniak assigned squeed and unassigned s-urbaniak Jul 14, 2016
@iaguis
Copy link
Member

iaguis commented Jul 14, 2016

LGTM

I did a test build and it worked. This is really great! :D

I think we can trim the builder ACI a bit, I'll open an issue in coreos/rkt-builder.

@iaguis
Copy link
Member

iaguis commented Jul 14, 2016

rkt/rkt-builder#1

@iaguis iaguis merged commit 6faf683 into rkt:master Jul 14, 2016
@s-urbaniak
Copy link
Contributor Author

@iaguis thanks! 👍

@squeed
Copy link
Contributor

squeed commented Jul 14, 2016

I think that git clean -ffdx should still be there?

@s-urbaniak
Copy link
Contributor Author

@squeed I don't think so, but let me recheck.

@s-urbaniak s-urbaniak deleted the rkt-in-rkt branch November 7, 2016 12:59
@lucab lucab unassigned squeed Apr 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants