[pw-ci] [PATCH v3 2/2] obs_mon: Add OpenSuse Build Service support
Michael Santana
msantana at redhat.com
Mon Sep 20 19:29:10 UTC 2021
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
>
More information about the Pwci
mailing list