Bug 1867510: fix CVP issues due to incorrect labels set by paulfantom · Pull Request #348 · openshift/telemeter

paulfantom · 2020-08-10T10:35:06Z

Fixing maintainer label to end with @redhat.com as this is expected
Overriding summary label as it is inherited from base image and results with an incorrect value
~~explicit golang 1.14 usage~~
~~explicit usage of openshift base image (as it is used in OSBS)~~

/cc @openshift/openshift-team-monitoring

openshift-ci-robot · 2020-08-10T10:35:11Z

@paulfantom: An error was encountered adding this pull request to the external tracker bugs for bug 1867510 on the Bugzilla server at https://bugzilla.redhat.com:

JSONRPC error 32000: There was an error reported for a GitHub REST call. URL: https://api.github.com/repos/openshift/telemeter/pulls/348 Error: 403 Forbidden at /loader/0x55966fadada8/Bugzilla/Extension/ExternalBugs/Type/GitHub.pm line 111. at /loader/0x55966fadada8/Bugzilla/Extension/ExternalBugs/Type/GitHub.pm line 111. eval {...} called at /loader/0x55966fadada8/Bugzilla/Extension/ExternalBugs/Type/GitHub.pm line 98 Bugzilla::Extension::ExternalBugs::Type::GitHub::_do_rest_call('Bugzilla::Extension::ExternalBugs::Type::GitHub=HASH(0x559672...', 'https://api.github.com/repos/openshift/telemeter/pulls/348', 'GET') called at /loader/0x55966fadada8/Bugzilla/Extension/ExternalBugs/Type/GitHub.pm line 62 Bugzilla::Extension::ExternalBugs::Type::GitHub::get_data('Bugzilla::Extension::ExternalBugs::Type::GitHub=HASH(0x559672...', 'Bugzilla::Extension::ExternalBugs::Bug=HASH(0x559671f92a20)') called at /loader/0x55966fadada8/Bugzilla/Extension/ExternalBugs/Bug.pm line 302 eval {...} called at /loader/0x55966fadada8/Bugzilla/Extension/ExternalBugs/Bug.pm line 302 Bugzilla::Extension::ExternalBugs::Bug::update_ext_info('Bugzilla::Extension::ExternalBugs::Bug=HASH(0x559671f92a20)', 1) called at /loader/0x55966fadada8/Bugzilla/Extension/ExternalBugs/Bug.pm line 125 Bugzilla::Extension::ExternalBugs::Bug::create('Bugzilla::Extension::ExternalBugs::Bug', 'HASH(0x559671f8eb10)') called at /var/www/html/bugzilla/extensions/ExternalBugs/Extension.pm line 877 Bugzilla::Extension::ExternalBugs::bug_start_of_update('Bugzilla::Extension::ExternalBugs=HASH(0x5596720ff118)', 'HASH(0x559672bf8aa0)') called at /var/www/html/bugzilla/Bugzilla/Hook.pm line 21 Bugzilla::Hook::process('bug_start_of_update', 'HASH(0x559672bf8aa0)') called at /var/www/html/bugzilla/Bugzilla/Bug.pm line 1170 Bugzilla::Bug::update('Bugzilla::Bug=HASH(0x5596724ce128)') called at /loader/0x55966fadada8/Bugzilla/Extension/ExternalBugs/WebService.pm line 88 Bugzilla::Extension::ExternalBugs::WebService::add_external_bug('Bugzilla::WebService::Server::JSONRPC::Bugzilla::Extension::E...', 'HASH(0x559672142430)') called at (eval 2287) line 1 eval ' $procedure->{code}->($self, @params) ;' called at /usr/share/perl5/vendor_perl/JSON/RPC/Legacy/Server.pm line 220 JSON::RPC::Legacy::Server::_handle('Bugzilla::WebService::Server::JSONRPC::Bugzilla::Extension::E...', 'HASH(0x55967217fd20)') called at /var/www/html/bugzilla/Bugzilla/WebService/Server/JSONRPC.pm line 295 Bugzilla::WebService::Server::JSONRPC::_handle('Bugzilla::WebService::Server::JSONRPC::Bugzilla::Extension::E...', 'HASH(0x55967217fd20)') called at /usr/share/perl5/vendor_perl/JSON/RPC/Legacy/Server.pm line 126 JSON::RPC::Legacy::Server::handle('Bugzilla::WebService::Server::JSONRPC::Bugzilla::Extension::E...') called at /var/www/html/bugzilla/Bugzilla/WebService/Server/JSONRPC.pm line 70 Bugzilla::WebService::Server::JSONRPC::handle('Bugzilla::WebService::Server::JSONRPC::Bugzilla::Extension::E...') called at /var/www/html/bugzilla/jsonrpc.cgi line 31 ModPerl::ROOT::Bugzilla::ModPerl::ResponseHandler::var_www_html_bugzilla_jsonrpc_2ecgi::handler('Apache2::RequestRec=SCALAR(0x559672ab41c0)') called at /usr/lib64/perl5/vendor_perl/ModPerl/RegistryCooker.pm line 207 eval {...} called at /usr/lib64/perl5/vendor_perl/ModPerl/RegistryCooker.pm line 207 ModPerl::RegistryCooker::run('Bugzilla::ModPerl::ResponseHandler=HASH(0x55967260bec0)') called at /usr/lib64/perl5/vendor_perl/ModPerl/RegistryCooker.pm line 173 ModPerl::RegistryCooker::default_handler('Bugzilla::ModPerl::ResponseHandler=HASH(0x55967260bec0)') called at /usr/lib64/perl5/vendor_perl/ModPerl/Registry.pm line 32 ModPerl::Registry::handler('Bugzilla::ModPerl::ResponseHandler', 'Apache2::RequestRec=SCALAR(0x559672ab41c0)') called at /var/www/html/bugzilla/mod_perl.pl line 139 Bugzilla::ModPerl::ResponseHandler::handler('Bugzilla::ModPerl::ResponseHandler', 'Apache2::RequestRec=SCALAR(0x559672ab41c0)') called at (eval 2287) line 0 eval {...} called at (eval 2287) line 0
Please contact an administrator to resolve this issue, then request a bug refresh with /bugzilla refresh.

