What happened:
Users do not have a view to see the overall status of a pipeline.
What you expected to happen:
UI Mock


How to reproduce it:
N/A
Few more pending items
Play button should start a fresh event and not restart last successful sha and an option to restartIt looks great feature 馃憤
By the way, is it OK that models has specific queries? Should these implementation are inside of datastore implementation?
Also, should we have database specific queries? It's may be hard to know where should be updated when we or someone add another database implementation.
It looks great feature 馃憤
By the way, is it OK thatmodelshas specific queries? Should these implementation are inside ofdatastoreimplementation?
Also, should we have database specific queries? It's may be hard to know where should be updated when we or someone add another database implementation.
Are you referring to PR: https://github.com/screwdriver-cd/models/pull/441/files#diff-43e1dd855c9ebdae88cbf0102c6e05b2R8
Good question, let me escalate
@adong Thank you for escalation. Should I create another issue?
@tk3fftk My replies inline
By the way, is it OK that
modelshas specific queries? Should these implementation are inside ofdatastoreimplementation?
I believe reasoning was we don't have datastore specific to DB engines like MySQL, only sequalize level with datastore-sequalize and complete raw queries need to be passed over to the datastore-sequalize with all parameters substituted. These raw queries contains data which is specific to model business logic, like the DB fields like jobIDs and values in them, we don't have any representation for those in the datastore implementation. And our datastore expects complete data to be passed over to them, so to enforce that separation of business logic specific data, raw queries was put in models
Also, should we have database specific queries? It's may be hard to know where should be updated when we or someone add another database implementation.
Challenge is having a common query which works for all DB implementations. Based on the use case we had, we were not able to find a single query which works in all DB implementations. If we can find one then we should get rid of current separate queries. And sequalize was of no help in doing these queries in a generic way. The generic problem was "Select top n matches for DB rows and group them together" eg: latest n builds for all the job ids which are sorted by created time
I got it. these seem to be needed. But I think we should avoid frequent use of that.
Re-opening to address UX alignment with mocks.
Most helpful comment
Are you referring to PR: https://github.com/screwdriver-cd/models/pull/441/files#diff-43e1dd855c9ebdae88cbf0102c6e05b2R8
Good question, let me escalate