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

[DCJ-400-gradle] Upgrade io.swagger.codegen.v3 deps 3.0.61 -> 3.0.62 #1801

Merged
merged 2 commits into from
Sep 4, 2024

Conversation

okotsopoulos
Copy link
Contributor

@okotsopoulos okotsopoulos commented Sep 4, 2024

Jira ticket: https://broadworkbench.atlassian.net/browse/DCJ-400

Addresses

This patch upgrade without changing configuration caused compilation of Swagger generated code to fail with the following errors:

/github/workspace/build/generated-src/swagger/src/main/java/bio/terra/model/ResourcePolicyModel.java:10: error: package io.swagger.configuration does not exist
import io.swagger.configuration.NotUndefined;
                               ^
/github/workspace/build/generated-src/swagger/src/main/java/bio/terra/model/ResourcePolicyModel.java:21: error: cannot find symbol
@NotUndefined
 ^

This was introduced in swagger-api/swagger-codegen-generators#1308

Summary of changes

  • Upgrade io.swagger.codegen.v3:swagger-codegen and io.swagger.codegen.v3:swagger-codegen-cli from 3.0.61 -> 3.0.62
  • For the time being, set validationMode=legacy along with these patch upgrades to mitigate another likely bug in swagger-codegen-generators.
  • Because io.swagger.codegen.v3 patch upgrades have lately included breaking changes, exclude them from our grouped minor and patch upgrade Dependabot PRs. Any available upgrades will be PRed separately.

Testing Strategy

CI build and tests should pass.

This patch upgrade without changing configuration caused compilation of Swagger generated code to fail with the following errors:

```
/github/workspace/build/generated-src/swagger/src/main/java/bio/terra/model/ResourcePolicyModel.java:10: error: package io.swagger.configuration does not exist
import io.swagger.configuration.NotUndefined;
                               ^
/github/workspace/build/generated-src/swagger/src/main/java/bio/terra/model/ResourcePolicyModel.java:21: error: cannot find symbol
@notundefined
 ^
```

For now, setting `validationMode=legacy` but think this might be another bug in swagger-codegen-generators.
Patch upgrades have lately included breaking changes.
But, we do want to keep swagger-codegen and swagger-codegen-cli versions in sync with each other, so introduce a new dependency grouping for them.
Copy link

sonarcloud bot commented Sep 4, 2024

@okotsopoulos okotsopoulos marked this pull request as ready for review September 4, 2024 16:33
@okotsopoulos okotsopoulos requested review from a team as code owners September 4, 2024 16:33
@okotsopoulos okotsopoulos requested review from rushtong and fboulnois and removed request for a team September 4, 2024 16:33
Copy link
Contributor

@fboulnois fboulnois left a comment

Choose a reason for hiding this comment

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

thanks for catching this 👍

Copy link
Contributor

@rushtong rushtong left a comment

Choose a reason for hiding this comment

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

👍🏽

@okotsopoulos okotsopoulos merged commit e956515 into develop Sep 4, 2024
14 checks passed
@okotsopoulos okotsopoulos deleted the okotsopo-DCJ-400-io.swagger.codegen.v3-3.0.62 branch September 4, 2024 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants