Skip to content

Conversation

@wetted
Copy link
Contributor

@wetted wetted commented Sep 9, 2022

No description provided.

@wetted wetted requested a review from sdelamo September 9, 2022 20:57
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 9, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@wetted wetted marked this pull request as draft September 9, 2022 21:25
@wetted
Copy link
Contributor Author

wetted commented Sep 9, 2022

This has some issues with three tests that worked on master but break now. But it doesn't break with this branch alone. If I change master from Micronaut 3.3.4 to 3.5.0, the same three tests break as well. I know at least one seems due to changes in the BeanContext refactoring that was done for Micronaut 3.5.0. Any ideas about this @dstepanov? Can you please have a look at RateLimitingQueueFactorySpec? I think that's a problem with changes to BeanContext or rather DefaultBeanContext in core. I don't know about the others yet (ConfigMapResourceReconcilerSpec and ConfigMapResourceReconcilerWithFiltersSpec).

@wetted wetted marked this pull request as ready for review October 13, 2022 22:23
@wetted wetted assigned timyates and unassigned timyates Oct 13, 2022
@wetted wetted requested review from sdelamo and timyates October 13, 2022 22:24
@timyates
Copy link
Contributor

The Unsupported class file major version 61 error seems to be caused by cglib when Spock tries to create a Stub (in some situations)

I could work around it by adding

configurations {
    testRuntimeClasspath {
        resolutionStrategy.dependencySubstitution {
            substitute module('cglib:cglib-nodep') using module('net.bytebuddy:byte-buddy:1.12.18')
        }
    }
}

To the builds in question (to replace cglib with byte-buddy at test runtime)

We add cglib in our internal tools

https://github.com/micronaut-projects/micronaut-build/blob/d3b3d0e7c4187af0c2c3d09986decc02322feccd/src/main/groovy/io/micronaut/build/MicronautBuildCommonPlugin.groovy#L85-L87

Should we be thinking about migrating to byte-buddy for 4.0.0? I don't know about the state of either library enough to make a call...

@melix @sdelamo @graemerocher

@graemerocher
Copy link
Contributor

Yes move to bytebuddy

Cglib is not maintained anymore

timyates referenced this pull request in micronaut-projects/micronaut-build Oct 14, 2022
We had issues with Groovy 4, cgilib, Spock and Java 17

https://github.com/micronaut-projects/micronaut-kubernetes/pull/510\#issuecomment-1278775288

This pr switches cgilib to byte buddy, as cgilib is no longer maintained
@timyates
Copy link
Contributor

timyates commented Oct 14, 2022

@wetted io.micronaut.build.shared.settings 6.0.1 is out which uses bytebuddy instead of cglib 🤞

@wetted
Copy link
Contributor Author

wetted commented Oct 14, 2022

@wetted io.micronaut.build.shared.settings 6.0.1 is out which uses bytebuddy instead of cglib 🤞

Thanks @timyates . I'll make the change and push it.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@timyates
Copy link
Contributor

Hmmm... fails on CI and fails differently locally

Did you get this passing locally @wetted ?

Maybe @alvarosanchez has seen the errors before?

Copy link
Member

@alvarosanchez alvarosanchez left a comment

Choose a reason for hiding this comment

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

The build is failing because the workflow is still calling a dependencyUpdates task that no longer exists.

In general, all workflows need to be updated "manually" comparing them with the project template. Also there are other workflows that should be removed (eg dependency-update.yml)

@wetted
Copy link
Contributor Author

wetted commented Dec 20, 2022

I'll fix the ci thing with workflows.

It still has two test issues. One has to do with duplicate beans. The other is a failure with: io.micronaut.context.exceptions.BeanInstantiationException: Bean definition [io.micronaut.kubernetes.client.operator.ConfigMapResourceReconciler] could not be loaded: Error instantiating bean of type [io.micronaut.kubernetes.client.operator.ControllerConfiguration]: Failed to resolve the 'apiGroup' for class io.kubernetes.client.openapi.models.V1ConfigMap from discovery cache.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@n0tl3ss
Copy link
Member

n0tl3ss commented Jan 27, 2023

Blocked by micronaut-projects/micronaut-core#8593

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@wetted wetted self-assigned this Mar 28, 2023
@wetted
Copy link
Contributor Author

wetted commented Mar 28, 2023

Interesting about all the failures. When I run locally only ConfigMapResourceReconcilerSpec and ConfigMapResourceReconcilerWithFiltersSpec are failing.

@wetted wetted removed their assignment Mar 30, 2023
@wetted wetted changed the base branch from 4.0.x to master April 4, 2023 13:42
@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 4, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sdelamo
Copy link
Contributor

sdelamo commented Apr 10, 2023

I assume this is superseded by #548

@sdelamo sdelamo closed this Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

7 participants