Skip to content

Comments

fix(hubble): validate new FE/BE runtime and enable executable backend#701

Open
KavanaN12 wants to merge 27 commits intoapache:masterfrom
KavanaN12:feat/hubble-2_0-validation
Open

fix(hubble): validate new FE/BE runtime and enable executable backend#701
KavanaN12 wants to merge 27 commits intoapache:masterfrom
KavanaN12:feat/hubble-2_0-validation

Conversation

@KavanaN12
Copy link

Purpose of the PR

This PR validates and fixes the runtime integration between the new HugeGraph Hubble 2.0 frontend and backend modules.
It solves the issue where the backend could not run as an executable JAR and the frontend could not communicate with the backend API during development.
After this change, both modules run successfully and communicate end-to-end.


Main Changes

Backend (hubble-be)

  • Added spring-boot-maven-plugin to package the backend as a runnable Spring Boot JAR.
  • Verified working startup using:
java -Dhubble.home.path=. -jar hugegraph-hubble/hubble-be/target/hubble-be-1.7.0.jar
  • Confirmed backend reports correct status:
{"status":"UP"}

Frontend (hubble-fe)

  • Configured development proxy in package.json:
"proxy": "http://localhost:8088"
  • Verified frontend successfully reaches backend through /api/* requests.
  • Confirmed:
GET /api/v1.2/graph-connections → 200 OK

Verifying These Changes

  • Backend successfully built using Maven
  • Backend successfully started and reachable
  • Frontend installed using Yarn
  • Frontend started using Node 18 with OpenSSL legacy provider
  • Confirmed frontend and backend communicate correctly
  • Full end-to-end functionality tested

Screenshots included in this PR (backend health result + frontend DevTools network success).


Does This PR Potentially Affect the Following?

  • Nope
  • Dependencies (add/update license info)
  • Modify configurations
  • The public API
  • Other impacts (type here if needed)

Documentation Status

  • Doc - TODO
  • Doc - Done
  • Doc - No Need
    (The change only improves runtime behavior and does not modify user-facing features.)

Additional Notes

This update removes setup blockers for contributors working with the Hubble 2.0 codebase and ensures the development workflow operates correctly using the updated architecture.

@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Dec 7, 2025
@github-actions github-actions bot added the hubble hugegraph-hubble label Dec 7, 2025
GitHub Copilot added 16 commits December 8, 2025 22:24
- Update Java distribution from 'adopt' to 'temurin' (officially recommended)
- Extend Java version matrix to test both Java 11 and Java 17 for better compatibility
- Fix test execution paths and add consistent error handling with explicit exit codes
- Correct coverage file path from 'target/jacoco.xml' to 'hugegraph-loader/target/jacoco.xml'
- Make coverage upload conditional on Java 11 only to avoid duplicate reports
- Improve install-hadoop.sh with comprehensive error handling and startup verification
- Enhance install-mysql.sh with Docker/native MySQL fallback and connection validation
- Strengthen install-hugegraph-from-source.sh with timeout handling and detailed logging
- Add parameter validation and improved error messages to all setup scripts
- Document all fixes in LOADER_CI_FIXES.md
…mentation

- Add LOADER_CI_COMPREHENSIVE_FIX_GUIDE.md with detailed problem analysis
- Add LOADER_CI_TROUBLESHOOTING.md with quick reference and common solutions
- Document root causes and fixes for each workflow issue
- Provide debugging tips and log analysis guidance
- Include environment variable reference and update schedule
… coverage paths

- Fix settings.xml backup error handling across all workflows (mkdir -p, conditional copy)
- Remove redundant cache steps (leverage setup-java built-in caching)
- Correct all coverage file paths to include module directories
- Add Java setup to license-checker workflow
- Fix grep error handling in RAT check (use -q flag instead of &&)
- Standardize codecov-action versions and parameters
- Add -ntp flag to all Maven commands for consistency

Fixes 9+ workflow issues affecting 551+ historical failed runs
…ad of npm

The hubble-fe project uses yarn.lock, not package-lock.json. The workflow was
trying to cache with a non-existent package-lock.json file, causing setup-node
to fail with cache resolution errors.

Changed:
- cache: 'npm'  cache: 'yarn'
- cache-dependency-path: 'hugegraph-hubble/hubble-fe/package-lock.json'  'hugegraph-hubble/hubble-fe/yarn.lock'

This fixes the failures in runs:
- #20010083965 (hubble-ci)
- #20010083969 (license-checker)
- #20010083957 (CodeQL)
- #20010083954 (loader-ci)
@github-actions github-actions bot added the loader hugegraph-loader label Dec 8, 2025
@codecov
Copy link

codecov bot commented Dec 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (b066b80) to head (cadbb23).
⚠️ Report is 67 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (b066b80) and HEAD (cadbb23). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (b066b80) HEAD (cadbb23)
2 1
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #701       +/-   ##
============================================
- Coverage     62.49%   0.00%   -62.50%     
============================================
  Files           262      34      -228     
  Lines          9541    3087     -6454     
  Branches        886     436      -450     
============================================
- Hits           5963       0     -5963     
+ Misses         3190    3087      -103     
+ Partials        388       0      -388     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

GitHub Copilot added 3 commits December 11, 2025 10:55
…mpatibility fixes

- fix(pom): exclude conflicting Lombok version from hugegraph-common transitive dependency to ensure CodeQL Analysis uses 1.18.30
- fix(loader-ci): add MAVEN_OPTS with --add-opens flags for Java 17 Lombok compatibility
- fix(hubble-ci): add MAVEN_OPTS with --add-opens and javac.processing flags for Java 11/21 Lombok compatibility
@KavanaN12 KavanaN12 closed this Dec 11, 2025
@KavanaN12 KavanaN12 reopened this Dec 11, 2025
@github-actions
Copy link

Due to the lack of activity, the current pr is marked as stale and will be closed after 180 days, any update will remove the stale label

@github-actions
Copy link

Due to the lack of activity, the current pr is marked as stale and will be closed after 180 days, any update will remove the stale label

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

Labels

hubble hugegraph-hubble hubble-be hubble-fe loader hugegraph-loader size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task] New Grpah-UI(hubble) backend module issues

1 participant