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

DAT-16200



 DevOps :: Support Docker like Liquibase Command Line #268

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

jandroav
Copy link
Contributor

No description provided.

chore: update Dockerfile.alpine WORKDIR to /liquibase/changelog for consistency
feat: add changelog.sql for test environment

feat: add mssql-jdbc-8.2.2.jre11.jar driver for test environment

feat: add example-changelog.xml for test environment

feat: add liquibase-redshift-4.25.0.jar extension for test environment

feat: add liquibase-mssql.properties for test environment

feat: add liquibase.properties for test environment

feat: add liquibase_command.sh script for test environment

feat: add liquibase_update.sh script for test environment

The above changes were made to add necessary files and configurations for the test environment. These files include a Dockerfile for building a test environment, a changelog.sql file for defining database schema changes, a mssql-jdbc-8.2.2.jre11.jar driver for connecting to the MSSQL database, an example-changelog.xml file for demonstrating how to define database changes using Liquibase, a liquibase-redshift-4.25.0.jar extension for supporting Redshift databases, a liquibase-mssql.properties file for configuring Liquibase to connect to the MSSQL database, a liquibase.properties file for configuring Liquibase with target and source database information, a liquibase_command.sh script for executing Liquibase tasks, and a liquibase_update.sh script for applying changes to the database. These additions are necessary to set up and manage the test environment effectively.
The .DS_Store file was deleted as it is an unnecessary file that is automatically generated by macOS. It is not needed for the project and can be safely removed.
@jnewton03
Copy link
Contributor

@jandroav are the binaries for driver and extension expected here?

The following files were removed:
- `.github/test/Dockerfile`: Removed Dockerfile used for Liquibase testing.
- `.github/test/drivers/mssql-jdbc-8.2.2.jre11.jar`: Removed JDBC driver for Microsoft SQL Server used in Liquibase testing.
- `.github/test/example-changelog.xml`: Removed example changelog file used in Liquibase testing.
- `.github/test/extensions/liquibase-redshift-4.25.0.jar`: Removed Liquibase extension for Amazon Redshift used in Liquibase testing.
- `.github/test/liquibase.properties`: Removed Liquibase properties file used in Liquibase testing.
- `.github/test/scripts/liquibase_command.sh`: Removed shell script for Liquibase command used in Liquibase testing.
- `.github/test/scripts/liquibase_update.sh`: Removed shell script for Liquibase update command used in Liquibase testing.

These files were no longer needed and were deleted to clean up the repository.
@jandroav
Copy link
Contributor Author

@jandroav are the binaries for driver and extension expected here?

no @jnewton03 , I just deleted all non-needed resources to test it.

jandroav and others added 5 commits January 8, 2024 09:10
…root to improve security and prevent potential privilege escalation
…liquibase command to remove the leading slash for consistency
… file structure

The file paths in the example commands were incorrect and did not match the actual file structure. This commit fixes the file paths in the example commands to accurately reflect the file structure.
@jandroav
Copy link
Contributor Author

jandroav commented Jan 8, 2024

@jnewton03 I modified the README.md file and updated the tests

@jnewton03 jnewton03 marked this pull request as draft January 11, 2024 23:33
@jnewton03
Copy link
Contributor

converting to draft to potentially be used at a later date

@jnewton03 jnewton03 closed this Jan 11, 2024
@jnewton03 jnewton03 reopened this Jan 11, 2024
@jandroav jandroav marked this pull request as ready for review January 12, 2024 12:54
@jandroav jandroav marked this pull request as draft January 12, 2024 13:14
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.

None yet

2 participants