[PATCH conductor] BZ834632: Fix deployment autoupdate

Tzu-Mainn Chen tzumainn at redhat.com
Fri Jun 29 14:42:57 UTC 2012


ACK'd and pushed to master.

Mainn

----- Original Message -----
> From: ifarkas at redhat.com
> To: aeolus-devel at lists.fedorahosted.org
> Sent: Thursday, June 28, 2012 4:59:22 AM
> Subject: [PATCH conductor] BZ834632: Fix deployment autoupdate
> 
> From: Imre Farkas <ifarkas at redhat.com>
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=834632
> 
> Autoupdate tried to update the deployment via a wrong url: the url
> prefix (e.g. 'conductor' in case of rpm) was added twice.
> ---
>  src/public/javascripts/backbone/models.js |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/public/javascripts/backbone/models.js
> b/src/public/javascripts/backbone/models.js
> index 4c3084d..8691741 100644
> --- a/src/public/javascripts/backbone/models.js
> +++ b/src/public/javascripts/backbone/models.js
> @@ -63,7 +63,7 @@ Conductor.Models.Deployment =
> Backbone.Model.extend({
>      this.instances = new Conductor.Models.Instances();
>      var self = this;
>      this.instances.url = function() {
> -        var path = Conductor.prefixedPath(self.urlRoot + '/' +
> self.id + '/instances');
> +        var path = self.urlRoot + '/' + self.id + '/instances';
>          return Conductor.parameterizedPath(path, self.queryParams);
>      };
>    }
> --
> 1.7.10.4
> 
> 



More information about the aeolus-devel mailing list