Details

In response to this:

Bug 1867510: use openshift base image; fix CVP issues due to incorrect labels set

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

paulfantom · 2020-08-10T10:40:17Z

/bugzilla refresh

openshift-ci-robot · 2020-08-10T10:40:22Z

@paulfantom: This pull request references Bugzilla bug 1867510, which is valid.

3 validation(s) were run on this bug

bug is open, matching expected state (open)
bug target release (4.6.0) matches configured target release for branch (4.6.0)
bug is in the state POST, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)

Details

In response to this:

/bugzilla refresh

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

s-urbaniak · 2020-08-10T11:05:19Z

/lgtm

s-urbaniak · 2020-08-10T11:06:08Z

the integration test look suspicious, maybe @openshift/openshift-team-telemetry can tell more?

openshift-bot · 2020-08-10T11:14:44Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T13:00:37Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T14:16:54Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T14:29:44Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T14:42:47Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T14:55:56Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T15:08:46Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T16:39:53Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T16:52:49Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T17:05:46Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T17:18:48Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T17:31:47Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T17:44:47Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T17:57:47Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T19:02:45Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T19:15:46Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T21:12:56Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-10T21:25:48Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-11T03:06:49Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-11T23:02:41Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-11T23:15:40Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-11T23:41:38Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-12T08:34:36Z

/retest

Please review the full test history for this PR and help us cut down flakes.

simonpasquier · 2020-08-12T08:42:04Z

/hold
looks like it's constantly failing.

s-urbaniak · 2020-08-18T07:32:36Z

/lgtm

s-urbaniak · 2020-08-18T07:33:17Z

@openshift/openshift-team-telemetry can you please advise on the failing integration test? 🤔

paulfantom · 2020-08-31T10:30:46Z

/retest

paulfantom · 2020-08-31T13:34:22Z

/retitle Bug 1867510: fix CVP issues due to incorrect labels set

paulfantom · 2020-08-31T13:34:56Z

removed changing base images and only added correct labels.

/unhold

openshift-ci-robot · 2020-08-31T13:35:21Z

@paulfantom: This pull request references Bugzilla bug 1867510, which is valid.

3 validation(s) were run on this bug

bug is open, matching expected state (open)
bug target release (4.6.0) matches configured target release for branch (4.6.0)
bug is in the state POST, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)

Details

In response to this:

Bug 1867510: fix CVP issues due to incorrect labels set

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

s-urbaniak · 2020-08-31T13:35:59Z

/lgtm

openshift-ci-robot · 2020-08-31T13:36:06Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: paulfantom, s-urbaniak

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

~~OWNERS~~ [paulfantom,s-urbaniak]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

openshift-bot · 2020-08-31T17:03:05Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2020-08-31T17:16:14Z

/retest

Please review the full test history for this PR and help us cut down flakes.

openshift-ci-robot · 2020-08-31T19:08:06Z

@paulfantom: All pull requests linked via external trackers have merged:

Bugzilla bug 1867510 has been moved to the MODIFIED state.

Details

In response to this:

Bug 1867510: fix CVP issues due to incorrect labels set

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

openshift-ci-robot requested a review from a team August 10, 2020 10:35

openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 10, 2020

openshift-ci-robot added bugzilla/severity-low Referenced Bugzilla bug's severity is low for the branch this PR is targeting. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. labels Aug 10, 2020

openshift-ci-robot assigned s-urbaniak Aug 10, 2020

openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Aug 10, 2020

openshift-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. and removed lgtm Indicates that a PR is ready to be merged. labels Aug 12, 2020

openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Aug 18, 2020

paulfantom force-pushed the cvp branch from 618ecbf to 8f1bed5 Compare August 18, 2020 12:59

openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Aug 18, 2020

squat mentioned this pull request Aug 25, 2020

Bug 1872080: Updating Dockerfile baseimages to mach ocp-build-data config #350

Merged

paulfantom force-pushed the cvp branch from 8f1bed5 to 0d9d6f9 Compare August 31, 2020 09:15

fix CVP issues due to incorrect labels set

849f4c2

paulfantom force-pushed the cvp branch from 0d9d6f9 to 849f4c2 Compare August 31, 2020 13:34

openshift-ci-robot changed the title ~~Bug 1867510: use openshift base image; fix CVP issues due to incorrect labels set~~ Bug 1867510: fix CVP issues due to incorrect labels set Aug 31, 2020

openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 31, 2020

openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Aug 31, 2020

openshift-merge-robot merged commit cca9f56 into openshift:master Aug 31, 2020

openshift-ci-robot mentioned this pull request Sep 1, 2020

Bug 1867510: Fix container image metadata openshift/kube-rbac-proxy#28

Merged

Conversation

paulfantom commented Aug 10, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

openshift-ci-robot commented Aug 10, 2020

Uh oh!

paulfantom commented Aug 10, 2020

Uh oh!

openshift-ci-robot commented Aug 10, 2020

Uh oh!

s-urbaniak commented Aug 10, 2020

Uh oh!

s-urbaniak commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 10, 2020

Uh oh!

openshift-bot commented Aug 11, 2020

Uh oh!

openshift-bot commented Aug 11, 2020

Uh oh!

openshift-bot commented Aug 11, 2020

Uh oh!

openshift-bot commented Aug 11, 2020

Uh oh!

openshift-bot commented Aug 12, 2020

Uh oh!

simonpasquier commented Aug 12, 2020

Uh oh!

s-urbaniak commented Aug 18, 2020

Uh oh!

s-urbaniak commented Aug 18, 2020

Uh oh!

paulfantom commented Aug 31, 2020

Uh oh!

paulfantom commented Aug 31, 2020

Uh oh!

paulfantom commented Aug 31, 2020

Uh oh!

openshift-ci-robot commented Aug 31, 2020

Uh oh!

s-urbaniak commented Aug 31, 2020

Uh oh!

openshift-ci-robot commented Aug 31, 2020

Uh oh!

openshift-bot commented Aug 31, 2020

Uh oh!

paulfantom commented Aug 10, 2020 •

edited

Loading