DPDK CI discussions
 help / color / Atom feed
* [dpdk-ci] OBS pull request in pw-ci
@ 2020-07-03 13:08 Juraj Linkeš
  2020-07-06 17:31 ` Aaron Conole
  0 siblings, 1 reply; 9+ messages in thread
From: Juraj Linkeš @ 2020-07-03 13:08 UTC (permalink / raw)
  To: Aaron Conole; +Cc: ci

[-- Attachment #1: Type: text/plain, Size: 1133 bytes --]

Hi folks,

I've submitted a PR with OBS changes to pw-ci: https://github.com/orgcandman/pw-ci/pull/12

I have a couple of points/questions:

*        The scripts are using https://build.opensuse.org/package/show/home:bluca:dpdk/dpdk to fork the debian and redhat spec and control files.

*        We should create a community OBS project (that the community can manage) where the forked packages will be created and where the build will run (OBS_TARGET_PROJECT in 3rd-party/dpdk/jenkins-rc)

o   The project should be configured according to what we want to build - distros/architectures. Can be done via OBS GUI.

*        Only after all patches have been submitted to a series branch does the DPDK Jenkins job upload and configure an OBS package. This means that we'll have a build just for the whole series, not for each patch. Not sure if this is a problem.

*        One thing that doesn't make sense to me is that the DPDK Jenkins job builds dpdk after each patch - is this needed, since Travis is going to do the same thing anyway?

*        I've added some more comments to the PR itself.

Regards,
Juraj

[-- Attachment #2: Type: text/html, Size: 7850 bytes --]

<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-2">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
	{font-family:Wingdings;
	panose-1:5 0 0 0 0 0 0 0 0 0;}
@font-face
	{font-family:"Cambria Math";
	panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
	{font-family:Calibri;
	panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
	{margin:0cm;
	margin-bottom:.0001pt;
	font-size:11.0pt;
	font-family:"Calibri",sans-serif;
	mso-fareast-language:EN-US;}
a:link, span.MsoHyperlink
	{mso-style-priority:99;
	color:#0563C1;
	text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
	{mso-style-priority:99;
	color:#954F72;
	text-decoration:underline;}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
	{mso-style-priority:34;
	margin-top:0cm;
	margin-right:0cm;
	margin-bottom:0cm;
	margin-left:36.0pt;
	margin-bottom:.0001pt;
	font-size:11.0pt;
	font-family:"Calibri",sans-serif;
	mso-fareast-language:EN-US;}
span.EmailStyle17
	{mso-style-type:personal-compose;
	font-family:"Calibri",sans-serif;
	color:windowtext;}
span.x
	{mso-style-name:x;}
.MsoChpDefault
	{mso-style-type:export-only;
	mso-fareast-language:EN-US;}
@page WordSection1
	{size:612.0pt 792.0pt;
	margin:70.85pt 70.85pt 70.85pt 70.85pt;}
div.WordSection1
	{page:WordSection1;}
/* List Definitions */
@list l0
	{mso-list-id:1663240175;
	mso-list-type:hybrid;
	mso-list-template-ids:736289244 68878337 68878339 68878341 68878337 68878339 68878341 68878337 68878339 68878341;}
@list l0:level1
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-18.0pt;
	font-family:Symbol;}
@list l0:level2
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-18.0pt;
	font-family:"Courier New";}
@list l0:level3
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-18.0pt;
	font-family:Wingdings;}
@list l0:level4
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-18.0pt;
	font-family:Symbol;}
@list l0:level5
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-18.0pt;
	font-family:"Courier New";}
@list l0:level6
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-18.0pt;
	font-family:Wingdings;}
@list l0:level7
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-18.0pt;
	font-family:Symbol;}
@list l0:level8
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-18.0pt;
	font-family:"Courier New";}
@list l0:level9
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-18.0pt;
	font-family:Wingdings;}
ol
	{margin-bottom:0cm;}
ul
	{margin-bottom:0cm;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="SK" link="#0563C1" vlink="#954F72">
<div class="WordSection1">
<p class="MsoNormal"><span lang="EN-US">Hi folks,<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"><o:p>&nbsp;</o:p></span></p>
<p class="MsoNormal"><span lang="EN-US">I've submitted a PR with OBS changes to pw-ci:
<a href="https://github.com/orgcandman/pw-ci/pull/12">https://github.com/orgcandman/pw-ci/pull/12</a><o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"><o:p>&nbsp;</o:p></span></p>
<p class="MsoNormal"><span lang="EN-US">I have a couple of points/questions:<o:p></o:p></span></p>
<p class="MsoListParagraph" style="text-indent:-18.0pt;mso-list:l0 level1 lfo1"><![if !supportLists]><span lang="EN-US" style="font-family:Symbol"><span style="mso-list:Ignore">&middot;<span style="font:7.0pt &quot;Times New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
</span></span></span><![endif]><span lang="EN-US">The scripts are using <a href="https://build.opensuse.org/package/show/home:bluca:dpdk/dpdk">
https://build.opensuse.org/package/show/home:bluca:dpdk/dpdk</a> to fork the debian and redhat spec and control files.<o:p></o:p></span></p>
<p class="MsoListParagraph" style="text-indent:-18.0pt;mso-list:l0 level1 lfo1"><![if !supportLists]><span lang="EN-US" style="font-family:Symbol"><span style="mso-list:Ignore">&middot;<span style="font:7.0pt &quot;Times New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
</span></span></span><![endif]><span lang="EN-US">We should create a community OBS project (that the community can manage) where the forked packages will be created and where the build will run (</span><span class="x">OBS_TARGET_PROJECT in 3rd-party/dpdk/jenkins-rc</span><span lang="EN-US">)<o:p></o:p></span></p>
<p class="MsoListParagraph" style="margin-left:72.0pt;text-indent:-18.0pt;mso-list:l0 level2 lfo1">
<![if !supportLists]><span lang="EN-US" style="font-family:&quot;Courier New&quot;"><span style="mso-list:Ignore">o<span style="font:7.0pt &quot;Times New Roman&quot;">&nbsp;&nbsp;
</span></span></span><![endif]><span lang="EN-US">The project should be configured according to what we want to build - distros/architectures. Can be done via OBS GUI.<o:p></o:p></span></p>
<p class="MsoListParagraph" style="text-indent:-18.0pt;mso-list:l0 level1 lfo1"><![if !supportLists]><span lang="EN-US" style="font-family:Symbol"><span style="mso-list:Ignore">&middot;<span style="font:7.0pt &quot;Times New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
</span></span></span><![endif]><span lang="EN-US">Only after all patches have been submitted to a series branch does the DPDK Jenkins job upload and configure an OBS package. This means that we'll have a build just for the whole series, not for each patch.
 Not sure if this is a problem.<o:p></o:p></span></p>
<p class="MsoListParagraph" style="text-indent:-18.0pt;mso-list:l0 level1 lfo1"><![if !supportLists]><span lang="EN-US" style="font-family:Symbol"><span style="mso-list:Ignore">&middot;<span style="font:7.0pt &quot;Times New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
</span></span></span><![endif]><span lang="EN-US">One thing that doesn't make sense to me is that the DPDK Jenkins job builds dpdk after each patch - is this needed, since Travis is going to do the same thing anyway?<o:p></o:p></span></p>
<p class="MsoListParagraph" style="text-indent:-18.0pt;mso-list:l0 level1 lfo1"><![if !supportLists]><span lang="EN-US" style="font-family:Symbol"><span style="mso-list:Ignore">&middot;<span style="font:7.0pt &quot;Times New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
</span></span></span><![endif]><span lang="EN-US">I've added some more comments to the PR itself.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"><o:p>&nbsp;</o:p></span></p>
<p class="MsoNormal"><span lang="EN-US">Regards,<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US">Juraj<o:p></o:p></span></p>
</div>
</body>
</html>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-ci] OBS pull request in pw-ci
  2020-07-03 13:08 [dpdk-ci] OBS pull request in pw-ci Juraj Linkeš
@ 2020-07-06 17:31 ` Aaron Conole
  2020-07-13 14:20   ` Juraj Linkeš
  0 siblings, 1 reply; 9+ messages in thread
From: Aaron Conole @ 2020-07-06 17:31 UTC (permalink / raw)
  To: Juraj Linkeš; +Cc: ci\

Juraj Linkeš <juraj.linkes@pantheon.tech> writes:

> Hi folks,
>
>  
>
> I've submitted a PR with OBS changes to pw-ci: https://github.com/orgcandman/pw-ci/pull/12

Cool!  I'll take a look.  I was on PTO, so didn't get a chance yet.

>  
>
> I have a couple of points/questions:
>
> ·        The scripts are using https://build.opensuse.org/package/show/home:bluca:dpdk/dpdk to fork the debian and
> redhat spec and control files.

Okay.  A quick look shows that it's configurable, though.  Should be
okay.  I refer to:

https://github.com/orgcandman/pw-ci/pull/12/commits/93f6baa497c85d7aca36cda6661e90f7a69d4709#diff-e2ab32f32889186bdeb0ab4f0ef55282

> ·        We should create a community OBS project (that the community can manage) where the forked packages will be
> created and where the build will run (OBS_TARGET_PROJECT in 3rd-party/dpdk/jenkins-rc)

Okay.

> o   The project should be configured according to what we want to build - distros/architectures. Can be done via OBS GUI.
>
> ·        Only after all patches have been submitted to a series branch does the DPDK Jenkins job upload and configure an OBS
> package. This means that we'll have a build just for the whole series, not for each patch. Not sure if this is a problem.

The flip side is the amount of time to build.  We'd prefer to build just
one at a time.

> ·        One thing that doesn't make sense to me is that the DPDK Jenkins job builds dpdk after each patch - is this needed,
> since Travis is going to do the same thing anyway?

Travis doesn't.  The ovsrobot travis build is configured to cancel a
build on a branch if a new one comes.  That way we only build the
latest.  Builds on travis take 40+ min to complete, so if a 20-patch
series came in, we wouldn't be done building it for more than half a
day.  Then if a v2+ is posted in rapid succession, we lost all this
time.  So for now, we only build the absolute last patch in the series
on Travis.

Also, the build server we run the robot on doesn't do the builds anymore
because it also got overloaded - so meson and ninja are not the actual
meson/ninja but just dummy commands.  :-/  Maybe I should re-enable and
see if things improved.

> ·        I've added some more comments to the PR itself.
>
>  
>
> Regards,
>
> Juraj


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-ci] OBS pull request in pw-ci
  2020-07-06 17:31 ` Aaron Conole
