search

Tendrl / monitoring-integration

Component that enables integration with external monitoring services.
GNU Lesser General Public License v2.1
4 stars 13 forks source link

argument processing in tendrl-upgrade is done after few migration steps #586

Closed mbukatov closed 5 years ago

mbukatov commented 5 years ago

When I run tendrl-upgrade with --help arugment, it tries to migrade graphite data first, and the shows the help as long as the 1st action haven't failed. This is very unusual design choice.

One can try this just after cloning the repo:

$ python2 ./tendrl/monitoring_integration/upgrades/delete_dashboards.py --help

 Modifying brick path separator 

find: The relative path ‘.cabal/bin’ is included in the PATH environment variable, which is insecure in combination with the -execdir action of find.  Please remove that entry from $PATH

 Migrating graphite data 

sh: django-admin: command not found
sh: django-admin: command not found
Failed in deleting dashboards with error: No section: 'security'
mbukatov commented 5 years ago

fixed by https://github.com/Tendrl/monitoring-integration/pull/587