DAT-19653 DevOps :: Fix firebird and hsqldb infra #996
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…titan references to streamline configuration and improve clarity 🔧 (docker-compose): update docker-compose configuration for hsqldb and firebird services to ensure proper setup and environment variables 📝 (harness-config): update database prefixes in harness-config.yml from 'titan' to 'docker' for consistency with new setup 🗑️ (titan-installer): remove titan-installer.sh as it is no longer needed in the current setup
…ntries to streamline the workflow configuration and reduce maintenance overhead.
…ervices to initialize databases with SQL scripts
…rt configuration for consistency and to avoid conflicts with other services
…iguration to streamline the setup ✨ (docker-compose.yml): add FIREBIRD_DATABASE_DEFAULT_CHARSET environment variable for Firebird services to ensure UTF-8 compatibility ✨ (docker-compose.yml): swap port mappings for Firebird services to avoid conflicts and ensure proper access 📝 (docker/data): add new database files for lbcat and test to support testing scenarios
…lasspath entries to enhance readability and maintainability
automatic generation of source files and improve development workflow 🔧 (Eclipse settings): enable annotation processing and release settings to ensure proper handling of annotations and maintain compatibility with Java 11 features
… correct data directory for proper database access during tests
…consistency 🔧 (org.eclipse.jdt.apt.core.prefs): add missing newline at the end of org.eclipse.jdt.apt.core.prefs file for consistency 🔧 (org.eclipse.jdt.core.prefs): add missing newline at the end of org.eclipse.jdt.core.prefs file for consistency
…to use hsqldb instead of firebird for better compatibility and support
…ation 🔧 (docker-compose): update volume mapping to use new database directory for HSQLDB
…buntu-22.04 for improved compatibility and support 📝 (hsqldb): update hsqldb.properties to reflect the modified state and current timestamp 📝 (hsqldb): add initial log entries to hsqldb.log for better tracking of database operations
…ccess and prevent conflicts
…ccess and prevent conflicts
…e ready before proceeding
…itan instructions ♻️ Refactor create-infra.sh to remove unnecessary sleep command for improved efficiency 🐛 Remove obsolete HSQLDB log and properties files to clean up the project 🔧 Update test-config.yml to change prefix from 'titan' to 'docker' for consistency in database configuration
…cker containers to ensure services are ready and for better debugging
…ntainer name format for clarity and accuracy
…fine database settings and version information
…ntu-20.04 for compatibility with dependencies ✨ (main.yml): add support for Firebird database versions 3 and 4 in the workflow 📝 (hsqldb.properties): update properties file to reflect the database modification status and timestamp 📝 (hsqldb.log): add new log file for HSQLDB to capture database operations
…Actions workflow to enhance testing coverage and compatibility with various database systems
Tamelianovych
temporarily deployed
to
internal
GitHub Actions
Inactive
Tamelianovych
temporarily deployed
to
internal
GitHub Actions
Inactive
…e, init.sql, start.sh, create-infra.sh, docker-compose.yml, harness-config.yml): add support for HSQLDB versions 2.5, 2.6, and 2.7 to enhance database compatibility and testing capabilities ♻️ (automation-runner.sh, create-infra.sh): refactor HSQLDB version handling to accommodate new versions and remove deprecated references 📝 (README.md): update documentation to reflect new HSQLDB versions available for use
… in SQL queries for better data integrity
…tibility to streamline database initialization process
There was a problem hiding this comment.
Do we really need this file?
There was a problem hiding this comment.
Do we really need this file?
Tamelianovych
approved these changes
Feb 21, 2025
There was a problem hiding this comment.
Probably we can remove two files, all other looks good, Thanks @jandroav
…ean up the project and reduce clutter
Tamelianovych
approved these changes
Feb 21, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request primarily focuses on removing Titan and replacing it with Docker for managing databases in the testing environment. Additionally, it updates the CI environment and documentation to reflect these changes.
Key changes include:
Environment Updates:
ubuntu-22.04instead ofubuntu-20.04in the.github/workflows/advanced.ymlfile.Documentation Updates:
README.non-docker-test-environments.md.README.mdby removing redundant information about Titan.Configuration and Script Updates:
titan-installer.sh, from the configuration files and scripts. [1] [2] [3] [4]docker-compose.ymlto use Docker for managing HSQLDB and Firebird databases, including the addition of volume mappings for database initialization.harness-config.ymlandtest-config.ymlto replaceprefix: titanwithprefix: dockerand adjusted database URLs accordingly. [1] [2]Database Initialization:
hsqldb.propertiesandhsqldb.script) to ensure databases are properly set up when using Docker. [1] [2]