[pw-ci] [PATCH v3 2/2] obs_mon: Add OpenSuse Build Service support

[Michael Santana]

On Mon, Sep 20, 2021 at 8:42 AM Aaron Conole <aconole at redhat.com> wrote:
>
> Michael Santana <msantana at redhat.com> writes:
>
> > Add two scripts, one to get build status from obs and the other to get
> > the build logs from obs. The script to get the logs from obs will also
> > delete the build after retrieving the logs
> >
> > Signed-off-by: Michael Santana <msantana at redhat.com>
> > ---
> >  obs_get_logs.sh | 134 ++++++++++++++++++++++++++++++++++++++++++++++++
> >  obs_mon         | 115 +++++++++++++++++++++++++++++++++++++++++
> >  2 files changed, 249 insertions(+)
> >  create mode 100755 obs_get_logs.sh
> >  create mode 100755 obs_mon
>
> This looks very dpdk specific.  Do we need to use OSCBIN?  Can't we do
> this all via curl?  Also, I don't see anywhere this is integrated into
> ci_mon - how will this get called?
We must use OSCBIN. I tried using curl and it didn't work. Curl only
works for certain operations but not the ones we need.

Yes, it's very dpdk specific, thank you for pointing it out. I should
have made it with being generic in mind

Also thank you for pointing out that it is not integrated into ci_mon.
It must have been lost while rebasing
>
> Snippet below (the dpdk stuff is littered throughout so consider it a
> generic comment).
>
> > +repository[0]="home_rhobsrobot_dpdk_Ubuntu_20.04"
> > +repository[1]="home_rhobsrobot_dpdk_Debian_10"
> > +repository[2]="home_rhobsrobot_dpdk_CentOS_8"
> > +repository[3]="home_rhobsrobot_dpdk_CentOS_7"
> > +
> > +arch[0]="aarch64"
> > +arch[1]="i586"
> > +arch[2]="ppc64le"
> > +arch[3]="x86_64"
>
> Why is this pushed like so?  Can't we pass these back by returning
> multiple lines in make_result_for_series ?  I think it should work that
> way, right?  We do it with github actions.
That's one way to do it. That way it's not so dpdk specific


Thank you for all the feedback. will work on a new revision. Im sorry
this is taking so long
>