[ADMINAPI-1349] on-prerelease.yml improvements#260
Merged
jagudelo-gap merged 721 commits intomainfrom Feb 5, 2026
Merged
Conversation
…cation/EdFi.Ods.AdminApi.DBTests/Microsoft.Data.SqlClient-5.1.3 Bump Microsoft.Data.SqlClient from 5.1.1 to 5.1.3 in /Application/EdFi.Ods.AdminApi.DBTests
[ADMINAPI-974] Package updates for microsoft.data.sqlclient vulnerability
[ADMINAPI-973] - Fix publish packages script
… dbDeploy parameter
[ADMINAPI-972] - update minor version
* Copy Actions updates from Admin API 1 * Update file extension
* Add read me file and fix docker file warnings * Update single tenant/ multi tenant folder details * Add space * Re-arranging the layers on docker files for enabling caching * Update the SDK base layer --------- Co-authored-by: Stephen A. Fuqua <stephen@safnet.com>
Co-authored-by: Johnny Brenes <johnny.brenes@simpat.tech>
… Update of pending actions (#100)
[ADMINAPI-755] Improve registration error handling
[ADMINAPI-994] Remove layer from dev.Dockerfile
[ADMINAPI-995] Reviewed Skipped Checks
* Reformat file * Remove tenant modification code * Remove write access tests for the /v2/tenants endpoint * Update openssl version build docker image * Updates nginx for v1 --------- Co-authored-by: Jesus Pardo <jpardo@growthaccelerationpartners.com> Co-authored-by: David Jimenez <djimenez@growthaccelerationpartners.com>
* Refactor Dockerfile to use ARG for POSTGRES_USER and remove POSTGRES_PASSWORD environment variable * Refactor Dockerfile to remove POSTGRES_PASSWORD environment variable and ensure USER is set to POSTGRES_USER * Refactor Dockerfile to use ARG for ADMIN_API_VIRTUAL_NAME and ODS_VIRTUAL_NAME v1-gateway * Refactor Dockerfile to use ARG for POSTGRES_USER and remove POSTGRES_PASSWORD environment variable v1-database * Refactor Dockerfile to set ASPNETCORE_ENVIRONMENT ARG directly and ensure ENV variable is correctly assigned * Remove unnecessary blank line before EXPOSE instruction in Dockerfile
Fix typo
…on tests for Admin… (#368) * Copy over the integration tests from V1 * Running Tests * Integration tests * Some code cleanup * Fixes based on copilot comments * Update for 7.3 instead of 7.1
…369) * Pin and update apk package versions in Dockerfiles * Update package versions in Dockerfiles * Update Alpine package versions in Dockerfiles * Pin dos2unix and unzip versions in Dockerfile * Pin apk package versions and clean cache in Dockerfiles * Update Docker/Settings/V2/DB-Admin/pgsql/Dockerfile Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update Docker/V2/db.pgsql.admin.Dockerfile Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update Alpine package versions in Dockerfiles * Update apk install commands for dynamic package versions * Update dev.pgsql.Dockerfile * Update dev.pgsql.Dockerfile * Temporal Change * Update Dockerfile package versions and remove APK info step * Update dev.pgsql.Dockerfile * Update db.pgsql.admin.Dockerfile * Update Dockerfile * Update Dockerfiles to use Alpine 3.20 and specific package versions * Update package versions in PostgreSQL admin Dockerfiles * Update db.pgsql.admin.Dockerfile * Pin package versions in pgsql admin Dockerfiles --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Enhance health check functionality with multi-tenancy support and custom response formatting * Add unit tests for health check registration with multi-tenancy support * Fix the Copilot comments * Add the tag Databases to the health checker and return a similar response as in DMS * Add the response to the health service
…0.21 (#374) * Docker Security Vulnerability Remediation - .net 8.0.21 * Fix comment. Remove commented out code
…ice description (#377) * POST to profile with line breaks returns 400 with nice description * Remove e2e test added * Adds http file for Profiles
…ow (#378) * Improve installer version token replacement in workflow * Update .github/workflows/on-prerelease.yml Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ssors (#375) * Enhance Docker build process by adding QEMU setup and specifying target platforms * Update base image and dependencies in Dockerfile for improved compatibility * Fix base image version in Dockerfile to ensure compatibility * Fix string quotes in coverage arguments for consistency in RunTests function * Revert $coverageArgs * Refactor RunTests function to streamline coverage argument handling * Replace wget with curl for health check in ods_inspect.sh for improved reliability * Update OdsApi version --------- Co-authored-by: Juan Agudelo <jagudelo@growthaccelerationpartners.com>
…s (#379) * Docker updates for Admin API 2.3 - Part 1 * Change tag for 6x and fixes if condition * Reviewing compose files. * Tests fix * Removes Ods Api docker containers * Restoring back some dockerfile
…for 2.2.3. (#382) * Update Ed-Fi API Specifications repo with new spec for 2.2.3. * Adds specification for v1
…s - Fixes connection strings (#383) Fixes connection strings.
Update Docker configurations - Adjust permissions and set default ADMIN_API_VERSION
…for 2.3.0 (#384) * Update Ed-Fi API Specifications repo with new spec for 2.3.0 * Removes 2.2.3 file --------- Co-authored-by: Juan Agudelo <105452282+jagudelo-gap@users.noreply.github.com>
* Add support for multi-platform builds in Docker settings * Update Dockerfiles to remove multi-platform support and specify exact image version * Update unzip version in Dockerfile to 6.0-r15 * Update Dockerfile to use unzip version 6.0-r15 and correct base image tag to 7.3.1
…build-binaries.yml (#387) Update db-admin image to version v1.4.3 in compose-build-binaries.yml
Contributor
Test Results388 tests 387 ✅ 19s ⏱️ Results for commit 6ac18c3. ♻️ This comment has been updated with latest results. |
jagudelo-gap
changed the title
jpardogrowthaccelerationpartners
approved these changes
Feb 5, 2026
This file contains hidden or 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
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 updates the
.github/workflows/on-prerelease.ymlworkflow to improve compatibility and reliability by adjusting the shell and installation steps for the credential provider and package push process.Also, this PR reorganizes the
mainbranch where it was based on the wrong tag. This branch was created from tag 2.3.0 and includes changes we have so far in the main branch.Workflow improvements:
install-artifacts-credprovider.ps1) to a Bash script (install-artifacts-credprovider.sh) and set the shell tobashfor this step to increase compatibility across different environments.pwsh(PowerShell Core) for the package push step to ensure the correct shell is used for PowerShell commands.