Skip to content

Move manifest package to schema1#912

Merged
aaronlehmann merged 1 commit intodistribution:masterfrom
stevvooe:manifest-refactoring-schema1
Aug 21, 2015
Merged

Move manifest package to schema1#912
aaronlehmann merged 1 commit intodistribution:masterfrom
stevvooe:manifest-refactoring-schema1

Conversation

@stevvooe
Copy link
Collaborator

As we begin our march towards multi-arch, we must prepare for the reality of
multiple manifest schemas. This is the beginning of a set of changes to
facilitate this. We are both moving this package into its target position where
it may live peacefully next to other manfiest versions.

Signed-off-by: Stephen J Day stephen.day@docker.com

@stevvooe
Copy link
Collaborator Author

@aaronlehmann PTAL

@stevvooe stevvooe force-pushed the manifest-refactoring-schema1 branch 3 times, most recently from 2655cfc to 99bb7d2 Compare August 21, 2015 05:04

Choose a reason for hiding this comment

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

I assume that this is an intermediate step, and eventually methods like this will use abstractions provided by the manifest package, right? Same question for the notifications package.

No objection to doing it this way; just want to confirm that's the path forward.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@aaronlehmann That is the plan.

@aaronlehmann
Copy link

LGTM

@RichardScothern
Copy link

LGTM. Will need a rebase though.

As we begin our march towards multi-arch, we must prepare for the reality of
multiple manifest schemas. This is the beginning of a set of changes to
facilitate this. We are both moving this package into its target position where
it may live peacefully next to other manfiest versions.

Signed-off-by: Stephen J Day <stephen.day@docker.com>
@stevvooe stevvooe force-pushed the manifest-refactoring-schema1 branch from 99bb7d2 to 6712e60 Compare August 21, 2015 23:30
@stevvooe
Copy link
Collaborator Author

@RichardScothern @aaronlehmann Rebased.

aaronlehmann pushed a commit that referenced this pull request Aug 21, 2015
@aaronlehmann aaronlehmann merged commit 7c46c3f into distribution:master Aug 21, 2015
@stevvooe stevvooe deleted the manifest-refactoring-schema1 branch August 22, 2015 00:15
thaJeztah pushed a commit to thaJeztah/distribution that referenced this pull request Apr 22, 2021
…g-schema1

Move manifest package to schema1
thaJeztah pushed a commit to thaJeztah/distribution that referenced this pull request Jan 19, 2022
…g-schema1

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants