search

claudyus / LXC-Web-Panel

LXC Web Panel improved for lxc 1.0+
http://claudyus.github.io/LXC-Web-Panel/
Other
221 stars 68 forks source link

CT name causes problem with rootfs parse #86

Closed alitvak69 closed 8 years ago

alitvak69 commented 9 years ago

If I create a CT with host name foo-light, I can no longer make changes to the container. While more permissive regex is allowing hostname, it doesn't allow the same format for rootfs. So bug #69 is not fully fixed, please fix it.

claudyus commented 9 years ago

I yet pushed a fix for it, otherwise the package release process need review and I'm not going to release a new deb package immediately.

alitvak69 commented 9 years ago

Can I build a deb package based on your master then?

claudyus commented 9 years ago

Can you try a package from https://packagecloud.io/claudyus/LXC-Web-Panel

I'm trying a new debian package release process.

alitvak69 commented 9 years ago

I tried this using bash script. It looks interesting. One problem, new package overwrote security database lwp.db changing it to a default admin / admin.

Other then that dash in hostname works now. Default gateway on the other hand showed as unexpected format (highlighted yellow) but it let me apply anyway.

Thank you for a quick fix.

claudyus commented 9 years ago

Yes the database override on reinstall is a well know problem and is not easly fixable. I could suggest another login provider to workaround the problem or contribute in sqlalchemy migration (sql branch).

yswery commented 9 years ago

Can confirm on my end when you create a CT with a hyphen it you can not longer edit it later:

error image

claudyus commented 8 years ago

fixes released in 0.9