8000 Release v1.4.0 by s-urbaniak · Pull Request #2431 · rkt/rkt · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Feb 24, 2020. It is now read-only.

Release v1.4.0 #2431

Merged
merged 5 commits into from
Apr 15, 2016
Merged

Release v1.4.0 #2431

merged 5 commits into from
Apr 15, 2016

Conversation

s-urbaniak
Copy link
Contributor

No description provided.

@@ -455,7 +455,7 @@ func TestNetTemplates(t *testing.T) {
Type: "ptp",
Ipam: ipamTemplateT{
Type: "host-local",
Subnet: "11.11.3.0/24",
Subnet: "11.11.4.0/24",
Copy link
Member

Choose a reason for hiding this comment

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

oops, the script mistakenly changed this IP

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed


#### New features and UX changes

- Add `--user`/`--group` option to rkt run/prepare ([#2419](https://github.com/coreos/rkt/pull/2419)). This option allows overriding the user/group specified in the image manifest.
- config: add config subcommand ([2405](https://github.com/coreos/rkt/pull/2405)). This new subcommand prints the current rkt configuration. It can be used to get i.e. authentication credentials.
Copy link
Member

Choose a reason for hiding this comment

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

In the previous release notes, we used a # before the number. Example:

(#2405)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

- Ensure that the initial name and labels used for discovery match the name and labels in the Image Manifest as specified in the appc spec ([#2311](https://github.com/coreos/rkt/pull/2311)). Users wanting the latest image should use `rkt prepare/run/fetch example.com/aci` without any labels. If the discovery server supports the "latest" pattern, the user can bypass a locally cached image in the store and fetch an updated image using `rkt prepare/run/fetch --no-store example.com/aci` option.
- run: add user/group app flags ([2419](https://github.com/coreos/rkt/pull/2419)). This adds the `--user`/`--group` options to rkt run/prepare allowing to override the user/group specified in the image manifest.

- gc: Add flag 'mark-only' to mark garbage pods without deleting them ([2400](https://github.com/coreos/rkt/pull/2400), [2402](https://github.com/coreos/rkt/pull/2402)). This new flag moves exited/aborted pods to the exited-garbage/garbage directory but does not delete them. A third party application can use `rkt gc --mark- to mark the time when a pod is not running.
Copy link
Member

Choose a reason for hiding this comment

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

I'd say "to mark exited pods as garbage without deleting them"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed


- gc: Add flag 'mark-only' to mark garbage pods without deleting them ([2400](https://github.com/coreos/rkt/pull/2400), [2402](https://github.com/coreos/rkt/pull/2402)). This new flag moves exited/aborted pods to the exited-garbage/garbage directory but does not delete them. A third party application can use `rkt gc --mark- to mark the time when a pod is not running.

- kvm: Add support for capabilities limitation ([2222](https://github.com/coreos/rkt/pull/2222)). By default kvm flavor has got enabled every capability inside pod. This patch adds support for a restricted set of capabilities inside a rkt-kvm pod.
Copy link
Member

Choose a reason for hiding this comment

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

"Add support for app capabilities limitation"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@iaguis
Copy link
Member
iaguis commented Apr 15, 2016

I don't see #2402 mentioned on the changelog


#### Improved documentation

- documentation: remove superfluous dash ([#2413](https://github.com/coreos/rkt/pull/2413)).
Copy link
Member

Choose a reason for hiding this comment

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

Usually we're not that detailed with small changes. We just say doc improvements or something like that.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed, squashed in one line


- kvm: Add support for app capabilities limitation ([#2222](https://github.com/coreos/rkt/pull/2222)). By default kvm flavor has got enabled every capability inside pod. This patch adds support for a restricted set of capabilities inside a rkt-kvm pod.

- kvm: new volumes support ([#2328](https://github.com/coreos/rkt/pull/2328)). This provides new volumes support based on bind mounting volumes in time of rootfs preparation for the kvm flavor.
Copy link
Member

Choose a reason for hiding this comment

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

oh, right, mounting single files as volumes was not working: bind-mounting files works on Linux but mounting regular files as a p9 mount does not work. The new implementation makes it work.

Copy link
Member

Choose a reason for hiding this comment

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

I suggest:

kvm: refactor volumes support (#2328). This allows users to share regular files as volumes in addition to directories.

Copy link
Member

Choose a reason for hiding this comment

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

I think this should be in the "bug fix" section.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@s-urbaniak
Copy link
Contributor Author


- documentation: remove superfluous dash ([#2413](https://github.com/coreos/rkt/pull/2413)).

- functional tests: Add new test with systemd-proxyd ([#2257](https://github.com/coreos/rkt/pull/2257)). Adds a new test and documentation how to use systemd-proxyd with rkt pods.
Copy link
Member

Choose a reason for hiding this comment

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

Btw, my tests yesterday (https://gist.github.com/alban/b79c699e60ee7ce5ade64e167d418bb1) show that it is not working on old distributions with an old systemd. We should report a bug about this to disable the tests when systemd-proxyd is not installed or something.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed below

@s-urbaniak s-urbaniak force-pushed the release-v1.4.0 branch 2 times, most recently from 818ca5b to e92c3b0 Compare April 15, 2016 10:25

- rkt: calculate real dataDir path ([#2399](https://github.com/coreos/rkt/pull/2399)). Fixes garbage collection when the data directory specified by `--dir` contains a symlink component.

- stage1/init: fix docker volume semantics ([#2409](https://github.com/coreos/rkt/pull/2409)). Fixes a bug in docker volume semantics when rkt runs with the option `--pod-manifest`.
Copy link
Member

Choose a reason for hiding this comment

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

Should mention what is this docker volume semantic we are talking about.

When a Docker image exposes a mount point that is not mounted by a host volume, Docker volume semantics expects the files in the directory to be available to the application. This was partially fixed in rkt 1.3.0 via #2315 but the bug remained when rkt runs with the option --pod-manifest. This is now fully fixed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@s-urbaniak s-urbaniak force-pushed the release-v1.4.0 branch 3 times, most recently from 692c1bb to d91f27d Compare April 15, 2016 11:59

#### New features and UX changes

- Add `--user`/`--group` option to rkt run/prepare ([#2419](https://github.com/coreos/rkt/pull/2419)). This option allows overriding the user/group specified in the image manifest.
- config: add config subcommand ([#2405](https://github.com/coreos/rkt/pull/2405)). This new subcommand prints the current rkt configuration. It can be used to get i.e. authentication credentials.
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

See rkt's config subcommand documentation.

Copy link
Contributor Author< F438 /span>

Choose a reason for hiding this comment

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

fixed


- rkt: calculate real dataDir path ([#2399](https://github.com/coreos/rkt/pull/2399)). Fixes garbage collection when the data directory specified by `--dir` contains a symlink component.

- stage1/init: fix docker volume semantics ([#2409](https://github.com/coreos/rkt/pull/2409)). Fixes a bug in docker volume semantics when rkt runs with the option `--pod-manifest`. When a Docker image exposes a mount point that is not mounted by a host volume, Docker volume semantics expect the files in the directory to be available to the application. This was partially fixed in rkt 1.3.0 via ([#2315](https://github.com/coreos/rkt/pull/2315)) but the bug remained when rkt runs with the option `--pod-manifest`. This is now fully fixed.
Copy link
Member

Choose a reason for hiding this comment

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

either put "via" in the parenthesis, or remove the parenthesis


- rkt/image: check that discovery labels match manifest labels ([#2311](https://github.com/coreos/rkt/pull/2311)).

- store: fix comment + add link in db test ([#2407](https://github.com/coreos/rkt/pull/2407)).
Copy link
Member

Choose a reason for hiding this comment

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

This PR is probably not worth mentioning...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed

@s-urbaniak s-urbaniak force-pushed the release-v1.4.0 branch 2 times, most recently from 9d80f76 to 00569d7 Compare April 15, 2016 12:56

- rkt/image: check that discovery labels match manifest labels ([#2311](https://github.com/coreos/rkt/pull/2311)).

- store: fix multi process with multi goroutines race on db. ([#2391](https://github.com/coreos/rkt/pull/2391)). This was a bug when multiple `rkt fetch` commands were executed concurrently.
Copy link
Member

Choose a reason for hiding this comment

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

Remove the . before the ([#2391](https://github.com/coreos/rkt/pull/2391)).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

Sergiusz Urbaniak added 5 commits April 15, 2016 16:21
Currently `build-rir.sh` inherits the environment during the build. This
is unnecessary and can actually break the build on a custom host
PATH environment.

This fixes it.
@s-urbaniak s-urbaniak self-assigned this Apr 15, 2016
@s-urbaniak s-urbaniak added this to the v1.4.0 milestone Apr 15, 2016
@iaguis
Copy link
Member
iaguis commented Apr 15, 2016

LGTM

@s-urbaniak s-urbaniak merged commit 02438c5 into rkt:master Apr 15, 2016
@jonboulle jonboulle mentioned this pull request Apr 15, 2016
@s-urbaniak s-urbaniak deleted the release-v1.4.0 branch April 20, 2016 06:59
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
0