Uploaded image for project: 'Magnolia CLI'
  1. Magnolia CLI
  2. NPMCLI-169

CLI should give clear and consistant ERR messages about migrating config to v3

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Neutral Neutral
    • 3.0.0
    • None
    • None
    • Yes
    • Kromeriz 136, Kromeriz 137
    • 3

      We want to reduce the migration pain as much as possible for developers, and provide them with a relatively good experience.
      We have two main migration scenarios - the config file & the prototypes - best that developer tackles them both at once.
      Whenever we notice an out of date configuration, we should notify the developer in a consistant way. Now we have two detection mechanisms that do different things - one for the jumpstart command, and one for the create commands.

      (We need to know the documentation heading for migration to v3. placeholder used below.)

      Recommendations (open to additional input/discussion)

      Create- commands

      • Create commands should change INFO to ERR "Prototype directory structure has changed in version >= 3.0.0."
      • Docs link should link directly to the migration topic (which still needs to be created). Reason: Ideally user should migrate config file and prototypes at same time.
      • Execution should stop so user does not get the later "_default" does not exist. Reason - user should focus on the migration and not get confused by other stuff. 
        ERR! Prototype directory structure has changed in version >= 3.0.0.
Please migrate both mgnl-cli.json and any customized prototypes to new directory structure.
See https://documentation.magnolia-cms.com/display/DOCS56/Magnolia+CLI#[migration heading]

      create-light-module command

      • Also show error about v3 when trying to run "create-light-module"

      Jumpstart command

      • Should have similar syntax to the create commands and should also point to magnoila docs.
      • I would remove the how-to tips and the long json snippet, because migration has gotten more complicated - ie they should also remove the big "lightDevFoldersInModule" which is no longer used. They should just go to the docs to learn how to migrate with a best-practice procedure. (which we can easily tune cause its in docs!) 
        ERR! Configuration of mgnl-cli.json has changed in version >= 3.0.0.
Please migrate both mgnl-cli.json and any customized prototypes to new directory structure.
See https://documentation.magnolia-cms.com/display/DOCS56/Magnolia+CLI#[migration heading]

      Notes:

      Current create command output

      info Using configuration at /Users/cz/Documents/a-projects/testing/cli-update-to-3/old-project/mgnl-cli.json
info Using prototypes at /Users/cz/Documents/a-projects/testing/cli-update-to-3/old-project/mgnl-cli-prototypes
info Prototype directory structure has changed in version >= 3.0.0.
info Please migrate your prototypes to new directory structure.
info See https://documentation.magnolia-cms.com/display/DOCS56/Magnolia+CLI#MagnoliaCLI-Prototypes
info No path option provided, component will be created relative to the current folder.
ERR! /Users/cz/Documents/a-projects/testing/cli-update-to-3/old-project/mgnl-cli-prototypes/component/_default doesn't exist. Please ensure that your mgnl-cli-prototypes folder contains prototype with name _default.

      Current jumpstart command output:

      info Using configuration at /Users/cz/Documents/a-projects/testing/cli-update-to-3/old-project/mgnl-cli.json
info Using prototypes at /Users/cz/Documents/a-projects/testing/cli-update-to-3/old-project/mgnl-cli-prototypes
ERR! Your mgnl-cli.json configuration (/Users/cz/Documents/a-projects/testing/cli-update-to-3/old-project/mgnl-cli.json) does not contain the new resolveArtifactUrl, tomcat and availableWebapps nodes introduced in version 3.0.


Please either:
1. Remove or rename the mgnl-cli.json file and run the “mgnl customize-local-config” command in that directory to recreate the file with the new nodes - and then reapply your customizations, or
2. Edit the mgnl-cli.json file and add the following under the setupMagnolia node (Note that downloadUrl has changed):

  "downloadUrl": "https://nexus.magnolia-cms.com/service/local/artifact/maven/content?r=${repository}&g=${groupId}&a=${artifactId}&e=${extension}&v=${version}",
  "resolveArtifactUrl": "https://nexus.magnolia-cms.com/service/local/artifact/maven/resolve?r=${repository}&g=${groupId}&a=${artifactId}&e=${extension}&v=${version}",
  "tomcat": {
    "repositories": {
      "releases": "magnolia.public.releases",
      "snapshots": "magnolia.public.snapshots"
    },
    "groupId": "info.magnolia.tomcat.barebone",
    "artifactId": "magnolia-tomcat-barebone",
    "extension": "zip",
    "version": "LATEST"
  },
  "availableWebapps": {
    "magnolia-empty-webapp": {
      "repositories": {
        "releases": "magnolia.public.releases",
        "snapshots": "magnolia.public.snapshots"
      },
      "groupId": "info.magnolia",
      "artifactId": "magnolia-empty-webapp"
    },
    "magnolia-community-webapp": {
      "repositories": {
        "releases": "magnolia.public.releases",
        "snapshots": "magnolia.public.snapshots"
      },
      "groupId": "info.magnolia.bundle",
      "artifactId": "magnolia-community-webapp"
    },
    "magnolia-community-demo-webapp": {
      "repositories": {
        "releases": "magnolia.public.releases",
        "snapshots": "magnolia.public.snapshots"
      },
      "groupId": "info.magnolia.bundle",
      "artifactId": "magnolia-community-demo-webapp"
    },
    "magnolia-enterprise-standard-webapp": {
      "repositories": {
        "releases": "magnolia.enterprise.releases",
        "snapshots": "magnolia.enterprise.snapshots"
      },
      "repository": "magnolia.enterprise.releases",
      "groupId": "info.magnolia.eebundle",
      "artifactId": "magnolia-enterprise-standard-webapp"
    },
    "magnolia-enterprise-standard-demo-webapp": {
      "repositories": {
        "releases": "magnolia.enterprise.releases",
        "snapshots": "magnolia.enterprise.snapshots"
      },
      "groupId": "info.magnolia.eebundle",
      "artifactId": "magnolia-enterprise-standard-demo-webapp"
    },
    "magnolia-enterprise-pro-webapp": {
      "repositories": {
        "releases": "magnolia.enterprise.releases",
        "snapshots": "magnolia.enterprise.snapshots"
      },
      "groupId": "info.magnolia.eebundle",
      "artifactId": "magnolia-enterprise-pro-webapp"
    },
    "magnolia-enterprise-pro-demo-webapp": {
      "repositories": {
        "releases": "magnolia.enterprise.releases",
        "snapshots": "magnolia.enterprise.snapshots"
      },
      "groupId": "info.magnolia.eebundle",
      "artifactId": "magnolia-enterprise-pro-demo-webapp"
    },
    "magnolia-now-dev-webapp": {
      "repositories": {
        "releases": "magnolia.enterprise.releases",
        "snapshots": "magnolia.enterprise.snapshots"
      },
      "groupId": "info.magnolia.now",
      "artifactId": "magnolia-now-dev-webapp"
    }
  }

        Acceptance criteria

              ajuran Antonín Juran
              czimmermann Christopher Zimmermann
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Task DoD