@ 2020-07-13 14:20   ` Juraj Linkeš
  2020-07-13 20:17     ` Aaron Conole
  0 siblings, 1 reply; 9+ messages in thread
From: Juraj Linkeš @ 2020-07-13 14:20 UTC (permalink / raw)
  To: Aaron Conole; +Cc: ci

Hi Aaron,

Have you had a chance to look at the pull request?

Thanks,
Juraj

> -----Original Message-----
> From: Aaron Conole <aconole@redhat.com>
> Sent: Monday, July 6, 2020 7:32 PM
> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Cc: ci@dpdk.org
> Subject: Re: OBS pull request in pw-ci
> 
> Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
> 
> > Hi folks,
> >
> >
> >
> > I've submitted a PR with OBS changes to pw-ci:
> > https://github.com/orgcandman/pw-ci/pull/12
> 
> Cool!  I'll take a look.  I was on PTO, so didn't get a chance yet.
> 
> >
> >
> > I have a couple of points/questions:
> >
> > ·        The scripts are using
> https://build.opensuse.org/package/show/home:bluca:dpdk/dpdk to fork the
> debian and
> > redhat spec and control files.
> 
> Okay.  A quick look shows that it's configurable, though.  Should be okay.  I refer
> to:
> 
> https://github.com/orgcandman/pw-
> ci/pull/12/commits/93f6baa497c85d7aca36cda6661e90f7a69d4709#diff-
> e2ab32f32889186bdeb0ab4f0ef55282
> 
> > ·        We should create a community OBS project (that the community can
> manage) where the forked packages will be
> > created and where the build will run (OBS_TARGET_PROJECT in
> > 3rd-party/dpdk/jenkins-rc)
> 
> Okay.
> 
> > o   The project should be configured according to what we want to build -
> distros/architectures. Can be done via OBS GUI.
> >
> > ·        Only after all patches have been submitted to a series branch does the
> DPDK Jenkins job upload and configure an OBS
> > package. This means that we'll have a build just for the whole series, not for
> each patch. Not sure if this is a problem.
> 
> The flip side is the amount of time to build.  We'd prefer to build just one at a
> time.
> 
> > ·        One thing that doesn't make sense to me is that the DPDK Jenkins job
> builds dpdk after each patch - is this needed,
> > since Travis is going to do the same thing anyway?
> 
> Travis doesn't.  The ovsrobot travis build is configured to cancel a build on a
> branch if a new one comes.  That way we only build the latest.  Builds on travis
> take 40+ min to complete, so if a 20-patch series came in, we wouldn't be done
> building it for more than half a day.  Then if a v2+ is posted in rapid succession,
> we lost all this time.  So for now, we only build the absolute last patch in the
> series on Travis.
> 
> Also, the build server we run the robot on doesn't do the builds anymore
> because it also got overloaded - so meson and ninja are not the actual
> meson/ninja but just dummy commands.  :-/  Maybe I should re-enable and see
> if things improved.
> 
> > ·        I've added some more comments to the PR itself.
> >
> >
> >
> > Regards,
> >
> > Juraj
> 


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-ci] OBS pull request in pw-ci
  2020-07-13 14:20   ` Juraj Linkeš
@ 2020-07-13 20:17     ` Aaron Conole
  2020-07-15 19:13       ` Aaron Conole
  0 siblings, 1 reply; 9+ messages in thread
From: Aaron Conole @ 2020-07-13 20:17 UTC (permalink / raw)
  To: Juraj Linkeš; +Cc: ci\

Juraj Linkeš <juraj.linkes@pantheon.tech> writes:

> Hi Aaron,
>
> Have you had a chance to look at the pull request?

Only a bit.  I will do something more thorough on Wednesday.

> Thanks,
> Juraj
>
>> -----Original Message-----
>> From: Aaron Conole <aconole@redhat.com>
>> Sent: Monday, July 6, 2020 7:32 PM
>> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
>> Cc: ci@dpdk.org
>> Subject: Re: OBS pull request in pw-ci
>> 
>> Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
>> 
>> > Hi folks,
>> >
>> >
>> >
>> > I've submitted a PR with OBS changes to pw-ci:
>> > https://github.com/orgcandman/pw-ci/pull/12
>> 
>> Cool!  I'll take a look.  I was on PTO, so didn't get a chance yet.
>> 
>> >
>> >
>> > I have a couple of points/questions:
>> >
>> > ·        The scripts are using
>> https://build.opensuse.org/package/show/home:bluca:dpdk/dpdk to fork the
>> debian and
>> > redhat spec and control files.
>> 
>> Okay.  A quick look shows that it's configurable, though.  Should be okay.  I refer
>> to:
>> 
>> https://github.com/orgcandman/pw-
>> ci/pull/12/commits/93f6baa497c85d7aca36cda6661e90f7a69d4709#diff-
>> e2ab32f32889186bdeb0ab4f0ef55282
>> 
>> > ·        We should create a community OBS project (that the community can
>> manage) where the forked packages will be
>> > created and where the build will run (OBS_TARGET_PROJECT in
>> > 3rd-party/dpdk/jenkins-rc)
>> 
>> Okay.
>> 
>> > o   The project should be configured according to what we want to build -
>> distros/architectures. Can be done via OBS GUI.
>> >
>> > ·        Only after all patches have been submitted to a series branch does the
>> DPDK Jenkins job upload and configure an OBS
>> > package. This means that we'll have a build just for the whole series, not for
>> each patch. Not sure if this is a problem.
>> 
>> The flip side is the amount of time to build.  We'd prefer to build just one at a
>> time.
>> 
>> > ·        One thing that doesn't make sense to me is that the DPDK Jenkins job
>> builds dpdk after each patch - is this needed,
>> > since Travis is going to do the same thing anyway?
>> 
>> Travis doesn't.  The ovsrobot travis build is configured to cancel a build on a
>> branch if a new one comes.  That way we only build the latest.  Builds on travis
>> take 40+ min to complete, so if a 20-patch series came in, we wouldn't be done
>> building it for more than half a day.  Then if a v2+ is posted in rapid succession,
>> we lost all this time.  So for now, we only build the absolute last patch in the
>> series on Travis.
>> 
>> Also, the build server we run the robot on doesn't do the builds anymore
>> because it also got overloaded - so meson and ninja are not the actual
>> meson/ninja but just dummy commands.  :-/  Maybe I should re-enable and see
>> if things improved.
>> 
>> > ·        I've added some more comments to the PR itself.
>> >
>> >
>> >
>> > Regards,
>> >
>> > Juraj
>> 


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-ci] OBS pull request in pw-ci
  2020-07-13 20:17     ` Aaron Conole
