Skip to content
This repository was archived by the owner on Jul 6, 2021. It is now read-only.

#180 Fix md reports #144

Merged
merged 4 commits into from
Dec 12, 2018
Merged

#180 Fix md reports #144

merged 4 commits into from
Dec 12, 2018

Conversation

dmius
Copy link
Collaborator

@dmius dmius commented Dec 11, 2018

No description provided.

Nastradamus
Nastradamus previously approved these changes Dec 11, 2018
{{ range $skey, $host := .hosts.replicas }}
#### DB slave server: `{{ $host }}` ####
#### Replica (`{{ $host }}`) ####
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replicas have the same set of extensions -- no need to check it

{{ range $key, $value := (index (index .results .hosts.master) "data") }}
Extension: `{{ $key }}`, Installed version: `{{ $value.installed_version}}`
### Master (`{{.hosts.master}}`) ###
Extension name | Installed version
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about available version and the flag showing if the installed version is up-to-date?

@dmius dmius requested a review from NikolayS December 12, 2018 06:24
@dmius dmius merged commit 0b03411 into master Dec 12, 2018
@NikolayS NikolayS deleted the 180-dmius-md-fixes branch December 14, 2018 02:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants