#1 ✓resolved
tuulos (at gmail)

Error trying to save configuration

Reported by tuulos (at gmail) | October 5th, 2008 @ 03:54 PM

When I try to save a new host row I get "Request Failed".

See detailed discussion here

Comments and changes to this ticket

  • tuulos (at gmail)

    tuulos (at gmail) October 5th, 2008 @ 04:00 PM

    • State changed from “new” to “open”
  • Christopher Brooks

    Christopher Brooks October 18th, 2008 @ 09:46 PM

    • Assigned user set to “tuulos (at gmail)”

    I was able to reproduce this error by doing the following on a fresh install of ubuntu-jeos 8.04.1:

    sudo adduser --home /home/disco --shell /bin/bash disco sudo usermod -a -G admin disco su disco

    sudo apt-get install openssh-server openssh-client lighttpd erlang python python-setuptools python-dev git-core wget gcc build-essential

    cd ~ git clone git://github.com/tuulos/disco.git disco

    wget http://pypi.python.org/packages/... tar -xvf python-cjson-1.0.5.tar.gz cd python-cjson-1.0.5/ sudo python setup.py install cd .. sudo rm -rf python-cjson-1.0.5 rm -rf python-cjson-1.0.5.tar.gz

    wget http://discoproject.org/disco-0.... tar -xvf disco-0.1.1.tar.gz rm -rf ../../disco-0.1.1.tar.gz

    ssh-keygen -N '' -f ~/.ssh/id_dsa cat ~/.ssh/id_dsa.pub >> ~/.ssh/authorized_keys

    cd disco/conf

    sudo ln -s /etc/init.d/sshd /etc/rc2.d/S98sshd

    //change the master port to be enabled //start the master //start the node //hit the master ip:7000 and add "localhost" to the table and "1" to the workers" //click "save table

  • Attila Babo

    Attila Babo January 27th, 2009 @ 04:29 PM

    I had the same problem with a fresh installation, the field name was wrong on config.js. This patch is a possible fix:

    --- a/master/www/config.js +++ b/master/www/config.js @@ -37,7 +37,7 @@ function whitelist(){

    function save_settings(){

         var s = {}
    
    
    •  $.each($(".setting"), function(){
      
      
    •  $.each($("setting"), function(){
               s[$(this).attr("id")] = $(this).val();
       });
       post_req("/disco/ctrl/save_settings",
      
      
  • tuulos (at gmail)

    tuulos (at gmail) January 28th, 2009 @ 12:01 AM

    Attila,

    I think that your patch doesn't fix it.

    ".setting" with a dot is jquery's way of referring to all elements of the class "setting".

    Thanks for the idea anyways. I've had trouble repeating this bug but I'm working on it..

  • Attila Babo

    Attila Babo January 28th, 2009 @ 08:38 AM

    You are right, that's even a different settings. After adding some debug to the Erlang and Javascript part I figured out what's happening. You need to enter the value in the field and press enter before sending it. Without the enter javascript happily sends an empty string. save_config_table: [[<<>>,<<"4">>]] save_config_table: [[<<"ec2-67-202-53-140.compute-1.amazonaws.com">>,<<"4">>]]

  • Attila Babo

    Attila Babo January 29th, 2009 @ 05:40 AM

    Patching config.js solves the problem:

    --- a/master/www/config.js +++ b/master/www/config.js @@ -66,7 +66,7 @@ function add_to_blacklist(){ function send_table(){

         var table = $("tbody > tr").map(function(){
                 var arr = $.makeArray($(this).children().map(function(){
    
    
    •                    return jQuery.trim($(this).text());}));
      
      
    •                   return jQuery.trim($(this).text() || $(this).find('input').val());}));
               arr.shift();  // throw away the first node - 'remove'
               return JSON.stringify(arr);
       });
      
      
  • tuulos (at gmail)

    tuulos (at gmail) April 6th, 2009 @ 02:32 AM

    • State changed from “open” to “resolved”

    Thanks Attila! Patch merged.

  • tuulos (at gmail)

    tuulos (at gmail) April 6th, 2009 @ 02:32 AM

    (from [e36a7041c503a9be5b7d2768bd64c46a6819203f]) Fix for sending invalid configuration data. [#1 state:resolved] http://github.com/tuulos/disco/c...

  • tuulos (at gmail)

    tuulos (at gmail) April 6th, 2009 @ 02:32 AM

    (from [d100ac1e2117789e5061ade04153aa5d41db3d80]) Revert "Fix for sending invalid configuration data. [#1 state:resolved]"

    This reverts commit e36a7041c503a9be5b7d2768bd64c46a6819203f. http://github.com/tuulos/disco/c...

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Disco is an open-source implementation of the Map-Reduce framework for distributed computing. As the original framework, Disco supports parallel computations over large data sets on unreliable cluster of computers.

Shared Ticket Bins

People watching this ticket

Tags

Referenced by

Pages