Output information from the config folder, config import and config actions section of recipe application

Created on 5 July 2024, 9 months ago

Problem/Motivation

In #3427558: Output information from the recipe application process , we added output from the recipe application process. However, it does not appear that that we output any information about what is happening when we get to the config section of the recipe, nor the importing config from the /config folder.

Steps to reproduce

Apply a recipe. You will see output from:

Progress bar of applying the recipe

  • Recipes applied
  • Modules installed
  • Themes installed
  • Content created

Verify you don't see output from:

  • config folder being imported
  • config:import
  • config:actions

Proposed resolution

Add additional messaging.

Remaining tasks

Doing it

User interface changes

N/A

API changes

TBD

Data model changes

TBD

Feature request
Status

Active

Version

11.0

Component

Code

Created by

🇺🇸United States thejimbirch Cape Cod, Massachusetts

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @thejimbirch
  • 🇺🇸United States thejimbirch Cape Cod, Massachusetts
  • 🇺🇸United States ashrafabed

    “I had to use a debugger to figure in some cases why the configs changed and why it was unhappy about it.”
    This (slack) comment was in relation to recipes defaulting to strict: true as part of a conversation about whether that should continue to be the case.

    Based on my (brief) exposure to the issue via that conversation, I felt that “strict: true” was not the problem, the problem was the lack of visibility into whether all configs from a recipe were actually applied.

    So I recommend displaying some sort of notice to explain how config was applied. For example, “Configuration for the following modules was/was not overridden because X”

    I wish I could give a more specific recommendation, but I’m not prepared to at this time (would need to explore the issue more first)

  • 🇨🇭Switzerland berdir Switzerland

    Both can be true. strict: true by default being a problem and causing all kinds of issues, and not having useful output.

    > “Configuration for the following modules was/was not overridden because X

    This kind of what you get with strict: false actually. It proceeds and skips certain configs. And yes, I agree that this could be useful, but it could also be a lot and it's very technical. Maybe just for drush/core command. Maybe with an optional -v argument?

    strict: true throws an exception currently and tells you just that config foo doesn't match and the whole process aborts. (no "because X"). I think it should then essentially show a diff between those configs.

  • 🇺🇸United States thejimbirch Cape Cod, Massachusetts
Production build 0.71.5 2024