Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

schema: actually add the schema file #32

Merged
merged 2 commits into from
Apr 20, 2016

Conversation

vbatts
Copy link
Member

@vbatts vbatts commented Apr 15, 2016

I managed to forget adding the file in
#18

Signed-off-by: Vincent Batts vbatts@hashbangbash.com

"$ref": "defs-image.json#/definitions/manifestObject"
}
},
"annotations": {
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this part of the specification, yet?

Copy link
Member Author

Choose a reason for hiding this comment

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

oh oops. That is yank-and-paste cruft I was mulling on.

On Fri, Apr 15, 2016 at 2:50 PM Stephen Day notifications@github.com
wrote:

In schema/manifest-list-schema.json
#32 (comment)
:

  • "schemaVersion": {
  •  "description": "This field specifies the image manifest-list schema version as an integer",
    
  •  "id": "https://opencontainers.org/schema/image/manifest-list/schemaVersion",
    
  •  "type": "integer"
    
  • },
  • "mediaType": {
  •  "id": "https://opencontainers.org/schema/image/manifest-list/mediaType",
    
  •  "$ref": "defs-image.json#/definitions/mediaType"
    
  • },
  • "manifests": {
  •  "type": "array",
    
  •  "items": {
    
  •    "$ref": "defs-image.json#/definitions/manifestObject"
    
  •  }
    
  • },
  • "annotations": {

Is this part of the specification, yet?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
/~https://github.com/opencontainers/image-spec/pull/32/files/a5632104014382df2fc80ac79f88c26e0b286a4a#r59946235

Copy link
Member Author

Choose a reason for hiding this comment

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

updated, and added a commit to remove it from the image manifest schema for
the time being.

On Fri, Apr 15, 2016 at 3:01 PM Vincent Batts vbatts@hashbangbash.com
wrote:

oh oops. That is yank-and-paste cruft I was mulling on.

On Fri, Apr 15, 2016 at 2:50 PM Stephen Day notifications@github.com
wrote:

In schema/manifest-list-schema.json
#32 (comment)
:

  • "schemaVersion": {
  •  "description": "This field specifies the image manifest-list schema version as an integer",
    
  •  "id": "https://opencontainers.org/schema/image/manifest-list/schemaVersion",
    
  •  "type": "integer"
    
  • },
  • "mediaType": {
  •  "id": "https://opencontainers.org/schema/image/manifest-list/mediaType",
    
  •  "$ref": "defs-image.json#/definitions/mediaType"
    
  • },
  • "manifests": {
  •  "type": "array",
    
  •  "items": {
    
  •    "$ref": "defs-image.json#/definitions/manifestObject"
    
  •  }
    
  • },
  • "annotations": {

Is this part of the specification, yet?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
/~https://github.com/opencontainers/image-spec/pull/32/files/a5632104014382df2fc80ac79f88c26e0b286a4a#r59946235

vbatts added 2 commits April 15, 2016 18:01
I managed to forget adding the file in
opencontainers#18

Signed-off-by: Vincent Batts <vbatts@hashbangbash.com>
This was yank-and-paste from the runtime-spec that I was mulling on and
forgot to remove before the PR.

While this might can be added back, it will likely have its own
discussion.

Signed-off-by: Vincent Batts <vbatts@hashbangbash.com>
@vbatts vbatts force-pushed the schema_add_manifest_list branch from a563210 to 4b14700 Compare April 15, 2016 22:04
@philips
Copy link
Contributor

philips commented Apr 20, 2016

LGTM

@vbatts vbatts merged commit 91d41ab into opencontainers:master Apr 20, 2016
@vbatts vbatts deleted the schema_add_manifest_list branch April 20, 2016 17:39
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.

3 participants