openbaton / generic-vnfm

Repository containing the source code of the generic VNFM
Apache License 2.0
20 stars 20 forks source link

EMS installation broken #25

Closed xfxian closed 7 years ago

xfxian commented 7 years ago

In the develop branch, commit 7ebdb5b broke the EMS installation. In line 35, you grep for "ems" which matches on lots of packages including for example the description "filesystems". This leads to the EMS never being installed and all deployments failing.

xfxian commented 7 years ago

By the way, I am using the Generic VNFM 3.2.0 debian package. For some reason, it seems to contain code from the develop branch?

gc4rella commented 7 years ago

OK, we'll check the problem related with that commit.

By the way, I am using the Generic VNFM 3.2.0 debian package. For some reason, it seems to contain code from the develop branch?

That commit is not included as you can see in the tagged version: https://github.com/openbaton/generic-vnfm/blob/3.2.0/src/main/resources/user-data.sh

A tag is created out of a master branch which is the result of a set of features implemented in the develop branch.

xfxian commented 7 years ago

I see, it seems that for some reason though, the debian package contains another version of the file. I've just downloaded and extracted

http://get.openbaton.org/repos/openbaton/xenial/release/pool/main/o/openbaton-vnfm-generic/openbaton-vnfm-generic_3.2.0_all.deb

and confirmed that it contains the older line from the user-data.sh file:

result=$(dpkg -l | grep "ems" | wc -l)

gc4rella commented 7 years ago

adding @flaviomu who is generating debian packages and could provide more info about that.

flaviomu commented 7 years ago

The new debian package available for the VNFM Generic 3.2.0 contains the correct version of the "user-data" file