-
Notifications
You must be signed in to change notification settings - Fork 482
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
[Fabric E2E Sample] Adding CI pipelines #1013
Open
camaderal
wants to merge
10
commits into
feat/e2e-fabric-dataops-sample-v0-2
Choose a base branch
from
kitsune/ci_pipelines_
base: feat/e2e-fabric-dataops-sample-v0-2
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[Fabric E2E Sample] Adding CI pipelines #1013
camaderal
wants to merge
10
commits into
feat/e2e-fabric-dataops-sample-v0-2
from
kitsune/ci_pipelines_
+2,983
−433
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
camaderal
changed the title
Adding CI pipelines
[Fabric E2E Sample] Adding CI pipelines
Jan 10, 2025
5 tasks
3 tasks
* Adding repo setup script * Removing old script * Fixing different commit issues * Fix
…reation (#1053) * feat: add keyvault support for variable groups. Add service connection creation. * feat: scope service connection to resource group, pass azure subscription name * feat: add to readme * Fixing MI issues * env template changes * [Fabric E2E Sample] Deploying Fabric items to DEV state only (#1059) * Deploying Fabric items to DEV state only --------- Co-authored-by: Anuj Parashar <promisinganuj@gmail.com>
* yuna/readme-update * update * change the structure of readme * correct header level * add clean up pipeline in sequence and update sequence diagram * MOB Review for Readme * Fix links * Minor updates * Upating doco * Review comment incorporated --------- Co-authored-by: Christeena <cmaderal@microsoft.com> Co-authored-by: Anuj Parashar <promisinganuj@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.
Type of PR
Purpose
Added the following CI pipelines
$FABRIC_WORKSPACE_NAME-$PR_ID
doesn’t exist.$FABRIC_CUSTOM_POOL_NAME
if it doesn't exist. Assign it to the workspace.fabric/fabric_environment/spark_pool_settings.yml
file of the feature branch.feature-$PR-ID
if it doesn't exist.$FABRIC_CONNECTION_NAME-$PR_ID
if it doesn't exist.feature-$PR-ID
storage containerFABRIC_WORKSPACE_NAME
will be the feature workspaceFABRIC_WORKSPACE_NAME-$PR_ID
Added a fabric workspace unit test
get_workspace_id
get_item_id
andrun_job
so you can get the ids if you know the name. (Original and feature workspace should be sharing the same name for notebooks, pipelines, etc)Added a setup repository script
$GIT_DIRECTORY_NAME/fabric/workspace
instead.Does this introduce a breaking change? If yes, details on what can break
No.
Author pre-publish checklist
Validation steps
To test the pipelines, you need to have your git repositories to be a certain way. Here are steps on how to set it up.
Generate your Azure Devops Credentials then add the following env variables
GIT_USERNAME
andGIT_TOKEN
. Other than that make sure, these env variables are also set:GIT_ORGANIZATION_NAME
GIT_PROJECT_NAME
GIT_REPOSITORY_NAME
GIT_BRANCH_NAME
GIT_DIRECTORY_NAME
Run the
scripts/setup_repository.py
This would add all the necessary files to theGIT_DIRECTORY_NAME
path in the Azure DevOps repo. This will also create thefabric/workspace
folder where the fabric workspace will be synced. The file structure should look like this (minus thefabric/workspace
part which would be generated later):Either:
Create the variable group. Here are the required values:
WORKING_DIR
: Folder path where you committed your repository. Same asGIT_DIRECTORY_NAME
in the env variablesSUBSCRIPTION_ID
: Same as theSUBSCRIPTION_ID
in the env variablesRESOURCE_GROUP_NAME
: Same as theRESOURCE_GROUP_NAME
in the env variablesSTORAGE_ACCOUNT_NAME
: Created storage account nameSTORAGE_ACCOUNT_ROLE_DEFINITION_ID
(Role definition id given to fabric workspace to access storage account)STORAGE_CONTAINER_NAME
: Default is ”main”KEYVAULT_NAME
: Created key vault nameORGANIZATIONAL_NAME
: Same as theGIT_ORGANIZATION_NAME
in the env variablesPROJECT_NAME
: Same as theGIT_PROJECT_NAME
in the env variablesREPO_NAME
: Same as theGIT_REPOSITORY_NAME
in the env variablesFABRIC_WORKSPACE_GROUP_ADMIN
: Principal Id of the Group Admin for the workspace and cloud connectionFABRIC_WORKSPACE_NAME
: Created workspace nameFABRIC_CAPACITY_NAME
: Created capacity nameFABRIC_ENVIRONMENT_NAME
: Created environment nameFABRIC_LAKEHOUSE_NAME
: Created lakehouse nameFABRIC_SHORTCUT_NAME
: Default is “sc-adls-main”FABRIC_CUSTOM_POOL_NAME
: Created custom pool name.FABRIC_CONNECTION_NAME
: Created ADLS Cloud Connection name.Modify the following values in the pipelines and commit it.
Setup the pipeline then run it.
Known issue:
Issues Closed or Referenced