This repository was archived by the owner on Feb 15, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 13
This repository was archived by the owner on Feb 15, 2022. It is now read-only.
Dashboard missing data due to CLI generated pipelines rely on string matching #1389
Copy link
Copy link
Closed
microsoft/bedrock-cli
#29Labels
bugSomething isn't workingSomething isn't workingx-projectMapped to external projectsMapped to external projects
Milestone
Description
Describe the bug:
Enabling a preview feature in AzDO changes the title of merged pull requests. This results in unexpected behavior for introspection features of Bedrock and the dashboard.
Below we can clearly see there is a reliance on string matching:
- script: |-
set -e
cd "$HOME"/${MANIFEST_REPO##*/}
commitId=$(Build.SourceVersion)
commitId=$(echo "${commitId:0:7}")
latest_commit=$(git rev-parse --short HEAD)
url=$(git remote --verbose | grep origin | grep fetch | cut -f2 | cut -d' ' -f1)
repourl=${url##*@}
echo "Downloading SPK"
curl https://raw.githubusercontent.com/Microsoft/bedrock/master/gitops/azure-devops/build.sh > build.sh
chmod +x build.sh
. ./build.sh --source-only
get_spk_version
download_spk
message="$(Build.SourceVersionMessage)"
if [[ $message == *"Merged PR"* ]]; then
pr_id=$(echo $message | grep -oE '[0-9]+' | head -1 | sed -e 's/^0\+//')
./spk/spk deployment create -n $(INTROSPECTION_ACCOUNT_NAME) -k $(INTROSPECTION_ACCOUNT_KEY) -t $(INTROSPECTION_TABLE_NAME) -p $(INTROSPECTION_PARTITION_KEY) --p3 $(Build.BuildId) --hld-commit-id $commitId --manifest-commit-id $latest_commit --pr $pr_id --repository $repourl
else
./spk/spk deployment create -n $(INTROSPECTION_ACCOUNT_NAME) -k $(INTROSPECTION_ACCOUNT_KEY) -t $(INTROSPECTION_TABLE_NAME) -p $(INTROSPECTION_PARTITION_KEY) --p3 $(Build.BuildId) --hld-commit-id $commitId --manifest-commit-id $latest_commit --repository $repourl
fi
displayName: 'If configured, update manifest pipeline details in Spektate db'
condition: 'and(ne(variables[''INTROSPECTION_ACCOUNT_NAME''], ''''), ne(variables[''INTROSPECTION_ACCOUNT_KEY''], ''''),ne(variables[''INTROSPECTION_TABLE_NAME''], ''''),ne(variables[''INTROSPECTION_PARTITION_KEY''], ''''))'
The dashboard result:
To Reproduce:
Enable preview feature in AzDO. run pipelines, and then look at Spektate dashboard.
Expected behavior:
The dashboard should display the correct information.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingx-projectMapped to external projectsMapped to external projects

