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

Editing the manifest.json file by hand is time-consuming and error-prone #4704

Open
lbussell opened this issue Jun 28, 2023 · 4 comments
Open

Comments

@lbussell
Copy link
Contributor

Describe the Problem

.NET Docker's manifest.json is over 7000 lines long and can only be edited by hand. This is an extremely time-consuming and error-prone process that needs to happen every time we move or update floating tags, and add or remove linux distros.

Additionally, there is lots of duplication of the same information inside and outside of the manifest file. For example,

dotnet-docker/manifest.json

Lines 965 to 1009 in 5b22241

"productVersion": "$(dotnet|8.0|product-version)",
"sharedTags": {
"$(dotnet|8.0|product-version)": {},
"$(dotnet|8.0|product-version)-bookworm-slim": {},
"8.0-preview": {},
"8.0-preview-bookworm-slim": {},
"latest": {}
},
"platforms": [
{
"dockerfile": "src/runtime-deps/8.0/bookworm-slim/amd64",
"dockerfileTemplate": "eng/dockerfile-templates/runtime-deps/Dockerfile",
"os": "linux",
"osVersion": "bookworm-slim",
"tags": {
"$(dotnet|8.0|product-version)-bookworm-slim-amd64": {},
"8.0-preview-bookworm-slim-amd64": {}
}
},
{
"architecture": "arm",
"dockerfile": "src/runtime-deps/8.0/bookworm-slim/arm32v7",
"dockerfileTemplate": "eng/dockerfile-templates/runtime-deps/Dockerfile",
"os": "linux",
"osVersion": "bookworm-slim",
"tags": {
"$(dotnet|8.0|product-version)-bookworm-slim-arm32v7": {},
"8.0-preview-bookworm-slim-arm32v7": {}
},
"variant": "v7"
},
{
"architecture": "arm64",
"dockerfile": "src/runtime-deps/8.0/bookworm-slim/arm64v8",
"dockerfileTemplate": "eng/dockerfile-templates/runtime-deps/Dockerfile",
"os": "linux",
"osVersion": "bookworm-slim",
"tags": {
"$(dotnet|8.0|product-version)-bookworm-slim-arm64v8": {},
"8.0-preview-bookworm-slim-arm64v8": {}
},
"variant": "v8"
}
]
},

The .NET version, architecture, and operating system are repeated constantly throughout this snippet. The tags are plaintext except for the product version variables - while this is nice for flexibility we always update them based on a pattern. It is very easy to get wrong, see:

There are many more instances caught in code review.

Describe the Solution

The intention of this issue is to start a discussion on how we can improve the situation.

I have thought of a few potential solutions:

  • Generate dotnet-docker's mainfest.json using a template.
    • This allows us the same amount of flexibility that we already have by operating in plain-text, and also leaves other repos alone (.NET Fx, buildtools-prereqs).
  • Create a tool or new command in ImageBuilder that can operate on the manifest.json.
  • Simplify the manifest.json schema.
    • As far as I'm aware, the manifest.json schema has not changed dramatically since it was created many years ago. This could allow us to re-think many aspects of the repo and take advantage of buildx/buildkit/multi-arch Dockerfiles/whatever new Docker features exist today.
@mthalman
Copy link
Member

Related to dotnet/docker-tools#157

@richlander
Copy link
Member

@mthalman
Copy link
Member

[Triage]

We should pursue a low-cost design/POC that can be a solution here. The current manifest schema offers a lot of flexibility that we don't want to lose. Any new solution should be able to express those scenarios.

@lbussell lbussell changed the title Improve the editing experience for dotnet-docker's manifest Editing the manifest.json file by hand is time-consuming and error-prone May 6, 2024
@lbussell
Copy link
Contributor Author

lbussell commented May 6, 2024

Edited the title to reflect the issue and not a potential solution.

Also, solving #2316 might get us part-way to a solution here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Post Release
Development

No branches or pull requests

4 participants