Skip to content

Commit

Permalink
Fix doc-gen-job so that it has 3 retries before failing. (#7706)
Browse files Browse the repository at this point in the history
Summary:
Currently doc-gen-job will occasionally fail because of a race condition when updating the document job. This change will cause it to retry upto 3 times before eventually failing.

Pull Request resolved: facebookincubator/velox#7706

Reviewed By: bikramSingh91

Differential Revision: D51538017

Pulled By: kgpai

fbshipit-source-id: 0a33c6760e8616d63c7e51ee0787bc3d04890989
  • Loading branch information
kgpai authored and facebook-github-bot committed Jan 31, 2024
1 parent f4c326d commit 09d07d3
Showing 1 changed file with 24 additions and 19 deletions.
43 changes: 24 additions & 19 deletions .circleci/dist_compile.yml
Original file line number Diff line number Diff line change
Expand Up @@ -549,25 +549,30 @@ jobs:
- run:
name: "Build docs and update gh-pages"
command: |
git config --global user.email "[email protected]"
git config --global user.name "velox"
git checkout main
conda init bash
source ~/.bashrc
conda create -y --name docgenenv python=3.7
conda activate docgenenv
pip install sphinx sphinx-tabs breathe sphinx_rtd_theme chardet
source /opt/rh/gcc-toolset-9/enable
./scripts/gen-docs.sh docgenenv
git checkout gh-pages
cp -R velox/docs/_build/html/* docs
git add docs
if [ -n "$(git status --porcelain --untracked-files=no)" ]
then
git commit -m "Update documentation"
git push
fi
for i in {1..3}; do
make clean
git config --global user.email "[email protected]"
git config --global user.name "velox"
git checkout main
conda init bash
source ~/.bashrc
conda create -y --name docgenenv python=3.7
conda activate docgenenv
pip install sphinx sphinx-tabs breathe sphinx_rtd_theme chardet
source /opt/rh/gcc-toolset-9/enable
./scripts/gen-docs.sh docgenenv
git checkout gh-pages
cp -R velox/docs/_build/html/* docs
git add docs
if [ -n "$(git status --porcelain --untracked-files=no)" ]
then
git commit -m "Update documentation"
git push
if [ $? -eq 0 ]; then
break;
fi
fi
done
linux-pr-fuzzer-run:
Expand Down

0 comments on commit 09d07d3

Please sign in to comment.