Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nine ui python #1090

Merged
merged 16 commits into from Mar 12, 2014
Merged

Nine ui python #1090

merged 16 commits into from Mar 12, 2014

Conversation

tardyp
Copy link
Member

@tardyp tardyp commented Mar 7, 2014

Those are some small python code changes I need to make progress on the UI.

  • buildset properties are now published in buildsets/:bsid/properties
  • buildstep.addURL() is updating the db up to the UI
  • buildstep.setStateStrings is updating the db up to the UI

Pierre Tardy added 12 commits March 7, 2014 19:49
This is a new resource for buildset properties.

There is also a placeholder for build properties, but this first needs a db change
This adds url to the db, with all the tests, and race condition handling
Signed-off-by: Pierre Tardy <pierre.tardy@intel.com>
and supprt _add__ operator for it, so that description and descriptionSuffix
can be concatenated unrendered yet


Signed-off-by: Pierre Tardy <pierre.tardy@intel.com>
Signed-off-by: Pierre Tardy <pierre.tardy@intel.com>
Signed-off-by: Pierre Tardy <pierre.tardy@intel.com>
Signed-off-by: Pierre Tardy <pierre.tardy@intel.com>
Signed-off-by: Pierre Tardy <pierre.tardy@intel.com>
sa does not allow is None in the whereclauses,
we create a simple null constant for usage inside the db apis

Signed-off-by: Pierre Tardy <pierre.tardy@intel.com>
so that the sdist does not need node

Signed-off-by: Pierre Tardy <pierre.tardy@intel.com>
@tardyp
Copy link
Member Author

tardyp commented Mar 11, 2014

updated PR.
on top of unit tests fixes, and review comments fixes, I added to functionality:

master housekeeping is cleaning out pending Builds and buildrequest upon stop and startup. Please tell if there is an objection on this.

Also I added generic update events for buildrequests, so that the UI can watch buildrequest is the generic way.

name = types.String()
url = types.String()


Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks unused..

@djmitche
Copy link
Member

Looks good. A few minor things, but I think this is ready to merge.

Pierre Tardy added 3 commits March 11, 2014 15:45
Signed-off-by: Pierre Tardy <pierre.tardy@intel.com>
We just describe a little bit later instead of accepting
not rendered arguments

Signed-off-by: Pierre Tardy <pierre.tardy@intel.com>
@tardyp
Copy link
Member Author

tardyp commented Mar 11, 2014

ok. I think I'm done on this one. I'll push that tomorrow to watch metabuildbot potencial
failures

tardyp added a commit that referenced this pull request Mar 12, 2014
@tardyp tardyp merged commit df15653 into buildbot:nine Mar 12, 2014
@tardyp tardyp deleted the nine_demo_python branch March 17, 2014 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants