Publish docker images to ACR and use those in build process. #208
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
chrispsommers
added a commit
that referenced
this pull request
Oct 8, 2022
* Create dash-docker-p4c-bmv2-acr.yml * Update Makefile * Experimenting with ACR publish workflows Revert p4c-bmv2 tag * Update and rename dash-docker-p4c-bmv2-acr.yml to dash-p4c-bmv2-docker-acr.yml * Experimental ACR publish; change regname Use sonicdash ACR for p4c-bmv2. No publish yet. * Update dash-p4c-bmv2-docker-acr.yml Publish to ACR using secrets * Fix macro errors * Update dash-p4c-bmv2-docker-acr.yml Force trigger. * Uncomment docker push command * Build dash-grpc:1.43.2 docker and publish to ACR * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Publish docker images to ACR and use those in build process. (#208) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Incremental progress - publish to ACRs (#212) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Use make target to publish to ACR (just bmv2-bldr as first candidate). Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Call docker publish make targets from all CI scripts which publish. * Incremental progress publishing dockers to ACR (#213) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Spellcheck - add to wordslist * Sync to main via intermediate dev branch. (#216) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <marianp@mellanox.com> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Sanjay Thyamagundalu <tsanjay@gmail.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> * Incremental progress publishing dockers to ACR (#219) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <marianp@mellanox.com> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Fix ipv6 packet noise (#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix #205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Sanjay Thyamagundalu <tsanjay@gmail.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> * Remove publish steps from non-ACR CI scripts (consequence of merge). * Incremental progress publishing dockers to ACR. (#220) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <marianp@mellanox.com> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Fix ipv6 packet noise (#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix #205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Remove publish steps from non-ACR CI scripts (consequence of merge). Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Sanjay Thyamagundalu <tsanjay@gmail.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> * Trial of env var to define docker img * Add docker .env variable to CI triggers. * Fix stray conflict markers in Makefile. * Fix CI triggers. * Use .env files to define docker images. Fix CI triggers. Remove some docker image CI triggers. * add missing path * Fix path. * Trivial change to force a CI build. * Change Makefile to force CI rebuild. * Trivial change to docker image .env file to test CI triggers. * Change .env file to force CI. * Change grpc docker .env file to trigger CI rebuild. * Change saithrift-bldr docker .env to force CI rebuild. * Change saithrift-client .env file to force CI rebuild/ * Trial docker build/conditional publish. * Get env variable into context. * Correct env filename * Debugging env vars & docker save * Fix CI syntax * Add Makefile targets to upload/download docker image tarball artifact. * Revert to simple conditional job for build+publish. * Conditional CI for non-publish docker job. * Add conditionals to all docker build/publish workflows: run build-only in forks, run build-publish in main repo. * Documenting Docker workflows. * Spellcheck fixes. * Spellcheck fixes. * Add screen cap of skipped job. * Spellcheck * Update PR from dev branch (sync to main) (#224) * Update program-scale-testing-requirements-draft.md Update Scale #'s per Keysight/MSFT call; document is a version which needs an update. * Fix Spelling (#221) * Update AMD-Pensando_HA_Proposal.md Explicitly add 'parallel' to state synchronization stages (per @lguohan ) * Update AMD-Pensando_HA_Proposal.md (#210) * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Update AMD-Pensando_HA_Proposal.md * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <marianp@mellanox.com> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Update .wordlist.txt Updating w/Chris Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: Chris Sommers <31145757+chrispsommers@users.noreply.github.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: Chris Sommers <31145757+chrispsommers@users.noreply.github.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> * Update .wordlist.txt * Slide deck for today's SAI Challenger presentation * [doc]: Fix the typo of TCP UDP protocol number (#223) According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17. * Proofreading, image clarity. Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Ze Gan <ganze718@gmail.com> * Use env variable for DASH_ACR_LOGIN_SERVER instead of secret. * Use literal expression instead of env variable. * Use literal expression for ACR repository instead of secret. * Clarify docker publishing. * Spellcheck. * Document another dockerfile workflow. * spellcheck * CI triggers on PR to any branch, not just main. * Fixes per review feedback #225 (comment): - fix image; (docxkerhub->ACR in one case - replace "local" with "host" Expand on instructions for writing and running PTF tests (from verbal feedback in WG meeting, not an issue or PR review). Discovered an oversight in the dockerfile .env files which prevented overriding image macros (had to replace = with ?=); tested fix. * Spellcheck. * Spellcheck. * Spellcheck Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Sanjay Thyamagundalu <tsanjay@gmail.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Ze Gan <ganze718@gmail.com>
chrispsommers
added a commit
that referenced
this pull request
Nov 1, 2022
…ion branches (#272) * Create dash-docker-p4c-bmv2-acr.yml * Update Makefile * Experimenting with ACR publish workflows Revert p4c-bmv2 tag * Update and rename dash-docker-p4c-bmv2-acr.yml to dash-p4c-bmv2-docker-acr.yml * Experimental ACR publish; change regname Use sonicdash ACR for p4c-bmv2. No publish yet. * Update dash-p4c-bmv2-docker-acr.yml Publish to ACR using secrets * Fix macro errors * Update dash-p4c-bmv2-docker-acr.yml Force trigger. * Uncomment docker push command * Build dash-grpc:1.43.2 docker and publish to ACR * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Publish docker images to ACR and use those in build process. (#208) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Incremental progress - publish to ACRs (#212) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Use make target to publish to ACR (just bmv2-bldr as first candidate). Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Call docker publish make targets from all CI scripts which publish. * Incremental progress publishing dockers to ACR (#213) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Added SAI-Challenger to CI setup. 1. Added SAI-Challenger submodule. 2. Added SAI-Challenger basic test. 3. Changed Makefile to build/start/stop SAI-Challenger. Signed-off-by: Maksym Hedeon <maksym.hedeon@plvision.eu> * Added saithrift to SAI-Challenger client docker image - Added saithrift to SAI-Challenger client docker image - Changed SAI-C submodule branch Signed-off-by: Maksym Hedeon <maksym.hedeon@plvision.eu> * Update example vnet test. 1. Added SAI-Challenger submodule. 2. Added SAI-Challenger basic test. 3. Changed Makefile to build/start/stop SAI-Challenger. Signed-off-by: Kostiantyn Goloveshko <kostiantyn.goloveshko@plvision.eu> Co-authored-by: Maksym Hedeon <maksym.hedeon@plvision.eu> * Add vnet outbound test based on SAI description. Signed-off-by: Maksym Prytoliuk <maksym.prytoliuk@plvision.eu> * Fixup double network host usage for sai-challenger-client Signed-off-by: Konstantin Goloveshko <kostiantyn.goloveshko@plvision.eu> * Kdt 17: Update docker environment (#8) * Added cgyang submodule * Updated docker env for thrift tests * Makefile refactor Signed-off-by: Maksym Hedeon <maksym.hedeon@plvision.eu> * Fixup VNET inbound cleanup removals in test config. Signed-off-by: Konstantin Goloveshko <kostiantyn.goloveshko@plvision.eu> * Update CA_TO_PA entry in outbound test Signed-off-by: Maksym Prytoliuk <maksym.prytoliuk@plvision.eu> * Add vnet scenarios in DASH config format. * Added dash-style VNET inbound routing test * Added dash-style VNET outbound routing test * Read config from json (temp workaround) * Added JSON setup configs for testbed Signed-off-by: Anton Putria <anton.putrya@plvision.eu> Co-authored-by: Maksym Prytoliuk <maksym.prytoliuk@plvision.eu> * Fixed saigen links. Signed-off-by: Anton Putria <anton.putrya@plvision.eu> * Fixes in test scanario. Signed-off-by: Anton Putria <anton.putrya@plvision.eu> * Fixed path to saigen. Fixed inbound test. Signed-off-by: Anton Putria <anton.putrya@plvision.eu> * Update submodule Signed-off-by: Anton Putria <anton.putrya@plvision.eu> * Final test fixes to align with submodue version. Signed-off-by: Anton Putria <anton.putrya@plvision.eu> * Spellcheck - add to wordslist * Sync to main via intermediate dev branch. (#216) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <marianp@mellanox.com> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Sanjay Thyamagundalu <tsanjay@gmail.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> * Incremental progress publishing dockers to ACR (#219) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <marianp@mellanox.com> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Fix ipv6 packet noise (#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix #205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Sanjay Thyamagundalu <tsanjay@gmail.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> * Remove publish steps from non-ACR CI scripts (consequence of merge). * Incremental progress publishing dockers to ACR. (#220) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <marianp@mellanox.com> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Fix ipv6 packet noise (#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix #205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Remove publish steps from non-ACR CI scripts (consequence of merge). Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Sanjay Thyamagundalu <tsanjay@gmail.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> * Trial of env var to define docker img * Add docker .env variable to CI triggers. * Fix stray conflict markers in Makefile. * Fix CI triggers. * Use .env files to define docker images. Fix CI triggers. Remove some docker image CI triggers. * add missing path * Fix path. * Trivial change to force a CI build. * Change Makefile to force CI rebuild. * Trivial change to docker image .env file to test CI triggers. * Change .env file to force CI. * Change grpc docker .env file to trigger CI rebuild. * Change saithrift-bldr docker .env to force CI rebuild. * Change saithrift-client .env file to force CI rebuild/ * Fixed VNET outbound test scenario. Signed-off-by: Anton Putria <anton.putrya@plvision.eu> * Trial docker build/conditional publish. * Get env variable into context. * Correct env filename * Debugging env vars & docker save * Fix CI syntax * Add Makefile targets to upload/download docker image tarball artifact. * Revert to simple conditional job for build+publish. * Conditional CI for non-publish docker job. * Outbound test passes with traffic Fixed configuration based on changes in main. Signed-off-by: Maksym Prytoliuk <maksym.prytoliuk@plvision.eu> * Add conditionals to all docker build/publish workflows: run build-only in forks, run build-publish in main repo. * Documenting Docker workflows. * Spellcheck fixes. * Spellcheck fixes. * Add screen cap of skipped job. * Spellcheck * Update PR from dev branch (sync to main) (#224) * Update program-scale-testing-requirements-draft.md Update Scale #'s per Keysight/MSFT call; document is a version which needs an update. * Fix Spelling (#221) * Update AMD-Pensando_HA_Proposal.md Explicitly add 'parallel' to state synchronization stages (per @lguohan ) * Update AMD-Pensando_HA_Proposal.md (#210) * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Update AMD-Pensando_HA_Proposal.md * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <marianp@mellanox.com> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Update .wordlist.txt Updating w/Chris Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: Chris Sommers <31145757+chrispsommers@users.noreply.github.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: Chris Sommers <31145757+chrispsommers@users.noreply.github.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> * Update .wordlist.txt * Slide deck for today's SAI Challenger presentation * [doc]: Fix the typo of TCP UDP protocol number (#223) According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17. * Proofreading, image clarity. Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Ze Gan <ganze718@gmail.com> * Use env variable for DASH_ACR_LOGIN_SERVER instead of secret. * Use literal expression instead of env variable. * Use literal expression for ACR repository instead of secret. * Clarify docker publishing. * Spellcheck. * Document another dockerfile workflow. * spellcheck * CI triggers on PR to any branch, not just main. * Update documentation - Added dash-test-sai-challenger.md doc file under test/docs - Updated Makefile make run-saic-test-thrift target with passing parameters. Signed-off-by: Maksym Hedeon <maksym.hedeon@plvision.eu> * Updated SAI-Challenger submodule. Signed-off-by: Anton Putria <anton.putrya@plvision.eu> * Improvements to SAI-Challenger docker flows. - Use docker hub to pull images - Fixed make run-saic-tests to do not show errors - Updated manuals Signed-off-by: Anton Putria <anton.putrya@plvision.eu> * Fixes per review feedback #225 (comment): - fix image; (docxkerhub->ACR in one case - replace "local" with "host" Expand on instructions for writing and running PTF tests (from verbal feedback in WG meeting, not an issue or PR review). Discovered an oversight in the dockerfile .env files which prevented overriding image macros (had to replace = with ?=); tested fix. * Spellcheck. * Spellcheck. * Update dash-test-sai-challenger.md Signed-off-by: Mircea Dan Gheorghe <mircea-dan.gheorghe@keysight.com> * Add bmv2 SAI port attributes (num_active, port_list), dflt vlan, dflt vrf) * Added SAI-Challenger docker build verification to GitHub actions - Added SAI-Challenger docker verification action. - Fixed spellchecker and docker build issues. Signed-off-by: Anton Putria <anton.putrya@plvision.eu> * Add default .1Q bridge to bmv2 Signed-off-by: Maksym Prytoliuk <maksym.prytoliuk@plvision.eu> * Fixed SAI-Challenger user guide and file namings. Signed-off-by: Anton Putria <anton.putrya@plvision.eu> * Spellcheck * Scaled VNET outbound test using snappi and saigen. (1st edition) - Split scale and simple tests / jsons - Updated ixia controller version - Added dash_helper for traffic profile scaling. Signed-off-by: Maksym Hedeon <maksym.hedeon@plvision.eu> * spellcheck * Move saichallenger client docker to ACR. * Fix .yml formatting. * Fix dependency in action file. * Add missing .env file * remove space in action script, add .env dependencies * fix include path * Add saichalleneger tests to run-all-tests and CI * Fix docker image name. * Correct mixup in image names. * Remove stray hyphen * Run saichallenger workflows on any branch PR. * Added few more VNET scenario examples. - Extended VNET traffic scenarios. - Added docstrings for helper methods. - Removed unused code. Signed-off-by: Anton Putria <anton.putrya@plvision.eu> Co-authored-by: Maksym Hedeon <maksym.hedeon@plvision.eu> * Slides from DASH WG Meeting (SAI Challenger Episode 2) (#254) * Add outbound diagram Signed-off-by: Maksym Prytoliuk <maksym.prytoliuk@plvision.eu> * Added inbound to xfail test results. (#44) - Marked VNET Inbound Routing with xfail - Added pytest.ini with marks definitions. - Decreased duration of tests. Signed-off-by: Anton Putria <anton.putrya@plvision.eu> * Scaling test configuration documentation - Added scaling approach documentation. - Vnet2vnet traffic scaling scenario description. - Added docstring to the test cases. - Added more test examples. Signed-off-by: Anton Putria <anton.putrya@plvision.eu> Co-authored-by: Maksym Hedeon <maksym.hedeon@plvision.eu> Co-authored-by: Maksym Prytoliuk <maksym.prytoliuk@plvision.eu> * Submodule update and rename * Rename and update the SAI-Challenger submodule * Enabled additional tests based on the fixes in the SAI-Challenger. * Renamed test folder. Signed-off-by: Anton Putria <anton.putrya@plvision.eu> * PTF: add basic test infra and more test cases (#231) * ptf: add vnet sample test cases for dash - Common VNet API for all tests - Test cases: - Inbound VNI/ENI - Inbound Routing PA validate - Inbound Routing VNI match - Inbound Routing PA SRC IP - Outbound Routing Vnet direct - Outbound Routing direct - CT (just placeholder) - Route (basic) Signed-off-by: Volodymyr Mytnyk <volodymyrx.mytnyk@intel.com> Signed-off-by: Yuriy Harhas <yuriyx.harhas@intel.com> * ptf: run only bmv2 ready test case on CI Signed-off-by: Volodymyr Mytnyk <volodymyrx.mytnyk@intel.com> * ptf: test_saithrift_vnet: removed unused libs Signed-off-by: Volodymyr Mytnyk <volodymyrx.mytnyk@intel.com> * test-cases folder restructure. - created two subfolders in root: functional and scale - removed outdated bmv2_model folder - moved all existing content of test-cases to test-cases/scale Signed-off-by: Anton Putria <antonx.putria@intel.com> * Moved saidashvnet.py to a proper location. Signed-off-by: Anton Putria <antonx.putria@intel.com> * Fixed ENI creation in Outbound tests Signed-off-by: Yuriy Harhas <yuriyx.harhas@intel.com> * Spellchecker related fixes. Signed-off-by: Anton Putria <antonx.putria@intel.com> * Updated VNET test cases based on the PR #238. Signed-off-by: Yuriy Harhas <yuriyx.harhas@intel.com> Signed-off-by: Volodymyr Mytnyk <volodymyrx.mytnyk@intel.com> Signed-off-by: Yuriy Harhas <yuriyx.harhas@intel.com> Signed-off-by: Anton Putria <antonx.putria@intel.com> Co-authored-by: Anton Putria <antonx.putria@intel.com> Co-authored-by: Yuriy Harhas <yuriyx.harhas@intel.com> * Adding updated license files (#266) * Fixbug: SAI Server crash on DASH ACL API (#264) 1. Fix duplicate SAI ID 2. Wrong search action 3. Double delete entry if SAI write error 4. Wrong format of error message Signed-off-by: Ze Gan <ganze718@gmail.com> * Add VNET v6 PTF test case; Fix v4 PTF test case to address test hang issue (#263) This PR adds VNET v6 test case and also fixes the PTF thrift client issue (the v6 test case just hangs) #237. This is a joint work with @chrispsommers and @aputriax. * Fixed links in the README after renaming folders. (#262) Signed-off-by: Anton Putria <antonx.putria@intel.com> Signed-off-by: Anton Putria <antonx.putria@intel.com> * Correct CI file, was publishing wrong image. Some prior merge must have caused a mixup. Signed-off-by: Maksym Hedeon <maksym.hedeon@plvision.eu> Signed-off-by: Kostiantyn Goloveshko <kostiantyn.goloveshko@plvision.eu> Signed-off-by: Maksym Prytoliuk <maksym.prytoliuk@plvision.eu> Signed-off-by: Konstantin Goloveshko <kostiantyn.goloveshko@plvision.eu> Signed-off-by: Anton Putria <anton.putrya@plvision.eu> Signed-off-by: Marian Pritsak <marianp@mellanox.com> Signed-off-by: Mircea Dan Gheorghe <mircea-dan.gheorghe@keysight.com> Signed-off-by: Volodymyr Mytnyk <volodymyrx.mytnyk@intel.com> Signed-off-by: Yuriy Harhas <yuriyx.harhas@intel.com> Signed-off-by: Anton Putria <antonx.putria@intel.com> Signed-off-by: Ze Gan <ganze718@gmail.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Maksym Hedeon <106111454+maksymhedeon@users.noreply.github.com> Co-authored-by: Kostiantyn Goloveshko <kostyagoloveshko@gmail.com> Co-authored-by: Maksym Hedeon <maksym.hedeon@plvision.eu> Co-authored-by: Maksym Prytoliuk <71631949+maksym-prytoliuk-plv@users.noreply.github.com> Co-authored-by: Anton Putria <anton7811@gmail.com> Co-authored-by: Maksym Prytoliuk <maksym.prytoliuk@plvision.eu> Co-authored-by: Anton Putria <anton.putrya@plvision.eu> Co-authored-by: Sanjay Thyamagundalu <tsanjay@gmail.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Ze Gan <ganze718@gmail.com> Co-authored-by: MirceaDan <mircea-dan.gheorghe@keysight.com> Co-authored-by: Volodymyr Mytnyk <volodymyrx.mytnyk@intel.com> Co-authored-by: Anton Putria <antonx.putria@intel.com> Co-authored-by: Yuriy Harhas <yuriyx.harhas@intel.com> Co-authored-by: Mohammad Hanif <mohammad.hanif@broadcom.com>
vijasrin
pushed a commit
to vijasrin/DASH
that referenced
this pull request
Dec 8, 2022
* Create dash-docker-p4c-bmv2-acr.yml * Update Makefile * Experimenting with ACR publish workflows Revert p4c-bmv2 tag * Update and rename dash-docker-p4c-bmv2-acr.yml to dash-p4c-bmv2-docker-acr.yml * Experimental ACR publish; change regname Use sonicdash ACR for p4c-bmv2. No publish yet. * Update dash-p4c-bmv2-docker-acr.yml Publish to ACR using secrets * Fix macro errors * Update dash-p4c-bmv2-docker-acr.yml Force trigger. * Uncomment docker push command * Build dash-grpc:1.43.2 docker and publish to ACR * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Publish docker images to ACR and use those in build process. (sonic-net#208) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Incremental progress - publish to ACRs (sonic-net#212) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Use make target to publish to ACR (just bmv2-bldr as first candidate). Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Call docker publish make targets from all CI scripts which publish. * Incremental progress publishing dockers to ACR (sonic-net#213) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Spellcheck - add to wordslist * Sync to main via intermediate dev branch. (sonic-net#216) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (sonic-net#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (sonic-net#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Split SAI API (sonic-net#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <marianp@mellanox.com> * Add APP_DB to SAI mapping (sonic-net#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Sanjay Thyamagundalu <tsanjay@gmail.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> * Incremental progress publishing dockers to ACR (sonic-net#219) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (sonic-net#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (sonic-net#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Split SAI API (sonic-net#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <marianp@mellanox.com> * Add APP_DB to SAI mapping (sonic-net#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (sonic-net#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Fix ipv6 packet noise (sonic-net#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix sonic-net#205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Sanjay Thyamagundalu <tsanjay@gmail.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> * Remove publish steps from non-ACR CI scripts (consequence of merge). * Incremental progress publishing dockers to ACR. (sonic-net#220) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (sonic-net#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (sonic-net#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Split SAI API (sonic-net#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <marianp@mellanox.com> * Add APP_DB to SAI mapping (sonic-net#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (sonic-net#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Fix ipv6 packet noise (sonic-net#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix sonic-net#205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Remove publish steps from non-ACR CI scripts (consequence of merge). Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Sanjay Thyamagundalu <tsanjay@gmail.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> * Trial of env var to define docker img * Add docker .env variable to CI triggers. * Fix stray conflict markers in Makefile. * Fix CI triggers. * Use .env files to define docker images. Fix CI triggers. Remove some docker image CI triggers. * add missing path * Fix path. * Trivial change to force a CI build. * Change Makefile to force CI rebuild. * Trivial change to docker image .env file to test CI triggers. * Change .env file to force CI. * Change grpc docker .env file to trigger CI rebuild. * Change saithrift-bldr docker .env to force CI rebuild. * Change saithrift-client .env file to force CI rebuild/ * Trial docker build/conditional publish. * Get env variable into context. * Correct env filename * Debugging env vars & docker save * Fix CI syntax * Add Makefile targets to upload/download docker image tarball artifact. * Revert to simple conditional job for build+publish. * Conditional CI for non-publish docker job. * Add conditionals to all docker build/publish workflows: run build-only in forks, run build-publish in main repo. * Documenting Docker workflows. * Spellcheck fixes. * Spellcheck fixes. * Add screen cap of skipped job. * Spellcheck * Update PR from dev branch (sync to main) (sonic-net#224) * Update program-scale-testing-requirements-draft.md Update Scale #'s per Keysight/MSFT call; document is a version which needs an update. * Fix Spelling (sonic-net#221) * Update AMD-Pensando_HA_Proposal.md Explicitly add 'parallel' to state synchronization stages (per @lguohan ) * Update AMD-Pensando_HA_Proposal.md (sonic-net#210) * SAI apigen support for tables with no action parameters and a single action (sonic-net#207) * Update .wordlist.txt * Update AMD-Pensando_HA_Proposal.md * Doc dash as submodule (sonic-net#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> * Split SAI API (sonic-net#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <marianp@mellanox.com> * Add APP_DB to SAI mapping (sonic-net#102) * Add APP_DB to SAI mapping * Update .wordlist.txt Updating w/Chris Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: Chris Sommers <31145757+chrispsommers@users.noreply.github.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: Chris Sommers <31145757+chrispsommers@users.noreply.github.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> * Update .wordlist.txt * Slide deck for today's SAI Challenger presentation * [doc]: Fix the typo of TCP UDP protocol number (sonic-net#223) According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17. * Proofreading, image clarity. Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Ze Gan <ganze718@gmail.com> * Use env variable for DASH_ACR_LOGIN_SERVER instead of secret. * Use literal expression instead of env variable. * Use literal expression for ACR repository instead of secret. * Clarify docker publishing. * Spellcheck. * Document another dockerfile workflow. * spellcheck * CI triggers on PR to any branch, not just main. * Fixes per review feedback sonic-net#225 (comment): - fix image; (docxkerhub->ACR in one case - replace "local" with "host" Expand on instructions for writing and running PTF tests (from verbal feedback in WG meeting, not an issue or PR review). Discovered an oversight in the dockerfile .env files which prevented overriding image macros (had to replace = with ?=); tested fix. * Spellcheck. * Spellcheck. * Spellcheck Signed-off-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Sanjay Thyamagundalu <tsanjay@gmail.com> Co-authored-by: Mukesh Moopath Velayudhan <mukesh@pensando.io> Co-authored-by: KrisNey-MSFT <krisney@microsoft.com> Co-authored-by: Marian Pritsak <marianp@mellanox.com> Co-authored-by: Ze Gan <ganze718@gmail.com>
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.
Build processes which publish to ACR must be run from DASH repo or a branch; forks don't have access to ACR secrets. This incremental commit pushes changes to DASH
publish-dockers-to-acrbranch to result in publishing and consuming all docker images using Azure Container REgistry (ACR).Some actions will fail on the first pass until images get published, because they pull from same ACR registry. It's a chicken-and-egg problem which will be solved in subsequent changes.