@ 2020-07-15 19:13       ` Aaron Conole
  2020-07-16 11:58         ` Juraj Linkeš
  0 siblings, 1 reply; 9+ messages in thread
From: Aaron Conole @ 2020-07-15 19:13 UTC (permalink / raw)
  To: Juraj Linkeš; +Cc: ci\

Aaron Conole <aconole@redhat.com> writes:

> Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
>
>> Hi Aaron,
>>
>> Have you had a chance to look at the pull request?
>
> Only a bit.  I will do something more thorough on Wednesday.

I looked through it and it seems okay.  I'll set it up to do a test run
tomorrow, and merge it once that goes well.

If the obs source project changes (right now, it's set to
home:bluca:dpdk) how should we update it?  Meaning, where does that
variable come from (and if the robot needs to provide one at some point
ever, how can we do that)?

>> Thanks,
>> Juraj
>>
>>> -----Original Message-----
>>> From: Aaron Conole <aconole@redhat.com>
>>> Sent: Monday, July 6, 2020 7:32 PM
>>> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
>>> Cc: ci@dpdk.org
>>> Subject: Re: OBS pull request in pw-ci
>>> 
>>> Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
>>> 
>>> > Hi folks,
>>> >
>>> >
>>> >
>>> > I've submitted a PR with OBS changes to pw-ci:
>>> > https://github.com/orgcandman/pw-ci/pull/12
>>> 
>>> Cool!  I'll take a look.  I was on PTO, so didn't get a chance yet.
>>> 
>>> >
>>> >
>>> > I have a couple of points/questions:
>>> >
>>> > ·        The scripts are using
>>> https://build.opensuse.org/package/show/home:bluca:dpdk/dpdk to fork the
>>> debian and
>>> > redhat spec and control files.
>>> 
>>> Okay.  A quick look shows that it's configurable, though.  Should be okay.  I refer
>>> to:
>>> 
>>> https://github.com/orgcandman/pw-
>>> ci/pull/12/commits/93f6baa497c85d7aca36cda6661e90f7a69d4709#diff-
>>> e2ab32f32889186bdeb0ab4f0ef55282
>>> 
>>> > ·        We should create a community OBS project (that the community can
>>> manage) where the forked packages will be
>>> > created and where the build will run (OBS_TARGET_PROJECT in
>>> > 3rd-party/dpdk/jenkins-rc)
>>> 
>>> Okay.
>>> 
>>> > o   The project should be configured according to what we want to build -
>>> distros/architectures. Can be done via OBS GUI.
>>> >
>>> > ·        Only after all patches have been submitted to a series branch does the
>>> DPDK Jenkins job upload and configure an OBS
>>> > package. This means that we'll have a build just for the whole series, not for
>>> each patch. Not sure if this is a problem.
>>> 
>>> The flip side is the amount of time to build.  We'd prefer to build just one at a
>>> time.
>>> 
>>> > ·        One thing that doesn't make sense to me is that the DPDK Jenkins job
>>> builds dpdk after each patch - is this needed,
>>> > since Travis is going to do the same thing anyway?
>>> 
>>> Travis doesn't.  The ovsrobot travis build is configured to cancel a build on a
>>> branch if a new one comes.  That way we only build the latest.  Builds on travis
>>> take 40+ min to complete, so if a 20-patch series came in, we wouldn't be done
>>> building it for more than half a day.  Then if a v2+ is posted in rapid succession,
>>> we lost all this time.  So for now, we only build the absolute last patch in the
>>> series on Travis.
>>> 
>>> Also, the build server we run the robot on doesn't do the builds anymore
>>> because it also got overloaded - so meson and ninja are not the actual
>>> meson/ninja but just dummy commands.  :-/  Maybe I should re-enable and see
>>> if things improved.
>>> 
>>> > ·        I've added some more comments to the PR itself.
>>> >
>>> >
>>> >
>>> > Regards,
>>> >
>>> > Juraj
>>> 


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-ci] OBS pull request in pw-ci
  2020-07-15 19:13       ` Aaron Conole
@ 2020-07-16 11:58         ` Juraj Linkeš
  2020-07-16 12:54           ` Aaron Conole
  0 siblings, 1 reply; 9+ messages in thread
From: Juraj Linkeš @ 2020-07-16 11:58 UTC (permalink / raw)
  To: Aaron Conole; +Cc: ci

Hi Aaron,

The dpdk community decided that the best place to maintaing the .spec and .control files would be in an OBS repository. Luca is currently doing that so that's where it comes from. As to how to update it - I included the exact string "home:bluca:dpdk" so that it's clear we want to use this repo, but it should be handled the same way as UPLOAD_GIT_REPO. I didn't see how this is supplied from the code in pw-ci, so I assumed it's something that's changed locally. I followed the same logic for OBS_SOURCE_PACKAGE and OBS_GIT_URL - the values that are there are what we should be using for our use case at this point in time.

I have a minor question about the pw-ci repo - is this your personal repo (i.e. not maintained by dpdk community)?

One last thing - we'll talk today about setting up a community OBS project (to mirror https://travis-ci.com/github/ovsrobot/dpdk - is this actually managed by the community?) in the CI call.

Thanks,
Juraj

> -----Original Message-----
> From: Aaron Conole <aconole@redhat.com>
> Sent: Wednesday, July 15, 2020 9:14 PM
> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Cc: ci@dpdk.org
> Subject: Re: OBS pull request in pw-ci
> 
> Aaron Conole <aconole@redhat.com> writes:
> 
> > Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
> >
> >> Hi Aaron,
> >>
> >> Have you had a chance to look at the pull request?
> >
> > Only a bit.  I will do something more thorough on Wednesday.
> 
> I looked through it and it seems okay.  I'll set it up to do a test run tomorrow,
> and merge it once that goes well.
> 
> If the obs source project changes (right now, it's set to
> home:bluca:dpdk) how should we update it?  Meaning, where does that variable
> come from (and if the robot needs to provide one at some point ever, how can
> we do that)?
> 
> >> Thanks,
> >> Juraj
> >>
> >>> -----Original Message-----
> >>> From: Aaron Conole <aconole@redhat.com>
> >>> Sent: Monday, July 6, 2020 7:32 PM
> >>> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
> >>> Cc: ci@dpdk.org
> >>> Subject: Re: OBS pull request in pw-ci
> >>>
> >>> Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
> >>>
> >>> > Hi folks,
> >>> >
> >>> >
> >>> >
> >>> > I've submitted a PR with OBS changes to pw-ci:
> >>> > https://github.com/orgcandman/pw-ci/pull/12
> >>>
> >>> Cool!  I'll take a look.  I was on PTO, so didn't get a chance yet.
> >>>
> >>> >
> >>> >
> >>> > I have a couple of points/questions:
> >>> >
> >>> > ·        The scripts are using
> >>> https://build.opensuse.org/package/show/home:bluca:dpdk/dpdk to fork
> >>> the debian and
> >>> > redhat spec and control files.
> >>>
> >>> Okay.  A quick look shows that it's configurable, though.  Should be
> >>> okay.  I refer
> >>> to:
> >>>
> >>> https://github.com/orgcandman/pw-
> >>> ci/pull/12/commits/93f6baa497c85d7aca36cda6661e90f7a69d4709#diff-
> >>> e2ab32f32889186bdeb0ab4f0ef55282
> >>>
> >>> > ·        We should create a community OBS project (that the community can
> >>> manage) where the forked packages will be
> >>> > created and where the build will run (OBS_TARGET_PROJECT in
> >>> > 3rd-party/dpdk/jenkins-rc)
> >>>
> >>> Okay.
> >>>
> >>> > o   The project should be configured according to what we want to build -
> >>> distros/architectures. Can be done via OBS GUI.
> >>> >
> >>> > ·        Only after all patches have been submitted to a series branch does
> the
> >>> DPDK Jenkins job upload and configure an OBS
> >>> > package. This means that we'll have a build just for the whole
> >>> > series, not for
> >>> each patch. Not sure if this is a problem.
> >>>
> >>> The flip side is the amount of time to build.  We'd prefer to build
> >>> just one at a time.
> >>>
> >>> > ·        One thing that doesn't make sense to me is that the DPDK Jenkins job
> >>> builds dpdk after each patch - is this needed,
> >>> > since Travis is going to do the same thing anyway?
> >>>
> >>> Travis doesn't.  The ovsrobot travis build is configured to cancel a
> >>> build on a branch if a new one comes.  That way we only build the
> >>> latest.  Builds on travis take 40+ min to complete, so if a 20-patch
> >>> series came in, we wouldn't be done building it for more than half a
> >>> day.  Then if a v2+ is posted in rapid succession, we lost all this
> >>> time.  So for now, we only build the absolute last patch in the series on
> Travis.
> >>>
> >>> Also, the build server we run the robot on doesn't do the builds
> >>> anymore because it also got overloaded - so meson and ninja are not
> >>> the actual meson/ninja but just dummy commands.  :-/  Maybe I should
> >>> re-enable and see if things improved.
> >>>
> >>> > ·        I've added some more comments to the PR itself.
> >>> >
> >>> >
> >>> >
> >>> > Regards,
> >>> >
> >>> > Juraj
> >>>
> 


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-ci] OBS pull request in pw-ci
  2020-07-16 11:58         ` Juraj Linkeš
@ 2020-07-16 12:54           ` Aaron Conole
  2020-07-17  7:05             ` Juraj Linkeš
  0 siblings, 1 reply; 9+ messages in thread
From: Aaron Conole @ 2020-07-16 12:54 UTC (permalink / raw)
  To: Juraj Linkeš; +Cc: ci\

Juraj Linkeš <juraj.linkes@pantheon.tech> writes:

> Hi Aaron,
>
> The dpdk community decided that the best place to maintaing the .spec
> and .control files would be in an OBS repository. Luca is currently
> doing that so that's where it comes from. As to how to update it - I
> included the exact string "home:bluca:dpdk" so that it's clear we want
> to use this repo, but it should be handled the same way as
> UPLOAD_GIT_REPO. I didn't see how this is supplied from the code in
> pw-ci, so I assumed it's something that's changed locally. I followed
> the same logic for OBS_SOURCE_PACKAGE and OBS_GIT_URL - the values
> that are there are what we should be using for our use case at this
> point in time.
>
> I have a minor question about the pw-ci repo - is this your personal
> repo (i.e. not maintained by dpdk community)?

I maintain it - but it's publicly accessible / available.  I don't know
what else it means.

> One last thing - we'll talk today about setting up a community OBS
> project (to mirror https://travis-ci.com/github/ovsrobot/dpdk - is
> this actually managed by the community?) in the CI call.

Again, I maintain it - but it's for the community.  Actually the service
that runs the robot is a Red Hat server, but we choose to push it to a
public github and use a public CI so that it is all part of the
community.

I don't know what it means to be community maintained ;)

> Thanks,
> Juraj
>
>> -----Original Message-----
>> From: Aaron Conole <aconole@redhat.com>
>> Sent: Wednesday, July 15, 2020 9:14 PM
>> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
>> Cc: ci@dpdk.org
>> Subject: Re: OBS pull request in pw-ci
>> 
>> Aaron Conole <aconole@redhat.com> writes:
>> 
>> > Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
>> >
>> >> Hi Aaron,
>> >>
>> >> Have you had a chance to look at the pull request?
>> >
>> > Only a bit.  I will do something more thorough on Wednesday.
>> 
>> I looked through it and it seems okay.  I'll set it up to do a test run tomorrow,
>> and merge it once that goes well.
>> 
>> If the obs source project changes (right now, it's set to
>> home:bluca:dpdk) how should we update it?  Meaning, where does that variable
>> come from (and if the robot needs to provide one at some point ever, how can
>> we do that)?
>> 
>> >> Thanks,
>> >> Juraj
>> >>
>> >>> -----Original Message-----
>> >>> From: Aaron Conole <aconole@redhat.com>
>> >>> Sent: Monday, July 6, 2020 7:32 PM
>> >>> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
>> >>> Cc: ci@dpdk.org
>> >>> Subject: Re: OBS pull request in pw-ci
>> >>>
>> >>> Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
>> >>>
>> >>> > Hi folks,
>> >>> >
>> >>> >
>> >>> >
>> >>> > I've submitted a PR with OBS changes to pw-ci:
>> >>> > https://github.com/orgcandman/pw-ci/pull/12
>> >>>
>> >>> Cool!  I'll take a look.  I was on PTO, so didn't get a chance yet.
>> >>>
>> >>> >
>> >>> >
>> >>> > I have a couple of points/questions:
>> >>> >
>> >>> > ·        The scripts are using
>> >>> https://build.opensuse.org/package/show/home:bluca:dpdk/dpdk to fork
>> >>> the debian and
>> >>> > redhat spec and control files.
>> >>>
>> >>> Okay.  A quick look shows that it's configurable, though.  Should be
>> >>> okay.  I refer
>> >>> to:
>> >>>
>> >>> https://github.com/orgcandman/pw-
>> >>> ci/pull/12/commits/93f6baa497c85d7aca36cda6661e90f7a69d4709#diff-
>> >>> e2ab32f32889186bdeb0ab4f0ef55282
>> >>>
>> >>> > ·        We should create a community OBS project (that the community can
>> >>> manage) where the forked packages will be
>> >>> > created and where the build will run (OBS_TARGET_PROJECT in
>> >>> > 3rd-party/dpdk/jenkins-rc)
>> >>>
>> >>> Okay.
>> >>>
>> >>> > o   The project should be configured according to what we want to build -
>> >>> distros/architectures. Can be done via OBS GUI.
>> >>> >
>> >>> > ·        Only after all patches have been submitted to a series branch does
>> the
>> >>> DPDK Jenkins job upload and configure an OBS
>> >>> > package. This means that we'll have a build just for the whole
>> >>> > series, not for
>> >>> each patch. Not sure if this is a problem.
>> >>>
>> >>> The flip side is the amount of time to build.  We'd prefer to build
>> >>> just one at a time.
>> >>>
>> >>> > ·        One thing that doesn't make sense to me is that the DPDK Jenkins job
>> >>> builds dpdk after each patch - is this needed,
>> >>> > since Travis is going to do the same thing anyway?
>> >>>
>> >>> Travis doesn't.  The ovsrobot travis build is configured to cancel a
>> >>> build on a branch if a new one comes.  That way we only build the
>> >>> latest.  Builds on travis take 40+ min to complete, so if a 20-patch
>> >>> series came in, we wouldn't be done building it for more than half a
>> >>> day.  Then if a v2+ is posted in rapid succession, we lost all this
>> >>> time.  So for now, we only build the absolute last patch in the series on
>> Travis.
>> >>>
>> >>> Also, the build server we run the robot on doesn't do the builds
>> >>> anymore because it also got overloaded - so meson and ninja are not
>> >>> the actual meson/ninja but just dummy commands.  :-/  Maybe I should
>> >>> re-enable and see if things improved.
>> >>>
>> >>> > ·        I've added some more comments to the PR itself.
>> >>> >
>> >>> >
>> >>> >
>> >>> > Regards,
>> >>> >
>> >>> > Juraj
>> >>>
>> 


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-ci] OBS pull request in pw-ci
  2020-07-16 12:54           ` Aaron Conole
@ 2020-07-17  7:05             ` Juraj Linkeš
  2020-07-17 13:24               ` Aaron Conole
  0 siblings, 1 reply; 9+ messages in thread
From: Juraj Linkeš @ 2020-07-17  7:05 UTC (permalink / raw)
  To: Aaron Conole; +Cc: ci

Hi Aaron,

By community maintained I meant that it's not solely on one person's shoulders - that is, if need be, someone else (from the community) would also be able to do the maintenance work.

Juraj

> -----Original Message-----
> From: Aaron Conole <aconole@redhat.com>
> Sent: Thursday, July 16, 2020 2:54 PM
> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Cc: ci@dpdk.org
> Subject: Re: OBS pull request in pw-ci
> 
> Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
> 
> > Hi Aaron,
> >
> > The dpdk community decided that the best place to maintaing the .spec
> > and .control files would be in an OBS repository. Luca is currently
> > doing that so that's where it comes from. As to how to update it - I
> > included the exact string "home:bluca:dpdk" so that it's clear we want
> > to use this repo, but it should be handled the same way as
> > UPLOAD_GIT_REPO. I didn't see how this is supplied from the code in
> > pw-ci, so I assumed it's something that's changed locally. I followed
> > the same logic for OBS_SOURCE_PACKAGE and OBS_GIT_URL - the values
> > that are there are what we should be using for our use case at this
> > point in time.
> >
> > I have a minor question about the pw-ci repo - is this your personal
> > repo (i.e. not maintained by dpdk community)?
> 
> I maintain it - but it's publicly accessible / available.  I don't know what else it
> means.
> 
> > One last thing - we'll talk today about setting up a community OBS
> > project (to mirror https://travis-ci.com/github/ovsrobot/dpdk - is
> > this actually managed by the community?) in the CI call.
> 
> Again, I maintain it - but it's for the community.  Actually the service that runs
> the robot is a Red Hat server, but we choose to push it to a public github and use
> a public CI so that it is all part of the community.
> 
> I don't know what it means to be community maintained ;)
> 
> > Thanks,
> > Juraj
> >
> >> -----Original Message-----
> >> From: Aaron Conole <aconole@redhat.com>
> >> Sent: Wednesday, July 15, 2020 9:14 PM
> >> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
> >> Cc: ci@dpdk.org
> >> Subject: Re: OBS pull request in pw-ci
> >>
> >> Aaron Conole <aconole@redhat.com> writes:
> >>
> >> > Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
> >> >
> >> >> Hi Aaron,
> >> >>
> >> >> Have you had a chance to look at the pull request?
> >> >
> >> > Only a bit.  I will do something more thorough on Wednesday.
> >>
> >> I looked through it and it seems okay.  I'll set it up to do a test
> >> run tomorrow, and merge it once that goes well.
> >>
> >> If the obs source project changes (right now, it's set to
> >> home:bluca:dpdk) how should we update it?  Meaning, where does that
> >> variable come from (and if the robot needs to provide one at some
> >> point ever, how can we do that)?
> >>
> >> >> Thanks,
> >> >> Juraj
> >> >>
> >> >>> -----Original Message-----
> >> >>> From: Aaron Conole <aconole@redhat.com>
> >> >>> Sent: Monday, July 6, 2020 7:32 PM
> >> >>> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
> >> >>> Cc: ci@dpdk.org
> >> >>> Subject: Re: OBS pull request in pw-ci
> >> >>>
> >> >>> Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
> >> >>>
> >> >>> > Hi folks,
> >> >>> >
> >> >>> >
> >> >>> >
> >> >>> > I've submitted a PR with OBS changes to pw-ci:
> >> >>> > https://github.com/orgcandman/pw-ci/pull/12
> >> >>>
> >> >>> Cool!  I'll take a look.  I was on PTO, so didn't get a chance yet.
> >> >>>
> >> >>> >
> >> >>> >
> >> >>> > I have a couple of points/questions:
> >> >>> >
> >> >>> > ·        The scripts are using
> >> >>> https://build.opensuse.org/package/show/home:bluca:dpdk/dpdk to
> >> >>> fork the debian and
> >> >>> > redhat spec and control files.
> >> >>>
> >> >>> Okay.  A quick look shows that it's configurable, though.  Should
> >> >>> be okay.  I refer
> >> >>> to:
> >> >>>
> >> >>> https://github.com/orgcandman/pw-
> >> >>>
> ci/pull/12/commits/93f6baa497c85d7aca36cda6661e90f7a69d4709#diff-
> >> >>> e2ab32f32889186bdeb0ab4f0ef55282
> >> >>>
> >> >>> > ·        We should create a community OBS project (that the community
> can
> >> >>> manage) where the forked packages will be
> >> >>> > created and where the build will run (OBS_TARGET_PROJECT in
> >> >>> > 3rd-party/dpdk/jenkins-rc)
> >> >>>
> >> >>> Okay.
> >> >>>
> >> >>> > o   The project should be configured according to what we want to
> build -
> >> >>> distros/architectures. Can be done via OBS GUI.
> >> >>> >
> >> >>> > ·        Only after all patches have been submitted to a series branch does
> >> the
> >> >>> DPDK Jenkins job upload and configure an OBS
> >> >>> > package. This means that we'll have a build just for the whole
> >> >>> > series, not for
> >> >>> each patch. Not sure if this is a problem.
> >> >>>
> >> >>> The flip side is the amount of time to build.  We'd prefer to
> >> >>> build just one at a time.
> >> >>>
> >> >>> > ·        One thing that doesn't make sense to me is that the DPDK Jenkins
> job
> >> >>> builds dpdk after each patch - is this needed,
> >> >>> > since Travis is going to do the same thing anyway?
> >> >>>
> >> >>> Travis doesn't.  The ovsrobot travis build is configured to
> >> >>> cancel a build on a branch if a new one comes.  That way we only
> >> >>> build the latest.  Builds on travis take 40+ min to complete, so
> >> >>> if a 20-patch series came in, we wouldn't be done building it for
> >> >>> more than half a day.  Then if a v2+ is posted in rapid
> >> >>> succession, we lost all this time.  So for now, we only build the
> >> >>> absolute last patch in the series on
> >> Travis.
> >> >>>
> >> >>> Also, the build server we run the robot on doesn't do the builds
> >> >>> anymore because it also got overloaded - so meson and ninja are
> >> >>> not the actual meson/ninja but just dummy commands.  :-/  Maybe I
> >> >>> should re-enable and see if things improved.
> >> >>>
> >> >>> > ·        I've added some more comments to the PR itself.
> >> >>> >
> >> >>> >
> >> >>> >
> >> >>> > Regards,
> >> >>> >
> >> >>> > Juraj
> >> >>>
> >>
> 


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-ci] OBS pull request in pw-ci
  2020-07-17  7:05             ` Juraj Linkeš
