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

Configuration subsystem integration #167

Merged
merged 10 commits into from
May 3, 2021

Conversation

cidrblock
Copy link
Collaborator

@cidrblock cidrblock commented Apr 30, 2021

Integrate configuration subsystems into ansible-navigator:

Major changes:

  • changes in the settings file structure, nesting ee params under the execution-environment ket
  • actions that use class App now have an entry and exit function to normalize their ui handling and arg restoration
  • many changes to adapt to NOT_SET as the value for parameters that have not been set at all
  • remove all cli handling from action as the config subsystem takes care of it
  • action log now uses class App as it base to take advantage of enter, exit functionality
  • cli_args and config are removed as they are replaced by configuration subsystem
  • Moved all application initialization code into initialize.py
  • Normalize the signature of all prelogger code to return messages, errors, value
  • (where messages is a LogMessage, that can specify a log level)
  • introduce ability to write a default location log file with level debug if we sys.exit from the beginning
  • changes to tests and test fixtures as required

@cidrblock cidrblock changed the title DNM Configuration subsystem integration Configuration subsystem integration May 3, 2021
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.

None yet

2 participants