Skip to content

Manifest version incompatibility #19

@fcasson

Description

@fcasson

My manifest files look like this

>head manifest.yaml
version: 1
alias: some/alias
metadata:
- values:
    workflow:
      name: jintrac-imas
      repo: https://git.ccfe.ac.uk:jintrac/jintrac
      commit: 519ad9c1d7c814ed18efcd20c385efbf05b771b5

With version 0.13.0, we now get warnings like this

raise InvalidManifest(f"Unknown manifest version {version}.")
simdb.cli.manifest.InvalidManifest: Unknown manifest version 1.
warning: no version given in manifest, assuming version 2

Renaming the key from version: 1 to manifest_version: 1 seems to help. But the whole point of a format version for this file is to maintain back compatibility. So if you want to rename the version field (why??), then version 1 (or even 0) should be assumed if none other is found? The version field was added in version 1 and should be mandatory for all following versions.

In general, changes like this cause downstream cascades of work to update scripts. There are several deployments already in production! When can we release version 1.0.0 and get a stable client API ? Can we add tests which check compatibility of new releases with existing use cases?

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions