Skip to content

Replace genny code generation in pkg/maputil with generics#3401

Merged
misberner merged 5 commits intomasterfrom
mi/maputil-genny-to-generics
Oct 13, 2022
Merged

Replace genny code generation in pkg/maputil with generics#3401
misberner merged 5 commits intomasterfrom
mi/maputil-genny-to-generics

Conversation

@misberner
Copy link
Contributor

Description

Use generics instead of code generation for maputil.

Checklist

  • Investigated and inspected CI test results
  • Unit test and regression tests added
  • Evaluated and added CHANGELOG entry if required
  • Determined and documented upgrade steps
  • Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.

Testing Performed

CI

@ghost
Copy link

ghost commented Oct 12, 2022

Images are ready for the commit at 3253ff9.

To use with deploy scripts, first export MAIN_IMAGE_TAG=3.72.x-294-g3253ff99a3.

@misberner misberner force-pushed the mi/maputil-genny-to-generics branch from b9dbe9f to 16699c7 Compare October 13, 2022 08:26
@misberner
Copy link
Contributor Author

/retest

@openshift-ci
Copy link

openshift-ci bot commented Oct 13, 2022

@misberner: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/osd-gcp-qa-e2e-tests 0bc9c53 link false /test osd-gcp-qa-e2e-tests
ci/prow/osd-aws-qa-e2e-tests 0bc9c53 link false /test osd-aws-qa-e2e-tests
ci/prow/rosa-qa-e2e-tests 0bc9c53 link false /test rosa-qa-e2e-tests
ci/prow/gke-upgrade-tests 0bc9c53 link false /test gke-upgrade-tests
ci/prow/gke-postgres-qa-e2e-tests 3253ff9 link false /test gke-postgres-qa-e2e-tests
ci/prow/gke-qa-e2e-tests 3253ff9 link false /test gke-qa-e2e-tests

Full PR test history. Your PR dashboard.

Details

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. I understand the commands that are listed here.

@misberner
Copy link
Contributor Author

Verified that all test failures are either spurious (tests shouldn't even run, not sure what's up with that), or are due to an already addressed flake

@misberner misberner merged commit 75f005c into master Oct 13, 2022
@misberner misberner deleted the mi/maputil-genny-to-generics branch October 13, 2022 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants