(DataError) value too long for type character varying(100) for compact node

Bug #1472275 reported by Anastasia Palkina
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Low
Aleksei Chekunov

Bug Description

Custom ISO #218

1. Create any new environment
2. Add 1 controller
3. Choose Compact mode
4. Navigate and click "More info'
5. Rename node to very-very long name
6. Click "Enter"
7. New name of node does not save. Error in firebug console (see screen)

"build_id": "2015-07-07_11-22-57", "build_number": "218", "release_versions": {"2014.2.2-7.0": {"VERSION": {"build_id": "2015-07-07_11-22-57", "build_number": "218", "api": "1.0", "fuel-library_sha": "c536b49a78d9f05793e43dded895ff951205c7a1", "nailgun_sha": "1696af33e65c4604a345ac134b449ad4844ceb1d", "feature_groups": ["mirantis"], "openstack_version": "2014.2.2-7.0", "production": "docker", "python-fuelclient_sha": "315d8bf991fbe7e2ab91abfc1f59b2f24fd92f45", "astute_sha": "9cbb8ae5adbe6e758b24b3c1021aac1b662344e8", "fuel-ostf_sha": "a752c857deafd2629baf646b1b3188f02ff38084", "release": "7.0", "fuelmain_sha": "4f2dff3bdc327858fa45bcc2853cfbceae68a40c"}}}, "auth_required": true, "api": "1.0", "fuel-library_sha": "c536b49a78d9f05793e43dded895ff951205c7a1", "nailgun_sha": "1696af33e65c4604a345ac134b449ad4844ceb1d", "feature_groups": ["mirantis"], "openstack_version": "2014.2.2-7.0", "production": "docker", "python-fuelclient_sha": "315d8bf991fbe7e2ab91abfc1f59b2f24fd92f45", "astute_sha": "9cbb8ae5adbe6e758b24b3c1021aac1b662344e8", "fuel-ostf_sha": "a752c857deafd2629baf646b1b3188f02ff38084", "release": "7.0", "fuelmain_sha": "4f2dff3bdc327858fa45bcc2853cfbceae68a40c"

Revision history for this message
Anastasia Palkina (apalkina) wrote :
Changed in fuel:
status: New → Confirmed
Revision history for this message
Vladimir Sharshov (vsharshov) wrote :

Do not brake deployment or UI. Also we can catch this problem in UI side. Move it to UI team.

Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Fuel UI Team (fuel-ui)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

Fix proposed to branch: master
Review: https://review.openstack.org/209427

Changed in fuel:
assignee: Fuel UI Team (fuel-ui) → Julia Aranovich (jkirnosova)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/209427
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=99105a49b030c6d38b2a2dfc08f7b82946a2d939
Submitter: Jenkins
Branch: master

commit 99105a49b030c6d38b2a2dfc08f7b82946a2d939
Author: Julia Aranovich <email address hidden>
Date: Wed Aug 5 12:46:38 2015 +0300

    Limit node name length

    Closes-Bug: #1472275

    Change-Id: I636fdc2d423884b7cb714692b529d13be1d96697

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Anastasia Palkina (apalkina) wrote :

Reproduced on ISO #301

I entered name "ваолмирваолмроявламряорорчслоячрячолмряочсмромросрялочсрморосророрчорлядочрсмгрдгшяргрсрмгчсрягрмгсч"
You can see result on attached screen

"build_id": "301", "build_number": "301", "release_versions": {"2015.1.0-7.0": {"VERSION": {"build_id": "301", "build_number": "301", "api": "1.0", "fuel-library_sha": "5d50055aeca1dd0dc53b43825dc4c8f7780be9dd", "nailgun_sha": "4162b0c15adb425b37608c787944d1983f543aa8", "feature_groups": ["mirantis"], "fuel-nailgun-agent_sha": "d7027952870a35db8dc52f185bb1158cdd3d1ebd", "openstack_version": "2015.1.0-7.0", "fuel-agent_sha": "50e90af6e3d560e9085ff71d2950cfbcca91af67", "production": "docker", "python-fuelclient_sha": "486bde57cda1badb68f915f66c61b544108606f3", "astute_sha": "6c5b73f93e24cc781c809db9159927655ced5012", "fuel-ostf_sha": "2cd967dccd66cfc3a0abd6af9f31e5b4d150a11c", "release": "7.0", "fuelmain_sha": "a65d453215edb0284a2e4761be7a156bb5627677"}}}, "auth_required": true, "api": "1.0", "fuel-library_sha": "5d50055aeca1dd0dc53b43825dc4c8f7780be9dd", "nailgun_sha": "4162b0c15adb425b37608c787944d1983f543aa8", "feature_groups": ["mirantis"], "fuel-nailgun-agent_sha": "d7027952870a35db8dc52f185bb1158cdd3d1ebd", "openstack_version": "2015.1.0-7.0", "fuel-agent_sha": "50e90af6e3d560e9085ff71d2950cfbcca91af67", "production": "docker", "python-fuelclient_sha": "486bde57cda1badb68f915f66c61b544108606f3", "astute_sha": "6c5b73f93e24cc781c809db9159927655ced5012", "fuel-ostf_sha": "2cd967dccd66cfc3a0abd6af9f31e5b4d150a11c", "release": "7.0", "fuelmain_sha": "a65d453215edb0284a2e4761be7a156bb5627677"

Revision history for this message
Anastasia Palkina (apalkina) wrote :
Changed in fuel:
status: Fix Committed → Confirmed
milestone: 7.0 → 8.0
Changed in fuel:
assignee: Julia Aranovich (jkirnosova) → Fuel UI Team (fuel-ui)
Revision history for this message
Vitaly Kramskikh (vkramskikh) wrote :

Cannot reproduce your case. Please verify again - it seems to be fixed on backend

Changed in fuel:
status: Confirmed → Fix Committed
Revision history for this message
Anastasia Palkina (apalkina) wrote :

Reproduced it on ISO #141 without any problems (see screen)

{"fuel-createmirror_sha": "df6a93f7e2819d3dfa600052b0f901d9594eb0db", "api": "1.0", "fuelmain_sha": "3303f41f99cf9167da01d503dd5d2c8dab141447", "astute_sha": "a090546d43c770ac27ca81c6f8c78ff0ba4a93e0", "fuel-ostf_sha": "983d0e6fe64397d6ff3bd72311c26c44b02de3e8", "openstack_version": "2015.1.0-8.0", "build_id": "141", "build_number": "141", "auth_required": true, "fuel-library_sha": "cd1b4b67d2b00fb10264d6626327688b170f0bf8", "fuel-nailgun-agent_sha": "08e0a11cf1f29b705e4b910d9b9db5e9b708b6e3", "fuel-nailgun_sha": "1479c0b03ad928f2ea2a819fbf8218cff32e51b9", "fuel-agent_sha": "cf699820fb0a4d20bef001861e006dc9797b5733", "production": "docker", "python-fuelclient_sha": "769df968e19d95a4ab4f12b1d2c76d385cf3168c", "feature_groups": ["mirantis"], "release": "8.0", "release_versions": {"2015.1.0-8.0": {"VERSION": {"fuel-createmirror_sha": "df6a93f7e2819d3dfa600052b0f901d9594eb0db", "api": "1.0", "fuelmain_sha": "3303f41f99cf9167da01d503dd5d2c8dab141447", "astute_sha": "a090546d43c770ac27ca81c6f8c78ff0ba4a93e0", "fuel-ostf_sha": "983d0e6fe64397d6ff3bd72311c26c44b02de3e8", "openstack_version": "2015.1.0-8.0", "build_id": "141", "build_number": "141", "fuel-library_sha": "cd1b4b67d2b00fb10264d6626327688b170f0bf8", "fuel-nailgun-agent_sha": "08e0a11cf1f29b705e4b910d9b9db5e9b708b6e3", "fuel-nailgun_sha": "1479c0b03ad928f2ea2a819fbf8218cff32e51b9", "fuel-agent_sha": "cf699820fb0a4d20bef001861e006dc9797b5733", "production": "docker", "python-fuelclient_sha": "769df968e19d95a4ab4f12b1d2c76d385cf3168c", "feature_groups": ["mirantis"], "release": "8.0"}}}

Revision history for this message
Anastasia Palkina (apalkina) wrote :
Changed in fuel:
status: Fix Committed → Confirmed
Revision history for this message
Vitaly Kramskikh (vkramskikh) wrote :

Strange. Maybe some db settings are different on ISO and on dev env. Python guys, could you please take a look? UI has restriction - 100 symbols max length

Changed in fuel:
assignee: Fuel UI Team (fuel-ui) → Fuel Python Team (fuel-python)
Dmitry Pyzhov (dpyzhov)
tags: added: area-python
Artem Roma (aroma-x)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Artem Roma (aroma-x)
Revision history for this message
Artem Roma (aroma-x) wrote :

Described error occurs in case when non-ascii characters are used to set name for node. Problem lays in database settings as Nailgun passes the data in format of utf-8 encoded string while default encoding for db server is ascii. With all that being said I move this bug to fuel-library team.

Changed in fuel:
assignee: Artem Roma (aroma-x) → Fuel Library Team (fuel-library)
Revision history for this message
Evgeniy L (rustyrobot) wrote :

After encoding is fixed, please, make sure that it also works in case of upgrades.

Changed in fuel:
importance: Medium → Low
Dmitry Pyzhov (dpyzhov)
tags: added: area-library
removed: area-python
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

The task requested in #11 is to ensure that the table encoding for postgres tables is utf-8 instead of ascii when creating nailgun DB entries

Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Aleksei Chekunov (achekunov)
Revision history for this message
Aleksei Chekunov (achekunov) wrote :

table encoding is SQL_ASCII

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/243563

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/243563
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=e09c50bdd9a19632db4de552e6b53da30e172e20
Submitter: Jenkins
Branch: master

commit e09c50bdd9a19632db4de552e6b53da30e172e20
Author: Alexey Chekunov <email address hidden>
Date: Tue Nov 10 15:16:54 2015 +0300

    Change postgres DB encoding

    The default encoding (SQL_ASCII) cannot handle long non-ASCII strings properly
     and results in errors when a user defines a node with a long name containing cyrillic characters.
    Long non-ASCII strings are properly supported with UTF-8 encoding.

    Change-Id: I6ceebb0b6c57ffb005983df7093f7f82467d486b
    Closes-Bug: #1472275

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Anastasia Palkina (apalkina) wrote :

Verified on ISO #171

"release_versions": {"2015.1.0-8.0": {"VERSION": {"fuel-upgrade_sha": "1e894e26d4e1423a9b0d66abd6a79505f4175ff6", "fuel-createmirror_sha": "22a7aacd95bbdca69f9e0f08b70facabdec8fb28", "shotgun_sha": "c377d163519f6d10b69a654019d6086ba5f14edc", "api": "1.0", "fuelmain_sha": "266f9b374934c64629a84b5146632cc0de70ac91", "astute_sha": "959b06c5ef8143125efd1727d350c050a922eb12", "fuel-ostf_sha": "9690a2de829d3b063ed1e64b0b10dde39f711dc0", "openstack_version": "2015.1.0-8.0", "build_id": "171", "fuelmenu_sha": "06bbcebed6c8d0b0f9279e2997d2f958c800e98c", "build_number": "171", "fuel-library_sha": "31aa9fb129a8b8739f8ca6ea186caa3c9f46820f", "fuel-nailgun-agent_sha": "3e9d17211d65c80bf97c8d83979979f6c7feb687", "fuel-nailgun_sha": "37cfc3206d9c0b510112ea7a7f461e40f8c08d6a", "fuel-agent_sha": "61f741f4d89a3f1095dd8b494f82695fb69153d6", "production": "docker", "network-checker_sha": "a57e1d69acb5e765eb22cab0251c589cd76f51da", "feature_groups": ["mirantis"], "release": "8.0", "python-fuelclient_sha": "e685d68c1c0d0fa0491a250f07d9c3a8d0f9608c"}}}, "fuel-upgrade_sha": "1e894e26d4e1423a9b0d66abd6a79505f4175ff6", "fuel-createmirror_sha": "22a7aacd95bbdca69f9e0f08b70facabdec8fb28", "shotgun_sha": "c377d163519f6d10b69a654019d6086ba5f14edc", "api": "1.0", "fuelmain_sha": "266f9b374934c64629a84b5146632cc0de70ac91", "astute_sha": "959b06c5ef8143125efd1727d350c050a922eb12", "fuel-ostf_sha": "9690a2de829d3b063ed1e64b0b10dde39f711dc0", "openstack_version": "2015.1.0-8.0", "build_id": "171", "fuelmenu_sha": "06bbcebed6c8d0b0f9279e2997d2f958c800e98c", "build_number": "171", "auth_required": true, "fuel-library_sha": "31aa9fb129a8b8739f8ca6ea186caa3c9f46820f", "fuel-nailgun-agent_sha": "3e9d17211d65c80bf97c8d83979979f6c7feb687", "fuel-nailgun_sha": "37cfc3206d9c0b510112ea7a7f461e40f8c08d6a", "fuel-agent_sha": "61f741f4d89a3f1095dd8b494f82695fb69153d6", "production": "docker", "network-checker_sha": "a57e1d69acb5e765eb22cab0251c589cd76f51da", "feature_groups": ["mirantis"], "release": "8.0", "python-fuelclient_sha": "e685d68c1c0d0fa0491a250f07d9c3a8d0f9608c"

Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.