[JIRA] (OVIRT-1087) drop gerrit log watch emails in favor or real monitoring
by eedri (oVirt JIRA)
[ https://ovirt-jira.atlassian.net/browse/OVIRT-1087?page=com.atlassian.jir... ]
eedri reassigned OVIRT-1087:
----------------------------
Assignee: Evgheni Dereveanchin (was: infra)
> drop gerrit log watch emails in favor or real monitoring
> --------------------------------------------------------
>
> Key: OVIRT-1087
> URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1087
> Project: oVirt - virtualization made easy
> Issue Type: Task
> Reporter: eedri
> Assignee: Evgheni Dereveanchin
>
> Right now Gerrit sends regular emails from log-watch with various info, such as [1].
> We need to go over these alerts and see if some make sense to convert to alerts on nagios and disable the rest.
> [1].
> /changes/67155/revisions/57f6a24d70230c5e7 ... README/reviewed: 2 Time(s)
> /changes/67587/revisions/807e4950b3a74d127 ... 6f9f5be2/rebase: 1 Time(s)
> /changes/68315/revisions/182e8209319a41adc ... kefile/reviewed: 2 Time(s)
> /changes/70704/edit?list: 1 Time(s)
> /changes/71065/reviewers/1001042: 1 Time(s)
> /changes/71129/revisions/07edcd447865468bc ... 086f7545/drafts: 2 Time(s)
> /changes/71315/reviewers/1000458: 1 Time(s)
> /changes/71337/revisions/3f37d51915b0c7235 ... 829ed631/submit: 1 Time(s)
> /changes/?q=change:70704+has:draft&O=0: 1 Time(s)
> /changes/?q=is:open+owner:self&q=is:open+r ... +limit:10&O=881: 2 Time(s)
> 404 Not Found
> /changes/55626/detail?O=50004: 1 Time(s)
> /changes/70594/detail?O=50004: 2 Time(s)
> /changes/71272/detail?O=50004: 1 Time(s)
> /gerrit_ui/undefined.cache.js: 1 Time(s)
> /ovirt-system-tests: 2 Time(s)
--
This message was sent by Atlassian JIRA
(v1000.1092.1#100053)
7 years, 6 months
[JIRA] (OVIRT-1500) CI on ovirt-engine patch succeeded, but didn't mark CI=+1 on gerrit
by Allon Mureinik (oVirt JIRA)
[ https://ovirt-jira.atlassian.net/browse/OVIRT-1500?page=com.atlassian.jir... ]
Allon Mureinik commented on OVIRT-1500:
---------------------------------------
[~eedri] This is the first time I encountered it. If it's not worth the tradeoff, we can close the ticket - that's your call to make.
> CI on ovirt-engine patch succeeded, but didn't mark CI=+1 on gerrit
> -------------------------------------------------------------------
>
> Key: OVIRT-1500
> URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1500
> Project: oVirt - virtualization made easy
> Issue Type: Bug
> Components: Gerrit/git, oVirt CI
> Reporter: Allon Mureinik
> Assignee: infra
>
> In my recent engine patch https://gerrit.ovirt.org/#/c/78911 the CI passed, but was not marked as so in gerrit.
> Looking through gerrit's history, here's what I *think* is going on:
> 08:41 - patchset #1 is submitted
> 08:41 - CI (check-patch) starts for both EL7 and fc25
> _At this point I noticed a typo in the commit method. I fixed it and posted patchset #2_
> 08:42 - patchset #2 is submitted
> _CI was never triggered for patchset #2. Is this because the only difference between it and patchset #1 is the commit message?_
> 09:15 - CI for patchset #1 completes, and marks *patchset #1* with CI=+1. This is not inherited to patchset #2 for some reason.
> Expected behavior:
> If CI for patchset #1 passed and the only difference is the commit message (not the parent commit and not the content), I'd expect the CI score to be inherited.
--
This message was sent by Atlassian JIRA
(v1000.1092.1#100053)
7 years, 6 months
[JIRA] (OVIRT-1064) Re-factor experimental groovy script
by eedri (oVirt JIRA)
[ https://ovirt-jira.atlassian.net/browse/OVIRT-1064?page=com.atlassian.jir... ]
eedri commented on OVIRT-1064:
------------------------------
[~bkorren(a)redhat.com] is this in progress or done perhaps?
> Re-factor experimental groovy script
> ------------------------------------
>
> Key: OVIRT-1064
> URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1064
> Project: oVirt - virtualization made easy
> Issue Type: Improvement
> Components: oVirt CI
> Reporter: Barak Korren
> Assignee: infra
>
> The experimental Groovy scripts has several issues that should be resolved by re-factoring it:
> # Turn the variables defined in the top of the file into constants with upper case manes to ensure they're not used in functions by mistake
> # Remove unused arguments from functions
> # Move the main() function definition to the top of the file so its easier to quickly see an overview of the whole flow
> # Use the "global" constants inside main() instead of passing gazillion parameters to it.
> # Pass parameters directly to function instead of turning them into strings and using interpolation
> # Use 'for-each' loops instead of C-style indexed loops.
> # Use consistent indention throughout the file
--
This message was sent by Atlassian JIRA
(v1000.1092.1#100053)
7 years, 6 months
[JIRA] (OVIRT-1014) avoid repetition in automation/*packages
by eedri (oVirt JIRA)
[ https://ovirt-jira.atlassian.net/browse/OVIRT-1014?page=com.atlassian.jir... ]
eedri commented on OVIRT-1014:
------------------------------
[~danken(a)redhat.com] does this requirement is still needed?
[~bkorren(a)redhat.com] if its still relevant, maybe worth attaching it to the main ticket for V2 std-ci.
> avoid repetition in automation/*packages
> ----------------------------------------
>
> Key: OVIRT-1014
> URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1014
> Project: oVirt - virtualization made easy
> Issue Type: New Feature
> Reporter: danken
> Assignee: infra
>
> Currently we have many automation/*packages* files, mostly repeating each other.
> Most of the information there is then repeated in vdsm.spec as well.
> It would be nice to have a hierarchical way to define packages. E.g. having most packages in automation/build-artifacts-manual.packages, and adding el7-specific dependencies in avoid repetition in automation/build-artifacts-manual.packages.el7.
> It would be even better to have a single palce (yaml file?) to declare each required package and its version, for each platform/architecture. We can then use it to generate the spec file.
--
This message was sent by Atlassian JIRA
(v1000.1092.1#100053)
7 years, 6 months
[JIRA] (OVIRT-1009) Re: When trigged manually the check-merged job
by eedri (oVirt JIRA)
[ https://ovirt-jira.atlassian.net/browse/OVIRT-1009?page=com.atlassian.jir... ]
eedri updated OVIRT-1009:
-------------------------
Resolution: Cannot Reproduce
Status: Done (was: To Do)
please reopen if you see it again and provide details.
> Re: When trigged manually the check-merged job
> ----------------------------------------------
>
> Key: OVIRT-1009
> URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1009
> Project: oVirt - virtualization made easy
> Issue Type: By-EMAIL
> Reporter: eedri
> Assignee: infra
>
> Adding infra-support to open a ticket on it.
> But it will help if you'll provide example of what you are expecting to see
> so we'll check its possible.
> On Mon, Jan 9, 2017 at 11:32 AM, Yaniv Bronheim <ybronhei(a)redhat.com> wrote:
> > When trigged manually the check-merged job for specific patch we don't see
> > the link in the patch History - see in https://gerrit.ovirt.org/#/
> > c/69802/3
> >
> > it used to be different .. can we get the link back?
> >
> > Thanks.
> >
> > --
> > *Yaniv Bronhaim.*
> >
> --
> Eyal Edri
> Associate Manager
> RHV DevOps
> EMEA ENG Virtualization R&D
> Red Hat Israel
> phone: +972-9-7692018
> irc: eedri (on #tlv #rhev-dev #rhev-integ)
--
This message was sent by Atlassian JIRA
(v1000.1092.1#100053)
7 years, 6 months