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

Rebase to latest upstream release #248

Merged
merged 2,055 commits into from
Feb 2, 2022
Merged

Rebase to latest upstream release #248

merged 2,055 commits into from
Feb 2, 2022

Conversation

alexander-demicev
Copy link

@alexander-demicev alexander-demicev commented Dec 13, 2021

This commit rebases the cluster-api-provider-azure openshift patches on top of the kubernetes-sigs/ cluster-api-provider-azure main branch after.
There are several commits that we carry on top of the upstream cluster-api-provider-azure and the rebase process allows us to preserve those. Here is a description of the process I used to create this PR.

(replicated @elmiko's process within openshift/kubernetes-autoscaler)

Process

First we need to identify the carry commits that we currently have, this is done against our previous rebase (or fork in this case) to catch new changes. Once identified we will drop commits which have merged upstream and only carry unique commits. (see below for the carried and dropped commits).

Identify carry commits:

git log --oneline --no-merges c356b7e776a70f5ce70a039641ef5b2e992d2d43..openshift/master

where b4be01 is the last upstream commit in our fork. This is the
list of commits we will need to apply onto the new upstream version of the
cluster-api. ideally, some of these commits can be dropped.

After identifying the carry commits, the next step is to create the new commit-tree that will be used for the rebase and then cherry pick the carry commits into the new branch. The following commands cover these steps:

$ git remote update # make sure we update our refs
$ git checkout c356b7e776a70f5ce70a039641ef5b2e992d2d43
$ git checkout -b merge-tmp # create a temporary branch for our merge commit
$ git checkout openshift/master # we want to be at the tip of the openshift master branch when we run the next command
$ echo 'merge latest upstream changes' | git commit-tree merge-tmp^{tree} -p HEAD -p merge-tmp -F - # create a new merge commit for our history
deadbeef12345678 # id of new merge commit
$ git branch rebase-upstream-latest deadbeef12345678 # create a new branch for the cherry-pick work
$ git checkout rebase-upstream-latest
$ git cherry-pick <carry commits> # cherry pick the needed commits into the new branch

With the rebase-upstream-latest branch in place, I cherry picked the carry commits which we should carry.

Carried Commits

These commits are integral to our CI platform, or are specific to the releases we create for OpenShift.

   498c4ae9e UPSTREAM: <carry>: Add vendor directory
   3c05c04f3 UPSTREAM: <carry>: Add openshift specific changes

Changed Commits

---

Dropped Commits

Click to expand
  • 02b9633 UPSTREAM: : update golang version to 1.17
  • 987eadd UPSTREAM: : add BUILD_IMAGES to Makefile and update nondocker builds
  • 6cf23a6 UPSTREAM: : vendor: Bump API for Image Type field
  • 796ab2b UPSTREAM: : Add VM Support for Marketplace Purchase Plans
  • 2040aea UPSTREAM: : Revert: Populate Marketplace Purchase Plan
  • 06a8c54 UPSTREAM: : Populate Marketplace Purchase Plan
  • 23f979c UPSTREAM: : openshift: Migrate to opeshift/api
  • 6d354df UPSTREAM: : openshift: Update dependecies
  • d164055 UPSTREAM: : openshift: Add stack hub support to actuator
  • 5cbccc0 UPSTREAM: : openshift: Set up stack hub client
  • a47741f UPSTREAM: : openshift: Add stack hub services
  • 9ed7cbf UPSTREAM: : openshift: Add availability set name field to api
  • 047270d UPSTREAM: : openshift: Revendor azure sdk
  • c590127 UPSTREAM: : Update machine-api-operator dependencies
  • 6c62b91 UPSTREAM: : openshift: Fix AS spec pointer
  • d7b1ba9 UPSTREAM: : openshift: Skip AS creation when machineset label is missing
  • c57dbd0 UPSTREAM: : openshift: Add function for getting AS name
  • dbc5db2 UPSTREAM: : openshift: Add unit tests for availability sets
  • 6c40684 UPSTREAM: : openshift: Create availability set if no zone found for region
  • ebf347a UPSTREAM: : openshift: Add availabilitysets service
  • f58b315 UPSTREAM: : Enable accelerated networking
  • 50df88b UPSTREAM: : Azure library updates to support accelerated networking
  • c689e78 UPSTREAM: : Ensure HOME is set for cache setup
  • ee2200e UPSTREAM: : Update dependencies to K8s 1.22
  • a7c4034 UPSTREAM: : Bug 1993159: set Machine Spec during update
  • 1040454 UPSTREAM: : Bug 1988992: Create event only if the machine was modified
  • 5d94c79 UPSTREAM: : openshift: Fix spot instances price type
  • 7f78abe UPSTREAM: : openshift: Add in container vendor
  • 6bed0c3 UPSTREAM: : openshift: Updating ose-azure-machine-controllers images to be consistent with ART Reconciling with https://github.com/openshift/ocp-build-data/tree/691e628254f318ce56efda5edc7448ec743c37b8/images/ose-azure-machine-controllers.yml
  • 9dbeb20 UPSTREAM: : Update Git History check script to align with other repositories
  • 8301076 UPSTREAM: : openshift: Make test names more descriptive, fix wrong named test.
  • 3323fac UPSTREAM: : openshift: Improve logging message in case if vm not found
  • b7310cb UPSTREAM: : openshift: Changed Exist method logic, now all ProvisioningStates treats as existent, only in case if object not found machine counts non existent. Failed and Deleting provisioning states return error, for future reconcile. Logging extended.
  • 18bb900 UPSTREAM: : openshift: Introduce syncPeriod for reconciliation
  • b024905 UPSTREAM: : openshift: update controller-runtime dependency
  • 570afa5 UPSTREAM: : openshift: support dual-stack interfaces
  • b2eda16 UPSTREAM: : openshift: Update mao dependency for webhook v1 changes
  • 650c2cf UPSTREAM: : openshift: ensure CGO_ENABLED=0 on all makefile commands that need it
  • b70b9ef UPSTREAM: : openshift: update k8s dependencies to 1.21
  • 987bc3d UPSTREAM: : openshift: Update makefile to be able to run with podman and docker,change scripts to run without contianer
  • eb94fe6 UPSTREAM: : openshift: Only log error on wrong instance type for scale from zero
  • 053eaae UPSTREAM: : openshift: Ensure original conditions aren't mutated during reconcile
  • 231df73 UPSTREAM: : openshift: Add condition to show actuator exists condition on machine
  • 2a02dfe UPSTREAM: : openshift: Ensure response body is closed when we are finished with the request
  • adf818b UPSTREAM: : openshift: Updating ose-azure-machine-controllers builder & base images to be consistent with ART Reconciling with https://github.com/openshift/ocp-build-data/tree/5a1293dd0f380abf50c12d65c36655486d7745d0/images/ose-azure-machine-controllers.yml
  • 3442c7a UPSTREAM: : openshift: add SecurityProfile.EncryptionAtHost parameter to enable host-based VM encryption
  • a4c0a83 UPSTREAM: : openshift: Updating Azure VMSize list from autoscaler.
  • c66f7ba UPSTREAM: : openshift: Ensure Machine is marked interruptible as well as Node
  • 2caa107 UPSTREAM: : openshift: Update dependency for golang.org/x/crypto
  • 435b4de UPSTREAM: : openshift: Fix controllers for updated controller-runtime
  • 5c76700 UPSTREAM: : openshift: Bump K8s dependencies to 1.20
  • 075372e UPSTREAM: : openshift: Ensure MAO dependency is up to date to include newest metrics
  • 88b9023 UPSTREAM: : openshift: Update termination handler to use node conditions
  • 608ffa0 UPSTREAM: : openshift: Bump scripts to Go 1.15
  • 3b380c8 UPSTREAM: : openshift: Bump Azure autorest/adal to v0.9.5
  • d38af56 UPSTREAM: : openshift: bump Azure SDK version
  • 45a9327 UPSTREAM: : openshift: Add verify-diff check in generate task
  • 4090a69 UPSTREAM: : openshift: Adding managed disk parameters to support DES encryption
  • 6c9c959 UPSTREAM: : openshift: add bugzilla component and subcomponent to OWNERS
  • 9b790fe UPSTREAM: : openshift: Fix missing logs due to mixed klog versions
  • 04e680e UPSTREAM: : openshift: Fix missing logs due to mixed klog versions
  • dd5e711 UPSTREAM: : openshift: Ensure the Virtual Machine provider state is set to Unknown when Failed
  • 86841cb UPSTREAM: : openshift: machine: Return full invalid configuration error
  • 274ce1d UPSTREAM: : openshift: vm: Add AdditionalUnattendContent on Windows machines
  • cd15c1c UPSTREAM: : openshift: Bug 1878880: re-vendor machine-api-operator at e0db6b65
  • 5862e3d UPSTREAM: : openshift: Bump K8s dependencies to v1.19.0
  • db990d8 UPSTREAM: : openshift: Updating Dockerfile.rhel baseimages to match ocp-build-data config
  • 7276393 UPSTREAM: : openshift: Fail machine if create request doesn't succeed
  • 72169c5 UPSTREAM: : openshift: Add Dockerfile.rhel to match build configuration in ocp-build-data
  • 29efd36 UPSTREAM: : openshift: update make generate
  • 201dd37 UPSTREAM: : openshift: Slow the default lease retry and renew rate for machine controller
  • f138f6b UPSTREAM: : openshift: Assert VM exists if VM state is Creating
  • 0d53a0d UPSTREAM: : openshift: Ensure Interruptible Instance Label is propogated onto the Node
  • 5955daf UPSTREAM: : openshift: Add unit tests for different cloud envs
  • 5279a03 UPSTREAM: : openshift: Update dependencies
  • d95ffcf UPSTREAM: : openshift: Add support for different cloud environments
  • 3248f7d UPSTREAM: : openshift: Increase leader election lease time
  • f1d6f90 UPSTREAM: : openshift: Revendor mao to bring Bug 1857175: enforce clusterID label via webhook and preserve old behaviour in the backend machine-api-operator#644
  • 3dad165 UPSTREAM: : openshift: Ensure empty max price does not cause error when creating a spot VM
  • ab01fde UPSTREAM: : openshift: Make dockerfile consistent with other providers
  • 044a01d UPSTREAM: : openshift: Set spot label in machine spec
  • 85d1eca UPSTREAM: : openshift: Add termination handler tests
  • d1ac2e1 UPSTREAM: : openshift: Add termination handler
  • 3cda328 UPSTREAM: : openshift: Add termination handler to Makefile and Dockerfile
  • 88a538b UPSTREAM: : openshift: Bump golang.org/x/text to v0.3.3
  • fa840d1 UPSTREAM: : openshift: Revendor MAO
  • 2e64806 UPSTREAM: : openshift: Add metrics for machine controllers
  • d46153e UPSTREAM: : openshift: Skip SSH setup on Windows VM
  • 7fe5391 UPSTREAM: : openshift: Implement leader election for azure provider
  • b85933f UPSTREAM: : openshift: Update azure sdk
  • 7d9c909 UPSTREAM: : openshift: Add support for spot instances
  • ff90663 UPSTREAM: : openshift: Fix confusing log in azure
  • 6183e48 UPSTREAM: : openshift: Run make generate
  • b7e45ab UPSTREAM: : openshift: add make generate to generate deepcopy code
  • 58e4ac0 UPSTREAM: : openshift: vendor k8s.io/code-generator
  • f74f13d UPSTREAM: : openshift: Add configv1 to scheme
  • ddd15df UPSTREAM: : openshift: Revendor MAO
  • 5d5d67e UPSTREAM: : openshift: Expose spot instances to provider spec and reconcile label
  • e66168e UPSTREAM: : openshift: Rename machine controller import alias
  • 72e2c4c UPSTREAM: : openshift: Add health checks
  • af08a66 UPSTREAM: : openshift: Add dgrigore to approvers
  • 17d4edc UPSTREAM: : openshift: Update machine-api-operator-dependency
  • 5840552 UPSTREAM: : openshift: Use ToLower for Subscription ID and Resource Group Name in Machine ProviderID
  • 325e1d5 UPSTREAM: : openshift: Add unknown state for externally removed machines
  • cc1374d UPSTREAM: : openshift: Bump all dependencies to kube 1.18
  • 9034812 UPSTREAM: : openshift: Removed deprecated v1apha1 api
  • 639c065 UPSTREAM: : openshift: Update dependencies
  • ac0a254 UPSTREAM: : openshift: Return invalid configuration errors when creating NICs with invalid config
  • cdc42ef UPSTREAM: : openshift: Fix breaks due to updated dependencies
  • 54c676a UPSTREAM: : openshift: Update dependencies
  • 8e31251 UPSTREAM: : openshift: Use Go 1.13 errors instead of github.com/pkg/errors
  • ad6ae03 UPSTREAM: : openshift: Update vendor
  • 98d5d87 UPSTREAM: : openshift: Remove MachineSet manager from image
  • 73a28c1 UPSTREAM: : openshift: Update golang/crypto to bring CVE-2020-9283 fix
  • 7bdbd72 UPSTREAM: : openshift: Bump to Go 1.13
  • 3fd3692 UPSTREAM: : openshift: Update OWNERS with current team members
  • 0dc1da0 UPSTREAM: : openshift: Restore original make unit behaviour
  • a3d66b6 UPSTREAM: : openshift: Remove bison from OWNERS file
  • 4fc8bbe UPSTREAM: : openshift: Support private IPv6 configurations
  • b304c7b UPSTREAM: : openshift: Revendor mao with timeout formatting fix
  • a0b4361 UPSTREAM: : openshift: Don't re-reconcile on invalid configuration
  • d015d54 UPSTREAM: : openshift: Add envtest binaries for testing
  • 9db40c2 UPSTREAM: : openshift: make vendor
  • 85de3ba UPSTREAM: : openshift: Add envTest tests for reconciler
  • 07a1c29 UPSTREAM: : openshift: Add tests for reconcile
  • d3921c6 UPSTREAM: : openshift: Add machineset controller
  • 8b91ed1 UPSTREAM: : openshift: Remove fqdn from public ip creation
  • d80999f UPSTREAM: : openshift: add docker and build task to makefile
  • e441cdf Add bin/ directory to gitignore
  • 72c8756 UPSTREAM: : openshift: Remove modification of LastUpdate in machine scope
  • 832a43a UPSTREAM: : openshift: Remove deprecated check in update
  • 8bfc93e UPSTREAM: : openshift: Revendor MAO with new node draining feature
  • 06be56f UPSTREAM: : openshift: Save status before Patch()
  • 8a9592f UPSTREAM: : openshift: OWNERS: remove frobware
  • 611a78a UPSTREAM: : openshift: Add unit test task to makefile
  • 7ec5691 UPSTREAM: : openshift: Replace update with patch
  • 937b4e6 UPSTREAM: : openshift: Vendor dependencies
  • 52c2232 UPSTREAM: : openshift: Remove unnecessary machine specific client
  • f3f269f UPSTREAM: : openshift: Update owners
  • d9d24c5 UPSTREAM: : openshift: allow managed identity to be unset
  • 925b15b UPSTREAM: : openshift: Bring mao and drop cluster struct references to satisfy actuator interface
  • c3ff86a UPSTREAM: : openshift: Revendor to drop openshift/cluster-api in favour of machine-api-operator
  • 39f21d6 UPSTREAM: : openshift: Fix statusCode conversion
  • d6bb862 UPSTREAM: : openshift: update scripts for enabling go modules
  • 9f6020e UPSTREAM: : openshift: revendor with go modules
  • 5f5e557 UPSTREAM: : openshift: remove unused make targets
  • 4b17bb4 UPSTREAM: : openshift: delete unused files
  • cbe6bdd UPSTREAM: : openshift: Fail with InvalidMachineConfiguration in Create when appropriate
  • c1d771c UPSTREAM: : openshift: services/networkinterfaces: fix application secuirty group collection
  • 593cfd4 UPSTREAM: : openshift: api: allow setting the security group and application security groups for machine in spec
  • d1d0550 UPSTREAM: : openshift: services/newtorkinterfaces: allow providing the network security group and application security groups for the interface.
  • 21aa5c6 UPSTREAM: : openshift: services: add service to handle application security groups
  • 08ab433 UPSTREAM: : openshift: Fix bad references to cluster/v1alpha1
  • 3e3c9dc UPSTREAM: : openshift: Drop unused certificates service to drop kubeadmin deps
  • 5703973 UPSTREAM: : openshift: Revendor for kube-1.16.0
  • 719f40a UPSTREAM: : openshift: Revendor to bring Bug 1752088: UPSTREAM: <carry>: openshift: Drop isDeleteAllowed func cluster-api#72
  • 3294595 UPSTREAM: : openshift: Remove dead code
  • 35cc61a UPSTREAM: : openshift: tag instances with cluster id
  • 3f27389 UPSTREAM: : openshift: Resync every 20 seconds
  • 432573f UPSTREAM: : openshift: Apply 'make goimports'
  • 3902dd7 UPSTREAM: : openshift: Revendor to bring cluster-api bd4f0ba460758c33c609cb333721606c6d002c4e
  • b066ec0 UPSTREAM: : openshift: Update machine only when changed
  • e47ba9b UPSTREAM: : openshift: Skip deletion of public IPs with too long names
  • ad61b3d UPSTREAM: : openshift: Run 'make goimports'
  • 286810e UPSTREAM: : openshift: Extend makefile with 'make goimports' target
  • 309b4c2 UPSTREAM: : openshift: do not randomize public ip name, error instead
  • a23858d UPSTREAM: : openshift: Check if annotations are initialized before assigning a key
  • e90ca70 UPSTREAM: : openshift: Drop ssh SSHPrivateKey
  • 408d4c8 UPSTREAM: : openshift: Add unit test for exists()
  • e93128f UPSTREAM: : openshift: update deps and bazel generated files
  • ee1b070 UPSTREAM: : openshift: generate public ip name with at most 63 characters
  • 85c2f99 UPSTREAM: : openshift: create git history verification script
  • 44ea1c0 UPSTREAM: : openshift: Add support for machine tags
  • ed6726c UPSTREAM: : openshift: use location from API over secret and unit test
  • 8000a3e UPSTREAM: : openshift: run 'dep ensure' to bring new deps
  • 7478948 UPSTREAM: : openshift: set additional machine annotations/labels with instance bits
  • b05edbd UPSTREAM: : openshift: Remove mutating logic from exist and set vm state in update
  • 4c00262 UPSTREAM: : openshift: Rename close to persist, make it error aware, stop persisting on exist
  • 42ca455 UPSTREAM: : openshift: Add omitempty tag so empty strings are omitted from the output
  • 17a9415 UPSTREAM: : openshift: Makefile: add e2e machinations
  • 7edc0cb UPSTREAM: : openshift: Remove dep on bazel binary when running test-e2e target
  • 6a50a8c UPSTREAM: : openshift: Move machineSpec from azureprovider.k8s.io/v1alpha1 into azureproviderconfig.openshift.io/v1beta1
  • dafd985 UPSTREAM: : openshift: Allow to run disk delete ops async
  • 36124f1 UPSTREAM: : openshift: print successfully deleted vm only when err is nil
  • b0bf44f Verify bazel
  • 303eaa2 UPSTREAM: : openshift: Stop asigning random az
  • 1199644 UPSTREAM: : openshift: services/virtualnetworks: create in NetworkResourceGroup
  • be195bc UPSTREAM: : openshift: services/subnets: create in NetworkResourceGroup
  • 4e1e8a1 UPSTREAM: : openshift: Allow for separate Network resource group
  • cd9c957 UPSTREAM: : openshift: Update vendored github.com/openshift/kubernetes-drain
  • 28517a0 UPSTREAM: : openshift: run 'dep ensure' to bring new deps
  • cbf0eff UPSTREAM: : openshift: Record events for actuator CUD operations
  • 81fbb76 UPSTREAM: : openshift: run 'make generate'
  • 4783391 UPSTREAM: : openshift: allowing choosing availability zone for a machine
  • 04051e4 UPSTREAM: : openshift: Expect vnet and managed identity to be always set
  • f1d7e01 UPSTREAM: : openshift: Switch builds to use Go 1.12
  • f2a9dca UPSTREAM: : openshift: describe how to deploy machine API with azure machine controller over kubernetes
  • 790f3ac UPSTREAM: : openshift: Deploy machine API plane with azure actuator
  • 7f07df6 UPSTREAM: : openshift: Provide a list of RBAC rules for the actuator
  • 12d3c93 UPSTREAM: : openshift: Allow vnet and managed identity names to be set explicitly
  • c0d3008 UPSTREAM: : openshift: Add namespace flag
  • 0f6d8d2 UPSTREAM: : openshift: Update code to accept k8s 1.14
  • 72dedbc UPSTREAM: : openshift: sync openshift/cluster-api@openshift-4.2-cluster-api-0.1.0 and k8s to 1.14.1
  • 702ef3a UPSTREAM: : openshift: Set instance subnet and load balancers explicitly
  • 43af035 UPSTREAM: : openshift: Allow to run Create/Delete ops async
  • 7ae13d7 UPSTREAM: : openshift: Set internal DNS name
  • ede8126 UPSTREAM: : openshift: Publish machine conditions during Create and Update operations
  • adf4782 UPSTREAM: : openshift: Set machine.Spec.ProviderID
  • 77d9b20 UPSTREAM: : openshift: Set machine addresses
  • 4ee3fce UPSTREAM: : openshift: Allow to set public IP address when creating vm
  • 4986149 UPSTREAM: 202: openshift: delete OS Disk and network interface before creating new VM instance, else fails due to previous existence
  • a270710 UPSTREAM: : openshift: Update README.md
  • 84af947 UPSTREAM: : openshift: remove kubeadm cluser configuration from clusterprovider
  • 7e23f23 UPSTREAM: : openshift: build fixes due to new cluster-api vendor and addition of managed identity
  • fa79861 UPSTREAM: : openshift: revendor cluster-api dependency
  • 4f7348d UPSTREAM: : openshift: convert machine label into constant for easier change in future
  • cdf874c UPSTREAM: : openshift: revendor after previous changes
  • 92a7c16 UPSTREAM: : openshift: Build both openshift cluster-api manager and azure-provider manager
  • 34a3a8c UPSTREAM: : openshift: revendor after previous changes
  • 9856a74 UPSTREAM: : openshift: Use openshift cluster-api branch instead of sigs.k8s.io
  • 9a42fd6 UPSTREAM: : openshift: revendor after previous changes
  • 01eca66 UPSTREAM: : openshift: decouple machine actuator from cluster object
  • de271dd UPSTREAM: : openshift: support for resourceid as an alternate image location
  • bb8dae8 UPSTREAM: : openshift: simplify if ... else logic
  • 260e7c7 UPSTREAM: : openshift: revendor after previous changes
  • 4755563 UPSTREAM: : openshift: Add support for Custom data, read from Secrets for Ignition
  • 5c0e9c5 fix: Support Availability Zones based on regions, automatically known using ARM API (Bug 1860229: Fail machine if create request doesn't succeed #155)
  • 3d72a4e fix: Generate consistent names for Azure resources (Do not hardcode GOARCH/GOOS #149)
  • 14a13e8 Upstream: : openshift: Add docker images for CI
  • 3326844 fix: Restructure machine actuator for unittests, and add unittests (Add termination handler for spot instances #145)
  • bdbc5bd Remove hardcoded values from virtual machines, respect values from machineconfig instead (BUG 1836141: Assert VM exists if VM state is Creating #147)
  • 9a6f513 Update dependencies (Add support for spot instances #143)
  • a818b2d fix: New Compute and Network micro services convering machine, cluster to pure ARM calls (Add liveness/readiness probes #139)
  • 803aefe fix: Create a new Generic Service interface which provides an individual azure service easy for unittesting (Expose spot instances to provider spec and reconcile label #138)
  • cde05d6 Create certificates, admin kubeconfig, bootstrap token and discoveryhashes using kubeadm (Enable metrics #136)
  • fc16fa8 UPSTREAM: : openshift: Update owners

k8s-ci-robot and others added 30 commits August 12, 2021 15:40
…om-binaries

Inject Windows custom binaries for use in PRs and running against Kubernetes CI
…clusters-custom-vnet-delete

fix managed clusters custom VNet delete issue
Tilt requirements section added to docs
…tions-main

Ensure Github Actions run on main branch after rename
…ters

Remove dangling references to old branch
…ckstart

docs: clarify deploying workload clusters with tilt
…tion-release

Automate release via Github Action
…mplate

docs: mention Troubleshooting in the issue template
…n-message

Fix immutable error message for AzureEnvironment
Print VNET range when subnet CIDR range is invalid
@alexander-demicev
Copy link
Author

/retest

@openshift-ci
Copy link

openshift-ci bot commented Jan 6, 2022

@alexander-demichev: 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/goimports c701429 link true /test goimports
ci/prow/govet c701429 link true /test govet
ci/prow/generate c701429 link true /test generate
ci/prow/unit c701429 link true /test unit
ci/prow/images c701429 link true /test images
ci/prow/e2e-azure-operator c701429 link true /test e2e-azure-operator
ci/prow/git-history c701429 link true /test git-history
ci/prow/e2e-azure c701429 link false /test e2e-azure

Full PR test history. Your PR dashboard.

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.

OWNERS Outdated
- JoelSpeed
- elmiko
- Fedosin
- lobziik

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably not worth adding Denis here as he will be leaving the team shortly

@@ -0,0 +1,18 @@
FROM registry.ci.openshift.org/ocp/builder:rhel-8-golang-1.17-openshift-4.10 AS builder

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this need to be 4.11 now?

@JoelSpeed
Copy link

/approve

@openshift-ci
Copy link

openshift-ci bot commented Jan 31, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JoelSpeed

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

The pull request process is described here

Needs approval from an approver in each of these files:

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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 31, 2022
Copy link

@Fedosin Fedosin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Feb 1, 2022
@alexander-demicev
Copy link
Author

/retest

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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.