Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enable sample imagestreams and templates on Z #225

Merged
merged 7 commits into from Jun 20, 2020

Conversation

gabemontero
Copy link
Contributor

No description provided.

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jan 20, 2020
@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 20, 2020
@gabemontero
Copy link
Contributor Author

/retest

@gabemontero
Copy link
Contributor Author

/retest

@gabemontero
Copy link
Contributor Author

/test e2e-aws-operator
/test e2e-aws-image-ecosystem

@gabemontero
Copy link
Contributor Author

/test e2e-aws-jenkins

@gabemontero
Copy link
Contributor Author

need openshift/origin#24434 for e2e-aws-jenkins

@gabemontero
Copy link
Contributor Author

/test e2e-aws-jenkins

1 similar comment
@gabemontero
Copy link
Contributor Author

/test e2e-aws-jenkins

@openshift-ci-robot openshift-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jan 24, 2020
@gabemontero
Copy link
Contributor Author

ok @pweil- @bparees @adambkaplan @ausil @yselkowitz here is my original version of the unlock s390/ppc commit (which I had pointed @yselkowitz to), along with the s390x/ppc64le content from openshift/library which we just merged/processed

We can either use this PR (associating it with @yselkowitz's BZs to facilitate backports) or he can duplicate this in his.

Per our meeting, placing a

/hold

on this until we get the comeback on the test coverage.

In the interim, I'll monitor the e2e's in case any non-CI flakes/issues occur.

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 24, 2020
@gabemontero
Copy link
Contributor Author

/test unit

tmp/build/library-sync.sh Show resolved Hide resolved
tmp/build/library-sync.sh Outdated Show resolved Hide resolved
test/e2e/cluster_samples_operator_test.go Show resolved Hide resolved
@gabemontero
Copy link
Contributor Author

/test unit

@gabemontero
Copy link
Contributor Author

/test e2e-aws-operator

@gabemontero
Copy link
Contributor Author

/retest

@gabemontero
Copy link
Contributor Author

/test e2e-aws-operator

@gabemontero
Copy link
Contributor Author

/test e2e-aws-jenkins

@gabemontero
Copy link
Contributor Author

ok finally got a clean jenkins install, but of course are now running into the need for openshift/origin#24448

@gabemontero
Copy link
Contributor Author

/retest

@openshift-bot
Copy link
Contributor

/retest

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

@gabemontero
Copy link
Contributor Author

in ci-flake cycle ... pulling out of merge queue, try to let CI settle

/hold

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jun 20, 2020
@gabemontero
Copy link
Contributor Author

/test e2e-aws-builds
/test e2e-aws-jenkins

@gabemontero
Copy link
Contributor Author

/retest

@gabemontero
Copy link
Contributor Author

@akram @waveywaves it looks like one of the 3 changes between to 2 of you to
has broke pipeline_origin_bld.go again in 4.6 / master branch since I fixed it last month.

Perhaps related to you all trying to get the test passing in earlier versions of OCP (4.2 / 4.3 / 4.4) ?

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jun 20, 2020
@gabemontero
Copy link
Contributor Author

/override e2e-aws-builds

@gabemontero
Copy link
Contributor Author

/override e2e-aws-jenkins

@openshift-ci-robot
Copy link
Contributor

@gabemontero: /override requires a failed status context to operate on.
The following unknown contexts were given:

  • e2e-aws-builds

Only the following contexts were expected:

  • ci/prow/e2e-aws
  • ci/prow/e2e-aws-builds
  • ci/prow/e2e-aws-image-ecosystem
  • ci/prow/e2e-aws-jenkins
  • ci/prow/e2e-aws-operator
  • ci/prow/e2e-aws-upgrade
  • ci/prow/images
  • ci/prow/unit
  • tide

In response to this:

/override e2e-aws-builds

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.

1 similar comment
@openshift-ci-robot
Copy link
Contributor

@gabemontero: /override requires a failed status context to operate on.
The following unknown contexts were given:

  • e2e-aws-builds

Only the following contexts were expected:

  • ci/prow/e2e-aws
  • ci/prow/e2e-aws-builds
  • ci/prow/e2e-aws-image-ecosystem
  • ci/prow/e2e-aws-jenkins
  • ci/prow/e2e-aws-operator
  • ci/prow/e2e-aws-upgrade
  • ci/prow/images
  • ci/prow/unit
  • tide

In response to this:

/override e2e-aws-builds

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
Copy link
Contributor

@gabemontero: /override requires a failed status context to operate on.
The following unknown contexts were given:

  • e2e-aws-jenkins

Only the following contexts were expected:

  • ci/prow/e2e-aws
  • ci/prow/e2e-aws-builds
  • ci/prow/e2e-aws-image-ecosystem
  • ci/prow/e2e-aws-jenkins
  • ci/prow/e2e-aws-operator
  • ci/prow/e2e-aws-upgrade
  • ci/prow/images
  • ci/prow/unit
  • tide

In response to this:

/override e2e-aws-jenkins

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.

1 similar comment
@openshift-ci-robot
Copy link
Contributor

@gabemontero: /override requires a failed status context to operate on.
The following unknown contexts were given:

  • e2e-aws-jenkins

Only the following contexts were expected:

  • ci/prow/e2e-aws
  • ci/prow/e2e-aws-builds
  • ci/prow/e2e-aws-image-ecosystem
  • ci/prow/e2e-aws-jenkins
  • ci/prow/e2e-aws-operator
  • ci/prow/e2e-aws-upgrade
  • ci/prow/images
  • ci/prow/unit
  • tide

In response to this:

/override e2e-aws-jenkins

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.

@gabemontero
Copy link
Contributor Author

/override ci/prow/e2e-aws-builds
/override ci/prow/e2e-aws-jenkins

@gabemontero
Copy link
Contributor Author

gabemontero commented Jun 20, 2020

everything else in e2e-aws-builds passed in the last run of it
plus, this PR is only updating s390x/ppc64le, hence no updates to the existing x86_64 templates/imagestreams

and had prior clean runs prior to the recent changes in https://github.com/openshift/jenkins-client-plugin/commits/master/examples/jenkins-image-sample.groovy

hence chose ot use override

@gabemontero gabemontero merged commit 0038cc3 into openshift:master Jun 20, 2020
@openshift-ci-robot
Copy link
Contributor

@gabemontero: Overrode contexts on behalf of gabemontero: ci/prow/e2e-aws-builds, ci/prow/e2e-aws-jenkins

In response to this:

/override ci/prow/e2e-aws-builds
/override ci/prow/e2e-aws-jenkins

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.

1 similar comment
@openshift-ci-robot
Copy link
Contributor

@gabemontero: Overrode contexts on behalf of gabemontero: ci/prow/e2e-aws-builds, ci/prow/e2e-aws-jenkins

In response to this:

/override ci/prow/e2e-aws-builds
/override ci/prow/e2e-aws-jenkins

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.

@gabemontero gabemontero deleted the more-s390 branch June 20, 2020 19:01
@yselkowitz
Copy link
Contributor

/cherry-pick release-4.5

@openshift-cherrypick-robot

@yselkowitz: only openshift org members may request cherry picks. You can still do the cherry-pick manually.

In response to this:

/cherry-pick release-4.5

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.

1 similar comment
@openshift-cherrypick-robot

@yselkowitz: only openshift org members may request cherry picks. You can still do the cherry-pick manually.

In response to this:

/cherry-pick release-4.5

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.

@akram
Copy link

akram commented Jun 24, 2020

ack @gabemontero

@gabemontero gabemontero changed the title enable sample imagestreams and templates on Z Bug 1850694: enable sample imagestreams and templates on Z Jun 24, 2020
@openshift-ci-robot
Copy link
Contributor

@gabemontero: An error was encountered updating to the MODIFIED state for bug 1850694 on the Bugzilla server at https://bugzilla.redhat.com:

code 32000: Red Hat Bugzilla's database reported a query serialization error. Most likely this occurred because another user or process attempted to change the same data that you were attempting to change. Please press Back and retry the transaction.

 UPDATE bugs SET bug_status = ? WHERE bug_id = ? at /var/www/html/bugzilla/Bugzilla/Object.pm line 544. Bugzilla::Object::update('Bugzilla::Bug=HASH(0x55e857778910)') called at /var/www/html/bugzilla/Bugzilla/Bug.pm line 1168 Bugzilla::Bug::update('Bugzilla::Bug=HASH(0x55e857778910)') called at /var/www/html/bugzilla/Bugzilla/WebService/Bug.pm line 851 Bugzilla::WebService::Bug::update('Bugzilla::WebService::Server::REST::Bugzilla::WebService::Bug...', 'HASH(0x55e856bdfb68)') called at (eval 2168) line 1 eval ' $procedure->{code}->($self, $params) ;' called at /usr/share/perl5/vendor_perl/JSON/RPC/Legacy/Server.pm line 208 JSON::RPC::Legacy::Server::_handle('Bugzilla::WebService::Server::REST::Bugzilla::WebService::Bug...', 'HASH(0x55e856bc8708)') called at /var/www/html/bugzilla/Bugzilla/WebService/Server/JSONRPC.pm line 295 Bugzilla::WebService::Server::JSONRPC::_handle('Bugzilla::WebService::Server::REST::Bugzilla::WebService::Bug...', 'HASH(0x55e856bc8708)') called at /var/www/html/bugzilla/Bugzilla/WebService/Server/REST.pm line 98 Bugzilla::WebService::Server::REST::handle('Bugzilla::WebService::Server::REST::Bugzilla::WebService::Bug...') called at /var/www/html/bugzilla/rest.cgi line 30 ModPerl::ROOT::Bugzilla::ModPerl::ResponseHandler::var_www_html_bugzilla_rest_2ecgi::handler('Apache2::RequestRec=SCALAR(0x55e8571a2b78)') 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(0x55e8572e1240)') called at /usr/lib64/perl5/vendor_perl/ModPerl/RegistryCooker.pm line 173 ModPerl::RegistryCooker::default_handler('Bugzilla::ModPerl::ResponseHandler=HASH(0x55e8572e1240)') called at /usr/lib64/perl5/vendor_perl/ModPerl/Registry.pm line 32 ModPerl::Registry::handler('Bugzilla::ModPerl::ResponseHandler', 'Apache2::RequestRec=SCALAR(0x55e8571a2b78)') called at /var/www/html/bugzilla/mod_perl.pl line 139 Bugzilla::ModPerl::ResponseHandler::handler('Bugzilla::ModPerl::ResponseHandler', 'Apache2::RequestRec=SCALAR(0x55e8571a2b78)') called at (eval 2168) line 0 eval {...} called at (eval 2168) line 0 

Please contact an administrator to resolve this issue, then request a bug refresh with /bugzilla refresh.

In response to this:

Bug 1850694: enable sample imagestreams and templates on Z

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
Copy link
Contributor

@gabemontero: All pull requests linked via external trackers have merged: . Bugzilla bug 1850694 has been moved to the MODIFIED state.

In response to this:

Bug 1850694: enable sample imagestreams and templates on Z

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.

@gabemontero gabemontero changed the title Bug 1850694: enable sample imagestreams and templates on Z enable sample imagestreams and templates on Z Jun 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants