Skip to content

Switch all our tests to version 2 of config.toml#5335

Merged
AkihiroSuda merged 2 commits intocontainerd:masterfrom
dims:drop-support-for-version-1
Apr 12, 2021
Merged

Switch all our tests to version 2 of config.toml#5335
AkihiroSuda merged 2 commits intocontainerd:masterfrom
dims:drop-support-for-version-1

Conversation

@dims
Copy link
Copy Markdown
Member

@dims dims commented Apr 11, 2021

We are telling folks to switch to version 2, so we should do this ourselves as well :)

Signed-off-by: Davanum Srinivas [email protected]

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 11, 2021

Build succeeded.

@dims dims force-pushed the drop-support-for-version-1 branch from fb8c892 to 17b3e6f Compare April 12, 2021 00:23
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 12, 2021

Build succeeded.

@dims dims force-pushed the drop-support-for-version-1 branch 3 times, most recently from e68865a to 44a98a7 Compare April 12, 2021 02:41
@AkihiroSuda
Copy link
Copy Markdown
Member

This shouldn't happen until containerd v2.0 IMO.

Not for v1.5, at least.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 12, 2021

Build succeeded.

@AkihiroSuda AkihiroSuda added this to the 2.0 milestone Apr 12, 2021
@AkihiroSuda AkihiroSuda marked this pull request as draft April 12, 2021 04:19
@dims dims force-pushed the drop-support-for-version-1 branch from 44a98a7 to fcc052b Compare April 12, 2021 13:48
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 12, 2021

Build succeeded.

@dims dims force-pushed the drop-support-for-version-1 branch from fcc052b to 04b704e Compare April 12, 2021 14:13
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 12, 2021

Build succeeded.

@dims dims force-pushed the drop-support-for-version-1 branch from 04b704e to 2eabd28 Compare April 12, 2021 14:47
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 12, 2021

Build succeeded.

@dims dims force-pushed the drop-support-for-version-1 branch from 2eabd28 to 4848fa6 Compare April 12, 2021 15:26
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 12, 2021

Build succeeded.

@dims dims force-pushed the drop-support-for-version-1 branch from 4848fa6 to e032cc1 Compare April 12, 2021 16:10
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 12, 2021

Build succeeded.

@dims dims force-pushed the drop-support-for-version-1 branch from e032cc1 to 2e4bc20 Compare April 12, 2021 16:42
@dims dims changed the title [WIP] Drop support for Version 1 configuration Switch all our tests to version 2 of config.toml Apr 12, 2021
@dims dims marked this pull request as ready for review April 12, 2021 16:44
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 12, 2021

Build succeeded.

@AkihiroSuda AkihiroSuda removed this from the 2.0 milestone Apr 12, 2021
Comment thread services/server/config/config.go Outdated
@dims dims force-pushed the drop-support-for-version-1 branch from 2e4bc20 to aeda2a9 Compare April 12, 2021 17:02
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 12, 2021

Build succeeded.

Also warn when someone uses version 1

Signed-off-by: Davanum Srinivas <[email protected]>
@dims dims force-pushed the drop-support-for-version-1 branch from aeda2a9 to 9ad0879 Compare April 12, 2021 17:18
Signed-off-by: Davanum Srinivas <[email protected]>
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 12, 2021

Build succeeded.

Copy link
Copy Markdown
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants