Bug 636410
Summary: | Regression: Exception: sqlalchemy.exceptions.InvalidRequestError:No rows returned for one() | ||
---|---|---|---|
Product: | [Retired] Beaker | Reporter: | Šimon Lukašík <slukasik> |
Component: | command line | Assignee: | Raymond Mancy <rmancy> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | 0.7 | CC: | bpeck, dcallagh, ebaak, kbaker, mcsontos, rmancy, satqe-list |
Target Milestone: | --- | Keywords: | Regression |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-09-30 04:57:06 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 545868, 632609 |
Description
Šimon Lukašík
2010-09-22 07:42:51 UTC
I've found the problem. Investigating a fix. diff --git a/Server/bkr/server/model.py b/Server/bkr/server/model.py index 4c0914b..8fbb463 100644 --- a/Server/bkr/server/model.py +++ b/Server/bkr/server/model.py @@ -4265,8 +4265,9 @@ class GuestRecipe(Recipe): LabControllerDistro.distro == self.distro, LabControllerDistro.lab_controller == self.system.lab_controller ) - ).one().tree_path - recipe.setAttribute("location", "%s" % location) + ).first() + if location: + recipe.setAttribute("location", "%s" % location.tree_path) return Recipe.to_xml(self, recipe, clone, from_recipeset, from_machine) def _get_distro_requires(self): I think we still need the available() query, in case someone is looking for machines via System->Available that they may want to manually reserve. available_for_schedule() would ommit these machines. ack on the Available still. What about an ack on the patch itself? |