-
Couldn't load subscription status.
- Fork 881
release v1.3.0 #2355
release v1.3.0 #2355
Conversation
CHANGELOG.md
Outdated
|
|
||
| #### Bug fixes | ||
|
|
||
| - Propagate exit status from apps inside the pod to rkt ([#2308](https://github.com/coreos/rkt/pull/2308)). |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
0b6f5a7 to
a3dcae3
Compare
|
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. |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
a3dcae3 to
52e1858
Compare
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. |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point. Will do.
52e1858 to
2ef4525
Compare
|
Updated. |
|
The patches look good to me. I am running more tests. |
|
Results of my tests:
|
2ef4525 to
cf66400
Compare
|
Rebased. |
|
Smething about all the dependency bumps? |
|
surely we picked up some bug fixes, etc in the CNI bump? @steveej |
cf66400 to
00e233f
Compare
|
Added something about docker2aci |
|
Let's polish the CHANGELOG later. |
|
LGTM |
❗