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

docs : Automate release notes #410

Merged

Conversation

jcvenegas
Copy link
Contributor

This patch adds a script to generate release notes.

Usage
./docs/release_notes.sh <start-commit/tag>

start-commit/tag is the commit to start to get git changes.

Example:

$./docs/release_notes.sh 3.0.0-alpha.5 3.0.0.beta.1

Fixes #409

Signed-off-by: Jose Carlos Venegas Munoz [email protected]

@jcvenegas
Copy link
Contributor Author

Example running:

 ./docs/release_notes.sh 3.0.0-alpha.5 3.0.0.beta.1

Release 3.0.0.beta.1

Changes

  • qemu: Move from pc-lite to pc machine type
  • docs: Document limitations.
  • config: support kernel parameters
  • build: configuration file depends on build variables

Shortlog

8a38fa7 qemu: Move from pc-lite to pc machine type
56e96d3 docs: Document limitations.
528772f config: support kernel parameters
2c456d7 build: configuration file depends on build variables

Compatibility with Docker

Clear Containers 3.0.0.beta.1 is compatible with Docker v17.06-ce

OCI Runtime Specification

Clear Containers 3.0.0.beta.1 support the OCI Runtime Specification 1.0.0-rc5

Clear Linux Containers image

Clear Containers 3.0.0.beta.1 requires at least Clear Linux containers image 16910

Clear Linux Containers Kernel

Clear Containers 3.0.0.beta.1 requires at least Clear Linux Containers kernel cc-linux-4.9.35

Installation

Issues & limitations

Networking

Adding networks dynamically

Resource management

docker run -m

See issue #381 for more information.

docker run --cpus=

See issue #341 for more information.

docker run --kernel-memory=

See issue #388 for more information.

shm

cgroup constraints

Capabilities

See issue #51 for more information.

sysctl

tmpfs

Other

checkpoint and restore

docker stats

See issue #200 for more information.

runtime commands

ps command

See issue #95 for more information.

events command

See issue #379 for more information.

update command

See issue #380 for more information.

Networking

Support for joining an existing VM network

docker --net=host

docker run --link

Host resource sharing

docker --device

docker -v /dev/...

docker run --privileged

Other

Annotations

runtime commands

init command

spec command

More information Limitations

@coveralls
Copy link

coveralls commented Aug 11, 2017

Coverage Status

Coverage remained the same at 55.39% when pulling d5bec43 on jcvenegas:jcvenegas/automate-release-notes into 2eeb4f4 on clearcontainers:master.

@sboeuf
Copy link
Contributor

sboeuf commented Aug 11, 2017

@jcvenegas I like this kind of patch :)
Good thing to automate the release.

@grahamwhaley
Copy link
Contributor

grahamwhaley commented Aug 11, 2017

I think this is a good thing (tm) :-)
I will add the note - once we have run the script we should never blindly trust the output - we still need to visually check and potentially massage the data (sometimes commit messages can be trimmed or improved or don't need to belong in the release notes for instance).
And, a note to all, now you have even more motivation to be vigilant on your commit/merge subject lines, as they are likely to end up in the release notes.
lgtm
adding @jodh-intel as a reviewer.

Approved with PullApprove Approved with PullApprove

Copy link
Contributor

@jodh-intel jodh-intel left a comment

Choose a reason for hiding this comment

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

Just needs a header on the script and I'm happy with this.

@@ -0,0 +1,92 @@
#!/bin/bash

Copy link
Contributor

Choose a reason for hiding this comment

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

Missing copyright header.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added


script_dir=$(dirname "$0")

source "${script_dir}/../versions.txt"
Copy link
Contributor

Choose a reason for hiding this comment

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

This file gets everywhere! 😄

## Installation
- [Ubuntu][ubuntu]
- [Fedora][fedora]
- [Developers][developers]
Copy link
Contributor

Choose a reason for hiding this comment

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

(not related to this PR [yet], but it'll be nice to get CentOS/RHEL added here when #372 lands).

@jcvenegas jcvenegas force-pushed the jcvenegas/automate-release-notes branch from d5bec43 to 1527806 Compare August 11, 2017 13:05
@coveralls
Copy link

coveralls commented Aug 11, 2017

Coverage Status

Coverage remained the same at 55.39% when pulling 1527806 on jcvenegas:jcvenegas/automate-release-notes into 2eeb4f4 on clearcontainers:master.

@jcvenegas jcvenegas force-pushed the jcvenegas/automate-release-notes branch from 1527806 to fbd04de Compare August 11, 2017 19:02
@coveralls
Copy link

coveralls commented Aug 11, 2017

Coverage Status

Coverage remained the same at 55.39% when pulling fbd04de on jcvenegas:jcvenegas/automate-release-notes into 5dea7cb on clearcontainers:master.

@sboeuf
Copy link
Contributor

sboeuf commented Aug 11, 2017

Please rebase the PR so that it runs with the new agent code.

@devimc
Copy link

devimc commented Aug 14, 2017

lgtm

Approved with PullApprove Approved with PullApprove

This patch adds a script to generate release notes.

Usage
./docs/release_notes.sh <start-commit/tag> <New-version>

start-commit/tag is the commit to start to get git changes.

Example:

$./docs/release_notes.sh 3.0.0-alpha.5 3.0.0.beta.1

Fixes clearcontainers#409

Signed-off-by: Jose Carlos Venegas Munoz <[email protected]>
@jcvenegas jcvenegas force-pushed the jcvenegas/automate-release-notes branch from fbd04de to 4674607 Compare August 14, 2017 18:16
@coveralls
Copy link

coveralls commented Aug 14, 2017

Coverage Status

Coverage remained the same at 55.39% when pulling 4674607 on jcvenegas:jcvenegas/automate-release-notes into 7bde8a9 on clearcontainers:master.

@jodh-intel
Copy link
Contributor

jodh-intel commented Aug 16, 2017

lgtm

Approved with PullApprove Approved with PullApprove

@jodh-intel jodh-intel merged commit 743362f into clearcontainers:master Aug 16, 2017
mcastelino pushed a commit to mcastelino/runtime that referenced this pull request Dec 6, 2018
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.

6 participants