Skip to content

Grails 7 Migration - Rundeck 6.0 Compatibility#115

Merged
fdevans merged 22 commits intomasterfrom
grails7-upgrade
Apr 7, 2026
Merged

Grails 7 Migration - Rundeck 6.0 Compatibility#115
fdevans merged 22 commits intomasterfrom
grails7-upgrade

Conversation

@fdevans
Copy link
Copy Markdown
Contributor

@fdevans fdevans commented Apr 2, 2026

Overview

Updates this plugin for Grails 7 / Rundeck 6.0 compatibility.

Key Changes

  • Upgraded to Java 17
  • Updated to Grails 7 / Groovy 4 / Spring Boot 3
  • Modernized GitHub Actions workflows (Java 17, updated actions, YAML fixes)
  • Standardized Gradle to 8.14.3
  • Updated version to 2.1.8-grails7 format

Documentation

Migration details and handoff documentation: .github/Grails7-Handoff/

Testing

  • All CI workflows passing on grails7-upgrade branch
  • Plugin builds and tests successfully with Java 17
  • Compatible with Rundeck 6.0 (Grails 7)

Notes

  • Version uses temporary -grails7 suffix (will be removed at release)
  • Maintains backward compatibility with Rundeck 5.x

rmeloamaro and others added 19 commits April 16, 2025 17:06
- As per the requirements listed in v0.5.0 Python WinRM plugin: https://github.com/diyan/pywinrm
- Update groupId to com.github.rundeck-plugins
- Update version to X.Y.Z-grails7-upgrade-test
- Update rundeck-core dependency to 6.0.0-SNAPSHOT
- Enhanced diagnostics and error logging
- Fixed test compatibility issues (JUnit 4 vs 5, Java 17)
- Updated for Spring Boot 3 / Java 17
….plugins

- Changed groupId: com.github.rundeck-plugins → com.rundeck.plugins
- Updated Axion to 1.17.2 with prefix='' (no 'v' prefix)
- Removed hardcoded version (2.1.5-grails7-upgrade-test)
- Replaced external build-zip with inline pluginZip task (type: Jar, not Zip!)
- Added PackageCloud publishing configuration
- Version now managed dynamically from git tags

CRITICAL: ZIP plugins must use task type Jar, not Zip, for proper manifest handling.

See: .temp/PLUGIN_TAGGING_ARCHITECTURE.md and PACKAGECLOUD_MIGRATION_PLAN.md
- Added ReplaceTokens filter to process @Version@, @author@, @url@ tokens
- Added tokens map with version, date, author, url, title, description, name
- Import org.apache.tools.ant.filters.ReplaceTokens

This fixes the YAML parsing error where unprocessed tokens like @Version@
were being included in the published plugin.yaml file.
- Update Java 1.8 → 17 for Grails 7 compatibility
- Replace deprecated create-release and upload-release-asset actions
  with modern gh CLI (gh release create handles both steps)
- Update deprecated set-output syntax to use GITHUB_OUTPUT
- Update setup-java to v4 with explicit distribution
- Update Java from 1.8 to 17 for Grails 7 compatibility
- Update setup-java to v4 with explicit distribution
- Fix deprecated set-output syntax to use GITHUB_OUTPUT

Axion plugin 1.17.2 requires Java 11 minimum, was failing with Java 8.
Remove python2 from interpreter options in plugin.yaml and bump
Docker Python version to 3.11, aligned with pywinrm 0.5.0 minimum
requirement of Python 3.8+.
- Update actions/checkout@v2 to v4
- Update actions/setup-java@v1 to v4 with zulu distribution
- Fix ::set-output syntax to use GITHUB_OUTPUT

These changes ensure workflows remain functional after Node.js 20 deprecation in September 2026.
Upgrade Gradle wrapper to match rundeck/rundeckpro (8.14.3) for consistency.
This ensures all plugins use the same build tooling version.
Corrected distribution parameter indentation to align with java-version.
This fixes GitHub Actions validation errors.
- Fixed distribution parameter indentation (must align with java-version)
- Updated step name labels to 'Set up JDK 17'
- Updated Gradle wrapper scripts to 8.14.3
Corrected distribution parameter indentation to align with java-version (8 spaces).
Copilot AI review requested due to automatic review settings April 2, 2026 22:29

This comment was marked as outdated.

Copy link
Copy Markdown

@ronaveva ronaveva left a comment

Choose a reason for hiding this comment

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

LGTM

@fdevans fdevans merged commit 8a591ca into master Apr 7, 2026
2 checks passed
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.

4 participants