Skip to content

Mp/app 340/image update#684

Merged
mpeels merged 7 commits intomainfrom
mp/app-340/image-update
Mar 5, 2026
Merged

Mp/app 340/image update#684
mpeels merged 7 commits intomainfrom
mp/app-340/image-update

Conversation

@mpeels
Copy link
Contributor

@mpeels mpeels commented Mar 4, 2026

Notes

NBS 6 WildFly image is now available!

As of https://github.com/cdcent/NEDSSDev/pull/251, a WildFly image is published for developer use. This removes the need for multiple repos to have their own build instructions/scripts.

A couple of notes about this image:

  1. The WildFly image currently has a hardcoded database connection string of jdbc:sqlserver://nbs-mssql:1433, requiring us to update the database container name from rtr-mssql to nbs-mssql.
  2. There is currently no arm64 image

Updated MSSQL Image

A new version of the MSSQL image is now available. This new image has built in initialization functionality, removing the need to have our own Dockerfile and initialization scripts. More info here.

These initialization scripts are executed each time the container starts up, so some modifications needed to be made to allow for repeated execution.

For example, the backup of RDB and restore to rdb_modern should only occur if the rdb_modern is not already preset.

IF DB_ID('rdb_modern') IS NULL
BEGIN
    -- Create backup of RDB
    BACKUP DATABASE RDB TO DISK = '/tmp/RDB.bak' WITH COMPRESSION;

    -- Restore backup to rdb_modern
    RESTORE DATABASE [RDB_MODERN] FROM  DISK = N'/tmp/RDB.bak' WITH  FILE = 1,
    MOVE N'RDB' TO N'/var/opt/mssql/data/rdb_modern_data.mdf',
    MOVE N'RDB_log' TO N'/var/opt/mssql/data/rdb_modern_log.ldf',  NOUNLOAD,  STATS = 10


    EXEC dbo.sp_dbcmptlevel @dbname=N'RDB_MODERN', @new_cmptlevel=120
END

JIRA

@mpeels mpeels requested a review from a team as a code owner March 4, 2026 17:17
Copy link
Contributor

@mehansen mehansen left a comment

Choose a reason for hiding this comment

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

thanks for answering my questions! nothing blocking left - the liquibase depends_on functionality seems to be working

edit: saw your answer - that makes sense 👍

@mpeels mpeels merged commit b45819b into main Mar 5, 2026
1 check passed
@mpeels mpeels deleted the mp/app-340/image-update branch March 5, 2026 21:10
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.

3 participants