On 4/19/23 07:39, Ales Musil wrote:
> On Tue, Apr 18, 2023 at 4:17 PM Dumitru Ceara <[email protected]> wrote:
> 
>> Hi Ales,
>>
>> On 4/17/23 16:50, Enrique Llorente Pastora wrote:
>>> This is better than nothing, copr would be the usually thing to have,
>>> but looks like it needs
>>> a lot of administrative duties, we can improve this in the future.
>>>
>>> +1
>>>
>>> On Wed, Apr 12, 2023 at 10:59 AM Ales Musil <[email protected]
>>> <mailto:[email protected]>> wrote:
>>>
>>>     In order to have up-to-date Fedora RPMs from main
>>>     branch add job that will build the RPMs automatically
>>>     and publishes them as artifacts. Those artifacts are
>>>     available for download to any logged-in user on GH.
>>>
>>>     Reported-at: https://bugzilla.redhat.com/2178936
>>>     <https://bugzilla.redhat.com/2178936>
>>>     Signed-off-by: Ales Musil <[email protected] <mailto:
>> [email protected]>>
>>>     ---
>>>      .github/workflows/test.yml | 50
>> ++++++++++++++++++++++++++++++++++++++
>>>      1 file changed, 50 insertions(+)
>>>
>>>     diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
>>>     index b41f95936..126666cae 100644
>>>     --- a/.github/workflows/test.yml
>>>     +++ b/.github/workflows/test.yml
>>>     @@ -201,3 +201,53 @@ jobs:
>>>            with:
>>>              name: logs-osx-clang---disable-ssl
>>>              path: config.log
>>>     +
>>>     +  build-linux-rpm:
>>>     +    name: linux rpm fedora
>>>     +    runs-on: ubuntu-latest
>>>     +    container: fedora:latest
>>>     +    timeout-minutes: 30
>>>     +
>>>     +    strategy:
>>>     +      fail-fast: false
>>>     +
>>>     +    steps:
>>>     +      - name: install dependencies
>>>     +        run: dnf install -y dnf-plugins-core git rpm-build
>>>     +
>>>     +      - name: checkout
>>>     +        uses: actions/checkout@v3
>>>     +        with:
>>>     +          submodules: recursive
>>>     +
>>>     +      - name: install build dependencies
>>>     +        run: |
>>>     +          sed -e 's/@VERSION@/0.0.1/' rhel/ovn-fedora.spec.in
>>>     <http://ovn-fedora.spec.in> \
>>>     +          > /tmp/ovn.spec
>>>     +          dnf builddep -y /tmp/ovn.spec
>>>     +
>>>     +      - name: configure OvS
>>>     +        run: ./boot.sh && ./configure
>>>     +        working-directory: ovs
>>>     +
>>>     +      - name: make dist OvS
>>>     +        run: make dist
>>>     +        working-directory: ovs
>>>     +
>>>     +      - name: configure OVN
>>>     +        run:  ./boot.sh && ./configure
>>>     +
>>>     +      - name: make dist OVN
>>>     +        run: make dist
>>>     +
>>>     +      - name: build RPM
>>>     +        run:  make rpm-fedora
>>>     +
>>>     +      - name: upload rpm packages
>>>     +        uses: actions/upload-artifact@v3
>>>     +        with:
>>>     +          name: rpm-packages
>>>     +          path: |
>>>     +            rpm/rpmbuild/SRPMS/*.rpm
>>>     +            rpm/rpmbuild/RPMS/*/*.rpm
>>>     +          retention-days: 14
>>
>> Can we remove this part and store the RPMs indefinitely?
>>
>> I'm asking because I was thinking this might be useful moving
>> ovn-kubernetes CI to use upstream OVN code:
>>
>> https://github.com/ovn-org/ovn-kubernetes/pull/3376#issuecomment-1505058905
>>
>> Thanks,
>> Dumitru
>>
>>
> If we remove this part it won't be stored forever, but 90 days instead.
> I'm not sure how that affects the ovn-kubernetes actually?
> Maybe at some point when OVN development slows down to a point that
> we won't push anything new within 14 days? Also this job should be triggered
> every week so the oldest rpm should be a week old.
> 

You're right, I forgot we also run this weekly.  It's all good, thanks
for pointing it out!  I applied this patch to the main branch.

Regards,
Dumitru

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to