Skip to content

Fix solr:ui lockfile using stale kotlinx-datetime compat variant#4335

Merged
malliaridis merged 1 commit intoapache:mainfrom
janhoy:fix/kotlinx-datetime-lockfile-compat
Apr 26, 2026
Merged

Fix solr:ui lockfile using stale kotlinx-datetime compat variant#4335
malliaridis merged 1 commit intoapache:mainfrom
janhoy:fix/kotlinx-datetime-lockfile-compat

Conversation

@janhoy
Copy link
Copy Markdown
Contributor

@janhoy janhoy commented Apr 25, 2026

Minor fix after #4305

The solr/ui lockfile contained a stale entry for org.jetbrains.kotlinx:kotlinx-datetime:0.7.1-0.6.x-compat in the *ApiDependenciesMetadata / *CompileOnlyDependenciesMetadata configurations. This compat variant was written by a previous lockfile generation and was never refreshed because resolveAndLockAll does not resolve those internal KMP metadata configurations.

The fix was to delete the lockfile and regenerate it cleanly via ./gradlew :solr:ui:resolveAndLockAll --write-locks, which produced only plain 0.7.1 entries. No build logic changes were required.

The changelog entry title is also corrected from v0.7.1-0.6.x-compat to v0.7.1.

Regenerate the ui module lockfile via resolveAndLockAll --write-locks,
removing the stale 0.7.1-0.6.x-compat entry that had persisted in
*ApiDependenciesMetadata configurations from a prior lockfile generation.
Also correct the changelog entry title to reflect the plain 0.7.1 release.
@github-actions github-actions Bot added dependencies Dependency upgrades admin-ui labels Apr 25, 2026
@janhoy janhoy requested a review from malliaridis April 25, 2026 19:41
@janhoy
Copy link
Copy Markdown
Contributor Author

janhoy commented Apr 26, 2026

@malliaridis Surprised the lockfile of 'ui' can get stale like this and not be overwritten/re-generated when regenerating locks? Had to delete it and re-generate in order to fix the bogous version.

@malliaridis
Copy link
Copy Markdown
Contributor

Yeah, this is strange. I noticed the version drift recently, but I am not sure if that only happens on the ui module. It would make sense because it is the only multiplatform module that may get mixed up by the operating system that runs the locking (due to the multiplatform nature).

Need to investigate the cause if I find the time.

@malliaridis malliaridis merged commit e2a1819 into apache:main Apr 26, 2026
4 of 5 checks passed
@malliaridis
Copy link
Copy Markdown
Contributor

Moved forawrd and merged this one, so that I can update another PR and finalize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

admin-ui dependencies Dependency upgrades

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants