You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@aurora.apache.org by kevinburg <gi...@git.apache.org> on 2014/07/01 22:10:50 UTC

[GitHub] incubator-aurora pull request: Fix TaskObserver for json endpoint ...

GitHub user kevinburg opened a pull request:

    https://github.com/apache/incubator-aurora/pull/11

    Fix TaskObserver for json endpoint usage

    The '/j/task?task_id=...' endpoint requires TaskObserver._task to be exposed. I implemented a wrapper for it.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/foursquare/incubator-aurora master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-aurora/pull/11.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #11
    
----
commit 17babd1cf6fd9913f5e166ef08e72260679a58ae
Author: Kevin Burg <kb...@foursquare.com>
Date:   2014-06-30T23:56:40Z

    Fix TaskObserver for json endpoint usage

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-aurora pull request: Fix TaskObserver for json endpoint ...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-aurora/pull/11


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-aurora pull request: Fix TaskObserver for json endpoint ...

Posted by kevints <gi...@git.apache.org>.
Github user kevints commented on the pull request:

    https://github.com/apache/incubator-aurora/pull/11#issuecomment-47705601
  
    Thanks for the submission, do you mind reformatting this to ReviewBoard
    following the guidelines at
    http://aurora.incubator.apache.org/docs/howtocontribute/
    
    
    On Tue, Jul 1, 2014 at 1:10 PM, Kevin Burg <no...@github.com> wrote:
    
    > The '/j/task?task_id=...' endpoint requires TaskObserver._task to be
    > exposed. I implemented a wrapper for it.
    > ------------------------------
    > You can merge this Pull Request by running
    >
    >   git pull https://github.com/foursquare/incubator-aurora master
    >
    > Or view, comment on, or merge it at:
    >
    >   https://github.com/apache/incubator-aurora/pull/11
    > Commit Summary
    >
    >    - Fix TaskObserver for json endpoint usage
    >
    > File Changes
    >
    >    - *M* src/main/python/apache/thermos/observer/task_observer.py
    >    <https://github.com/apache/incubator-aurora/pull/11/files#diff-0> (10)
    >
    > Patch Links:
    >
    >    - https://github.com/apache/incubator-aurora/pull/11.patch
    >    - https://github.com/apache/incubator-aurora/pull/11.diff
    >
    > —
    > Reply to this email directly or view it on GitHub
    > <https://github.com/apache/incubator-aurora/pull/11>.
    >
    
    
    
    -- 
    Kevin Sweeney
    @kts


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---