modules/fm-orchestrator
1
0
Fork 0
mirror of https://pagure.io/fm-orchestrator.git synced 2026-04-13 22:39:58 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

No need of argument component_id for ComponentBuild.state_trace

Browse Source 
Signed-off-by: Chenxiong Qi <cqi@redhat.com>
This commit is contained in:
Chenxiong Qi
2019-07-18 22:27:59 +08:00
parent f5717e3469
commit 3b938dba26
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
module_build_service/models.py
View File

@@ -1199,10 +1199,9 @@ class ComponentBuild(MBSBase):
def from_component_nvr(cls, db_session, nvr, module_id):
return db_session.query(cls).filter_by(nvr=nvr, module_id=module_id).first()
def state_trace(self, db_session, component_id):
# FIXME: remove argument component_id, just use self.id
def state_trace(self, db_session):
return (
db_session.query(ComponentBuildTrace).filter_by(component_id=component_id)
db_session.query(ComponentBuildTrace).filter_by(component_id=self.id)
.order_by(ComponentBuildTrace.state_time)
.all()
)
@@ -1251,7 +1250,7 @@ class ComponentBuild(MBSBase):
"state_name": INVERSE_BUILD_STATES[record.state],
"reason": record.state_reason,
}
for record in self.state_trace(db_session, self.id)
for record in self.state_trace(db_session)
],
"state_url": state_url,
})