Thanks to visit codestin.com
Credit goes to github.com

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

Conversation

@iaguis
Copy link
Member

@iaguis iaguis commented Mar 31, 2016

CHANGELOG.md Outdated

#### Bug fixes

- Propagate exit status from apps inside the pod to rkt ([#2308](https://github.com/coreos/rkt/pull/2308)).
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this a bug fix? Seems like a kool feature?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, we were undecided about it being a feature or a bug fix... I think it's a bit of both

Copy link
Member Author

Choose a reason for hiding this comment

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

I can move it to features, it looks kooler.

@iaguis iaguis force-pushed the iaguis/release-v1.3.0 branch from 0b6f5a7 to a3dcae3 Compare March 31, 2016 16:21
@iaguis
Copy link
Member Author

iaguis commented Mar 31, 2016

Updated.

CHANGELOG.md Outdated
- The stage1 image is in the default directory configured at build time
- The stage1 image is the default one and it is in the same directory as the rkt binary
- Allow downloading of insecure public keys with the `pubkey` insecure option ([#2278](https://github.com/coreos/rkt/pull/2278)).
- Implement Docker volume semantics ([#2315](https://github.com/coreos/rkt/pull/231://github.com/coreos/rkt/pull/2315)). Docker volumes are initialized with the files in the image if they exist, unless a host directory is mounted there. Implement that behavior in rkt when it runs a Docker converted image.
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we have a doc on this?

Copy link
Member Author

Choose a reason for hiding this comment

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

Not really, but we print a message when this happens

Copy link
Member

Choose a reason for hiding this comment

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

The link to #2278 is invalid, it does not point to the correct URL

@iaguis iaguis force-pushed the iaguis/release-v1.3.0 branch from a3dcae3 to 52e1858 Compare March 31, 2016 16:24
CHANGELOG.md Outdated

#### New features and UX changes

- Propagate exit status from apps inside the pod to rkt ([#2308](https://github.com/coreos/rkt/pull/2308)). Previously, if an app exited with a non-zero exit status, rkt's exit status would still be 0. Now, if an app fails, its exit status will be propagated to the outside.
Copy link
Member

Choose a reason for hiding this comment

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

Since it was already mentioned in v1.1.0's changelog, you could add something like:

This was partially implemented in some stage1 flavors since rkt v1.1.0. This now works in the default coreos flavor.

Copy link
Member Author

Choose a reason for hiding this comment

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

Good point. Will do.

@iaguis iaguis force-pushed the iaguis/release-v1.3.0 branch from 52e1858 to 2ef4525 Compare April 1, 2016 09:48
@iaguis
Copy link
Member Author

iaguis commented Apr 1, 2016

Updated.

@alban
Copy link
Member

alban commented Apr 1, 2016

The patches look good to me. I am running more tests.

@alban
Copy link
Member

alban commented Apr 1, 2016

Results of my tests:

    rkt_tests.go:214: Error: ExpectRegex didn't find regex 'sha512-[0-9a-f]{32,64}'.
        Output: image: searching for app image coreos.com/etcd
        image: remote fetching from URL "https://github.com/coreos/etcd/releases/download/v2.1.2/etcd-v2.1.2-linux-amd64.aci"
fetch: unexpected EOF=                                         ] 589 KB/5.61 MB  0 B/5.61 MB

@jonboulle jonboulle added this to the v1.3.0 milestone Apr 1, 2016
@iaguis iaguis force-pushed the iaguis/release-v1.3.0 branch from 2ef4525 to cf66400 Compare April 1, 2016 13:49
@iaguis
Copy link
Member Author

iaguis commented Apr 1, 2016

Rebased.

@jonboulle
Copy link
Contributor

Smething about all the dependency bumps?

@jonboulle
Copy link
Contributor

surely we picked up some bug fixes, etc in the CNI bump? @steveej

@iaguis iaguis force-pushed the iaguis/release-v1.3.0 branch from cf66400 to 00e233f Compare April 1, 2016 13:57
@iaguis
Copy link
Member Author

iaguis commented Apr 1, 2016

Added something about docker2aci

@iaguis
Copy link
Member Author

iaguis commented Apr 1, 2016

Let's polish the CHANGELOG later.

@alban
Copy link
Member

alban commented Apr 1, 2016

LGTM

@iaguis iaguis merged commit b6a73a7 into rkt:master Apr 1, 2016
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.

3 participants