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

peadm should print a summary of plan parameters to the standard log stream [SOLARCH-546] #105

Open
Sharpie opened this issue Jun 24, 2020 · 0 comments

Comments

@Sharpie
Copy link
Member

Sharpie commented Jun 24, 2020

Use Case

The peadm::install and peadm::upgrade plans co-ordinate tasks across many nodes using a variety of inputs. When one of these plans fails, the output is often the first thing presented for debugging. Currently, this output lacks important information such as:

  • The peadm version being used

  • The nodes involved in the plan and their assigned roles

  • The PE version being installed.

  • The PE version being upgraded.

This lack of information leads to several follow-up questions before debugging can proceed.

Describe the Solution You Would Like

peadm plans that perform significant actions, such as install or upgrade, should log relevant parameters and version numbers using out::message so that most relevant information is captured in one log stream.

Describe Alternatives You've Considered

The current alternative is to manually gather plan inputs. This is tedious, often results in incomplete data, and can miss inconsistencies (i.e. the parameters the user thought they passed to the plan are not the parameters the plan ended up using due to inventory settings, peadm plans being called by other user-written plans, etc.).

@Sharpie Sharpie added the enhancement New feature or request label Jun 24, 2020
@timidri timidri added feature jira and removed enhancement New feature or request labels Jul 15, 2021
@timidri timidri changed the title peadm should print a summary of plan parameters to the standard log stream peadm should print a summary of plan parameters to the standard log stream [SOLARCH-546] Jul 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants