Re: Change in ovirt-host-deploy[ovirt-host-deploy-1.6]: metrics: Make collectd/fluentd packages optional

Description

On Thu, Apr 27, 2017 at 8:18 AM, Yedidyah Bar David <didi@redhat.com> wrote:

> On Thu, Apr 27, 2017 at 9:14 AM, Code Review <gerrit@ovirt.org> wrote:
> > From Jenkins CI:
> >
> > Jenkins CI has posted comments on this change.
> >
> > Change subject: metrics: Make collectd/fluentd packages optional
> > ......................................................................
> >
> >
> > Patch Set 1:
> >
> > Build Failed
> >
> > http://jenkins.ovirt.org/job/ovirt-host-deploy_4.1_check-
> patch-fc24-x86_64/49/ : FAILURE
>
> 06:13:43 http://mirrors.phx.ovirt.org/repos/yum/fedora-base-fc24/
> base/Packages/e/eclipse-equinox-osgi-4.6.0-0.6.
> git201605092000.fc24.x86_64.rpm:
> [Errno 14] HTTP Error 404 - Not Found
>
> Retrigger didn't help.
>
> Any idea?
>
>
Opening a ticket, multiple rpm files are missing on the mirror.
Also:

eclipse-filesystem-1.0-7.fc24.x86_64: failed to retrieve
Packages/e/eclipse-filesystem-1.0-7.fc24.x86_64.rpm from
fedora-base-fc24
error was [Errno 14] HTTP Error 404 - Not Found

> Thanks,
>
> >
> > http://jenkins.ovirt.org/job/ovirt-host-deploy_4.1_check-
> patch-el7-x86_64/46/ : SUCCESS
> >
> > –
> > To view, visit https://gerrit.ovirt.org/76098
> > To unsubscribe, visit https://gerrit.ovirt.org/settings
> >
> > Gerrit-MessageType: comment
> > Gerrit-Change-Id: Ic2df4e82c92cfbf6c7df15c1ceca0600626e4385
> > Gerrit-PatchSet: 1
> > Gerrit-Project: ovirt-host-deploy
> > Gerrit-Branch: ovirt-host-deploy-1.6
> > Gerrit-Owner: Yedidyah Bar David <didi@redhat.com>
> > Gerrit-Reviewer: Ido Rosenzwig <irosenzw@redhat.com>
> > Gerrit-Reviewer: Jenkins CI
> > Gerrit-Reviewer: Lev Veyde <lveyde@redhat.com>
> > Gerrit-Reviewer: Rafael Martins <rafael@rafaelmartins.eng.br>
> > Gerrit-Reviewer: Sandro Bonazzola <sbonazzo@redhat.com>
> > Gerrit-Reviewer: Simone Tiraboschi <stirabos@redhat.com>
> > Gerrit-Reviewer: Yedidyah Bar David <didi@redhat.com>
> > Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>
> > Gerrit-HasComments: No
>
>
>
> –
> Didi
> _______________________________________________
> Infra mailing list
> Infra@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/infra
>

SANDRO BONAZZOLA

ASSOCIATE MANAGER, SOFTWARE ENGINEERING, EMEA ENG VIRTUALIZATION R&D

Red Hat EMEA <https://www.redhat.com/>
<https://red.ht/sig>
TRIED. TESTED. TRUSTED. <https://redhat.com/trusted>

Activity

Show:

Barak Korren April 27, 2017 at 8:08 AM

Yedidyah Bar David April 27, 2017 at 7:26 AM

On Thu, Apr 27, 2017 at 10:06 AM, Barak Korren <bkorren@redhat.com> wrote:
> On 27 April 2017 at 10:02, Yedidyah Bar David <didi@redhat.com> wrote:
>>>
>>> Anything else that's missing?
>>
>> I still think we should consider on-demand caching instead of mirroring.
>>
>> On my local (VMs on my laptop and minidells at home) I use [1], with a
>> variation on the squid conf there. Works more-or-less ok.
>>
>> [1] https://github.com/yevmel/squid-rpm-cache/blob/master/store_id_program/store_id_program.py
>>
>
> Been there, done that, switched to mirroring.
> Please read the following to understand why we need mirrors:
> http://ovirt-infra-docs.readthedocs.io/en/latest/CI/Transactional_mirrors/index.html

Wasn't aware of this document, thanks for writing and sharing it.

It doesn't seem like it was a simple job, and it seems to still require
some significant on-going maintenance. Did you consider doing something
like this (snapshots, special treatment for repo metadata, etc) with
on-demand proxy caching? Is there some inherent reason why it won't
work, or you simply concluded it will require more work and not be
worth it?

I do not know squid very well, no idea if it's powerful enough to
implement something like this on it (or whatever other proxy cache
tool you want). So going this way might require implementing one
from scratch (probably using existing libraries to do much of the
low-level work). It would still be interesting to try, and imo much
more useful - if done well, can be used more-or-less unchanged,
with the default configuration, for many different uses/projects.

Also, it's not clear to me how (if at all) you handle mirroring
where the source changed in the middle of syncing from it.

>
> –
> Barak Korren
> RHV DevOps team , RHCE, RHCi
> Red Hat EMEA
> redhat.com | TRIED. TESTED. TRUSTED. | redhat.com/trusted


Didi

Barak Korren April 27, 2017 at 7:08 AM

On 27 April 2017 at 10:02, Yedidyah Bar David <didi@redhat.com> wrote:
>>
>> Anything else that's missing?
>
> I still think we should consider on-demand caching instead of mirroring.
>
> On my local (VMs on my laptop and minidells at home) I use [1], with a
> variation on the squid conf there. Works more-or-less ok.
>
> [1] https://github.com/yevmel/squid-rpm-cache/blob/master/store_id_program/store_id_program.py
>

Been there, done that, switched to mirroring.
Please read the following to understand why we need mirrors:
http://ovirt-infra-docs.readthedocs.io/en/latest/CI/Transactional_mirrors/index.html


Barak Korren
RHV DevOps team , RHCE, RHCi
Red Hat EMEA
redhat.com | TRIED. TESTED. TRUSTED. | redhat.com/trusted

Yedidyah Bar David April 27, 2017 at 7:03 AM

On Thu, Apr 27, 2017 at 9:41 AM, Barak Korren <bkorren@redhat.com> wrote:
>
>
> On 27 April 2017 at 09:28, Sandro Bonazzola <sbonazzo@redhat.com> wrote:
>>
>>
>>
>> Opening a ticket, multiple rpm files are missing on the mirror.
>> Also:
>>
>> eclipse-filesystem-1.0-7.fc24.x86_64: failed to retrieve
>> Packages/e/eclipse-filesystem-1.0-7.fc24.x86_64.rpm from fedora-base-fc24
>> error was [Errno 14] HTTP Error 404 - Not Found
>
>
> Yeah, so I shouldnt have tried to remove eclipse I guess...
> I guess its a -v for this:
> https://gerrit.ovirt.org/#/c/75913/

Seems like that.

>
> Anything else that's missing?

I still think we should consider on-demand caching instead of mirroring.

On my local (VMs on my laptop and minidells at home) I use [1], with a
variation on the squid conf there. Works more-or-less ok.

[1] https://github.com/yevmel/squid-rpm-cache/blob/master/store_id_program/store_id_program.py

>
> –
> Barak Korren
> RHV DevOps team , RHCE, RHCi
> Red Hat EMEA
> redhat.com | TRIED. TESTED. TRUSTED. | redhat.com/trusted


Didi

Barak Korren April 27, 2017 at 6:53 AM
Edited

Updated patch to restore eclipse on the mirrors, waiting for them to resync now.

Fixed

Details

Assignee

Reporter

Priority

Created April 27, 2017 at 6:29 AM
Updated April 30, 2017 at 8:58 AM
Resolved April 27, 2017 at 8:08 AM