Skip to content
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

Feature/block96 Undo the changes made in case of pipeline generation failure #108

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open
98 changes: 92 additions & 6 deletions scripts/pipelines/azure-devops/pipeline_generator.sh
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,6 @@ scriptFilePath=".pipelines/scripts" # Path to the scripts.
export provider="azure-devops"

function obtainHangarPath {

# This line goes to the script directory independent of wherever the user is and then jumps 3 directories back to get the path
hangarPath=$( cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null && cd ../../.. && pwd )
}
Expand All @@ -72,7 +71,20 @@ function createPipeline {
projectName=$(echo "$azRepoShow" | python -c "import sys, json; print(json.load(sys.stdin)['project']['name'])")

# Create Azure Pipeline
az pipelines create --name $pipelineName --yml-path "${pipelinePath}/${yamlFile}" --skip-first-run true --organization "https://dev.azure.com/$orgName" --project "$projectName" --repository "$repoName" --repository-type tfsgit

pipelineResult=$(az pipelines create --name $pipelineName --yml-path "${pipelinePath}/${yamlFile}" --skip-first-run true --organization "https://dev.azure.com/$orgName" --project "$projectName" --repository "$repoName" --repository-type tfsgit) || {
# if the script fails, clean pollution
echo "There was an error creating the pipeline! Please check if you specified the name and got all settings right!"

undoPreviousSteps

exit 127
}

pipelineId=$(echo "$pipelineResult" | python -c "import sys, json; print(json.load(sys.stdin)['id'])")

# pipeline got created successfully
undoStage=3
}

# Function that adds the variables to be used in the pipeline.
Expand All @@ -82,7 +94,15 @@ function addCommonPipelineVariables {
echo "Skipping creation of the variable artifactPath as the flag has not been used."
else
# Add the extra artifact to store variable.
az pipelines variable create --name "artifactPath" --pipeline-name "$pipelineName" --value "${artifactPath}"
$(az pipelines variable create --pipeline-name "$pipelineName" --value "${artifactPath}") || {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SC2091: Remove surrounding $() to avoid executing output (or use eval if intentional).

(at-me in a reply with help or ignore)


Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

# if the variable-creation fails, clean up

echo "There was an error creating the pipeline-artifact-path variable!. Exiting"

undoPreviousSteps

exit 127
}
fi
}

Expand Down Expand Up @@ -136,6 +156,56 @@ function createPR {
fi
}

function removeLocalBranches {
# visit the directory and switch to the branch which was present before
cd "${localDirectory}"

git checkout $originalBranch
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SC2154: originalBranch is referenced but not assigned.

Reply with "@sonatype-lift help" for info about LiftBot commands.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.


Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]


# delete branch
git branch -D ${sourceBranch}
}

function removeRemoteBranches {
cd "${localDirectory}"

# update list of remotes
git fetch

# delete mapped remote-branch
git push origin --delete ${sourceBranch}
}
Comment on lines +169 to +177
Copy link
Contributor

@patricpoba patricpoba May 26, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you have to delete the pull request created by the createPR step before removing the remote branch. that means you need another function to undo the createPR step.


function removePipeline {
az pipelines delete --id ${pipelineId}
}

function undoPreviousSteps {
# undo all actions taken

if [ ${undoStage} -gt 2 ]; then
echo "Removing pipeline with id: ${pipelineId}"
removePipeline
fi

if [ ${undoStage} -gt 1 ]; then
echo "Removing all remote branches!"

removeRemoteBranches
fi

if [ ${undoStage} -gt 0 ]; then
echo "Removing all local branches!"

removeLocalBranches
fi
}

if [[ "$help" == "true" ]]; then help; fi

# Undo-Level for the script. Used to clean up the resources in case of a failure
undoStage=0

obtainHangarPath

# Load common functions
Expand All @@ -149,19 +219,35 @@ importConfigFile

checkInstallations

obtainHangarPath
cd "${localDirectory}"

# store current branch into a variable (only used for rollback/undo)
originalBranch=$(git branch --show-current)

createNewBranch

undoStage=1

copyYAMLFile

copyCommonScript

type copyScript &> /dev/null && copyScript
type copyScript &> /dev/null && copyScript || {
BenjaminE98 marked this conversation as resolved.
Show resolved Hide resolved
undoPreviousSteps

exit 127
}

commitCommonFiles

type commitFiles &> /dev/null && commitFiles
# clean up remote-branches
undoStage=2

type commitFiles &> /dev/null && commitFiles || {
undoPreviousSteps

exit 127
}

createPipeline

Expand Down