Azure-Samples / modern-data-warehouse-dataops

DataOps for Microsoft Data Platform technologies. https://aka.ms/dataops-repo

MIT License

591 stars 463 forks source link

fix: REST API broken links, Int testing devcontainer upgrade #853

Closed ydaponte closed 6 days ago

ydaponte commented 6 days ago

Type of PR

Code changes
Test changes

Purpose

Fixing temporarily the REST API parking sensors broken links
Upgrading devcontainer for integration testing - debian version 12 and msodbcsql18
removing delete databricks cli line from main docker files as it fails on the rebuild container

Does this introduce a breaking change? If yes, details on what can break

Author pre-publish checklist

[ ] Added test to prove my fix is effective or new feature works
[X] No PII in logs
[ ] Made corresponding changes to the documentation

Validation steps

Launch devcontainer
deploy sample (deploy.sh)
configure git with adf
run ADF pipelines successfully

Issues Closed or Referenced

References #852

ydaponte commented 6 days ago

Thanks @LiliamLeme! As discussed in the phone - the devcontainer.json is the one running the post-create command. Also it is supposed to run when rebuilding as per the docker documentation here: https://code.visualstudio.com/docs/devcontainers/create-dev-container#_rebuild. Please if you find again the problem let me know and we open a specific issue for this one. Thanks again!