@ 2020-07-17 13:24               ` Aaron Conole
  0 siblings, 0 replies; 9+ messages in thread
From: Aaron Conole @ 2020-07-17 13:24 UTC (permalink / raw)
  To: Juraj Linkeš; +Cc: ci\

Juraj Linkeš <juraj.linkes@pantheon.tech> writes:

> Hi Aaron,
>
> By community maintained I meant that it's not solely on one person's
> shoulders - that is, if need be, someone else (from the community)
> would also be able to do the maintenance work.

Ahh - yes, David Marchand has access to those servers.

> Juraj
>
>> -----Original Message-----
>> From: Aaron Conole <aconole@redhat.com>
>> Sent: Thursday, July 16, 2020 2:54 PM
>> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
>> Cc: ci@dpdk.org
>> Subject: Re: OBS pull request in pw-ci
>> 
>> Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
>> 
>> > Hi Aaron,
>> >
>> > The dpdk community decided that the best place to maintaing the .spec
>> > and .control files would be in an OBS repository. Luca is currently
>> > doing that so that's where it comes from. As to how to update it - I
>> > included the exact string "home:bluca:dpdk" so that it's clear we want
>> > to use this repo, but it should be handled the same way as
>> > UPLOAD_GIT_REPO. I didn't see how this is supplied from the code in
>> > pw-ci, so I assumed it's something that's changed locally. I followed
>> > the same logic for OBS_SOURCE_PACKAGE and OBS_GIT_URL - the values
>> > that are there are what we should be using for our use case at this
>> > point in time.
>> >
>> > I have a minor question about the pw-ci repo - is this your personal
>> > repo (i.e. not maintained by dpdk community)?
>> 
>> I maintain it - but it's publicly accessible / available.  I don't know what else it
>> means.
>> 
>> > One last thing - we'll talk today about setting up a community OBS
>> > project (to mirror https://travis-ci.com/github/ovsrobot/dpdk - is
>> > this actually managed by the community?) in the CI call.
>> 
>> Again, I maintain it - but it's for the community.  Actually the service that runs
>> the robot is a Red Hat server, but we choose to push it to a public github and use
>> a public CI so that it is all part of the community.
>> 
>> I don't know what it means to be community maintained ;)
>> 
>> > Thanks,
>> > Juraj
>> >
>> >> -----Original Message-----
>> >> From: Aaron Conole <aconole@redhat.com>
>> >> Sent: Wednesday, July 15, 2020 9:14 PM
>> >> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
>> >> Cc: ci@dpdk.org
>> >> Subject: Re: OBS pull request in pw-ci
>> >>
>> >> Aaron Conole <aconole@redhat.com> writes:
>> >>
>> >> > Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
>> >> >
>> >> >> Hi Aaron,
>> >> >>
>> >> >> Have you had a chance to look at the pull request?
>> >> >
>> >> > Only a bit.  I will do something more thorough on Wednesday.
>> >>
>> >> I looked through it and it seems okay.  I'll set it up to do a test
>> >> run tomorrow, and merge it once that goes well.
>> >>
>> >> If the obs source project changes (right now, it's set to
>> >> home:bluca:dpdk) how should we update it?  Meaning, where does that
>> >> variable come from (and if the robot needs to provide one at some
>> >> point ever, how can we do that)?
>> >>
>> >> >> Thanks,
>> >> >> Juraj
>> >> >>
>> >> >>> -----Original Message-----
>> >> >>> From: Aaron Conole <aconole@redhat.com>
>> >> >>> Sent: Monday, July 6, 2020 7:32 PM
>> >> >>> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
>> >> >>> Cc: ci@dpdk.org
>> >> >>> Subject: Re: OBS pull request in pw-ci
>> >> >>>
>> >> >>> Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
>> >> >>>
>> >> >>> > Hi folks,
>> >> >>> >
>> >> >>> >
>> >> >>> >
>> >> >>> > I've submitted a PR with OBS changes to pw-ci:
>> >> >>> > https://github.com/orgcandman/pw-ci/pull/12
>> >> >>>
>> >> >>> Cool!  I'll take a look.  I was on PTO, so didn't get a chance yet.
>> >> >>>
>> >> >>> >
>> >> >>> >
>> >> >>> > I have a couple of points/questions:
>> >> >>> >
>> >> >>> > ·        The scripts are using
>> >> >>> https://build.opensuse.org/package/show/home:bluca:dpdk/dpdk to
>> >> >>> fork the debian and
>> >> >>> > redhat spec and control files.
>> >> >>>
>> >> >>> Okay.  A quick look shows that it's configurable, though.  Should
>> >> >>> be okay.  I refer
>> >> >>> to:
>> >> >>>
>> >> >>> https://github.com/orgcandman/pw-
>> >> >>>
>> ci/pull/12/commits/93f6baa497c85d7aca36cda6661e90f7a69d4709#diff-
>> >> >>> e2ab32f32889186bdeb0ab4f0ef55282
>> >> >>>
>> >> >>> > ·        We should create a community OBS project (that the community
>> can
>> >> >>> manage) where the forked packages will be
>> >> >>> > created and where the build will run (OBS_TARGET_PROJECT in
>> >> >>> > 3rd-party/dpdk/jenkins-rc)
>> >> >>>
>> >> >>> Okay.
>> >> >>>
>> >> >>> > o   The project should be configured according to what we want to
>> build -
>> >> >>> distros/architectures. Can be done via OBS GUI.
>> >> >>> >
>> >> >>> > ·        Only after all patches have been submitted to a series branch does
>> >> the
>> >> >>> DPDK Jenkins job upload and configure an OBS
>> >> >>> > package. This means that we'll have a build just for the whole
>> >> >>> > series, not for
>> >> >>> each patch. Not sure if this is a problem.
>> >> >>>
>> >> >>> The flip side is the amount of time to build.  We'd prefer to
>> >> >>> build just one at a time.
>> >> >>>
>> >> >>> > ·        One thing that doesn't make sense to me is that the DPDK Jenkins
>> job
>> >> >>> builds dpdk after each patch - is this needed,
>> >> >>> > since Travis is going to do the same thing anyway?
>> >> >>>
>> >> >>> Travis doesn't.  The ovsrobot travis build is configured to
>> >> >>> cancel a build on a branch if a new one comes.  That way we only
>> >> >>> build the latest.  Builds on travis take 40+ min to complete, so
>> >> >>> if a 20-patch series came in, we wouldn't be done building it for
>> >> >>> more than half a day.  Then if a v2+ is posted in rapid
>> >> >>> succession, we lost all this time.  So for now, we only build the
>> >> >>> absolute last patch in the series on
>> >> Travis.
>> >> >>>
>> >> >>> Also, the build server we run the robot on doesn't do the builds
>> >> >>> anymore because it also got overloaded - so meson and ninja are
>> >> >>> not the actual meson/ninja but just dummy commands.  :-/  Maybe I
>> >> >>> should re-enable and see if things improved.
>> >> >>>
>> >> >>> > ·        I've added some more comments to the PR itself.
>> >> >>> >
>> >> >>> >
>> >> >>> >
>> >> >>> > Regards,
>> >> >>> >
>> >> >>> > Juraj
>> >> >>>
>> >>
>> 


^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, back to index

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-03 13:08 [dpdk-ci] OBS pull request in pw-ci Juraj Linkeš
2020-07-06 17:31 ` Aaron Conole
2020-07-13 14:20   ` Juraj Linkeš
2020-07-13 20:17     ` Aaron Conole
2020-07-15 19:13       ` Aaron Conole
2020-07-16 11:58         ` Juraj Linkeš
2020-07-16 12:54           ` Aaron Conole
2020-07-17  7:05             ` Juraj Linkeš
2020-07-17 13:24               ` Aaron Conole

DPDK CI discussions

Archives are clonable:
	git clone --mirror http://inbox.dpdk.org/ci/0 ci/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ci ci/ http://inbox.dpdk.org/ci \
		ci@dpdk.org
	public-inbox-index ci


Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.ci


AGPL code for this site: git clone https://public-inbox.org/ public-inbox