Skip to content

Commit 8c6d69c

Browse files
svarnausvarnau
committed
Squashed commit of the following:
commit 19b0b6e Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Fri May 8 10:24:43 2026 -0700 stray quotes commit aa80494 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Fri May 8 10:20:16 2026 -0700 add backport title check commit 5832381 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Fri May 1 14:19:46 2026 -0700 Revert "testing action" This reverts commit b687798. commit b687798 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Fri May 1 13:50:46 2026 -0700 testing action commit 48bab34 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Fri May 1 13:49:53 2026 -0700 add upgrade check, remove broken actions commit ee2a297 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Fri May 1 11:12:35 2026 -0700 fix up docker call commit 2b77654 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Fri May 1 11:05:21 2026 -0700 test avoiding chown commit 41682ab Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Fri May 1 11:02:17 2026 -0700 Revert "check golint and pep8 linters" This reverts commit cd629a6. commit c0a4dbe Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Fri May 1 10:42:01 2026 -0700 try as 2 docker steps commit cd629a6 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Fri May 1 09:54:38 2026 -0700 check golint and pep8 linters commit abc46bc Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Fri May 1 09:23:09 2026 -0700 lint without custom container image commit 4c505f4 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Fri May 1 09:11:40 2026 -0700 checkpoint commit e3a1198 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Thu Apr 30 16:15:21 2026 -0700 safe dir git config commit 45c2701 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Thu Apr 30 16:10:30 2026 -0700 fix quoting in docker script commit b802db4 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Thu Apr 30 16:04:10 2026 -0700 debug path commit 632744a Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Thu Apr 30 15:49:21 2026 -0700 regex for bash ERE commit f5bd5ba Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Thu Apr 30 14:23:48 2026 -0700 ybci user and stray quote commit 05c64bf Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Thu Apr 30 14:05:47 2026 -0700 fix docker options commit 37ce8d3 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Thu Apr 30 13:52:38 2026 -0700 fix stray quote commit f8544df Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Thu Apr 30 13:50:50 2026 -0700 Please enter the commit message for your changes. Lines starting moved to workflows commit d6da17b Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Thu Apr 30 13:50:40 2026 -0700 move lint action file commit 4cfd1e4 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Thu Apr 30 13:45:33 2026 -0700 lint file commit a7ed90c Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Thu Apr 30 13:45:00 2026 -0700 lint file commit 4e29ce1 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Thu Apr 30 13:44:44 2026 -0700 move lint to separate file to change event triggers commit efc3b47 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Thu Apr 30 13:20:26 2026 -0700 reduced lint rule commit 2e1179a Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Wed Apr 29 16:49:05 2026 -0700 add PR checks, remove PR builds commit db30c62 Author: Steve Varnau <steve.varnau@yugabyte.com> Date: Tue Apr 28 09:49:50 2026 -0700 WIP - automation testing
1 parent 2a293ab commit 8c6d69c

7 files changed

Lines changed: 131 additions & 95 deletions

File tree

‎.github/workflows/build.yml‎

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -14,19 +14,6 @@ on:
1414
- '.gitignore'
1515
- '.github/vale-styles/**'
1616

17-
pull_request:
18-
branches:
19-
- master
20-
# Duplicated list because github does not support YAML anchors.
21-
# https://github.com/actions/runner/issues/1182
22-
paths-ignore:
23-
- README.md
24-
- 'docs/**'
25-
- 'architecture/**'
26-
- 'managed/**'
27-
- '.gitignore'
28-
- '.github/vale-styles/**'
29-
3017
# A workflow run is made up of one or more jobs that can run sequentially or in parallel
3118
jobs:
3219
# This workflow contains a single job called "build"

‎.github/workflows/managed-java-checks.yaml‎

Lines changed: 0 additions & 38 deletions
This file was deleted.

‎.github/workflows/pull_check_upgrade.yml‎

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
name: "yb-gh"
2+
3+
on:
4+
pull_request:
5+
types: [opened, edited, reopened, synchronize]
6+
paths:
7+
- 'src/**/*.proto'
8+
9+
jobs:
10+
pr-upgrade:
11+
runs-on: ubuntu-latest
12+
steps:
13+
- name: Check PR upgrade info
14+
run: |
15+
body="${{ github.event.pull_request.body }}"
16+
UPG_RE='upgrade.*(downgrade|rollback)\ssafety'
17+
# downcase the body for regex matching
18+
if [[ ! "${body,,}" =~ $UPG_RE ]]; then
19+
echo "::error ::PR description must include upgrade/rollback safety information"
20+
echo "::error ::for any changes to .proto files."
21+
echo "::error ::This should include whether the change is safe for rollback"
22+
echo "::error ::and any special procedures needed."
23+
echo "::error ::Example: Upgrade/Downgrade Safety: ..."
24+
exit 1
25+
fi

‎.github/workflows/pull_checks.yml‎

Lines changed: 59 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,59 @@
1+
name: "yb-gh"
2+
3+
# The title/description checks are only really needed on open/edit events,
4+
# But then it won't be available from head commit if the PR is synchronized.
5+
on:
6+
pull_request:
7+
types: [opened, edited, synchronize, reopened]
8+
paths-ignore:
9+
- README.md
10+
- 'docs/**'
11+
12+
jobs:
13+
pr-title:
14+
runs-on: ubuntu-latest
15+
steps:
16+
- name: Check PR title
17+
run: |
18+
TITLE_RE='^(\[(BACKPORT )?([0-9]+(\.[0-9]+)+)\])?'
19+
TITLE_RE+='\[((#[0-9]+(,#[0-9]+)*)|([A-Z]+-[0-9]+(,[A-Z]+-[0-9]+)*))\]'
20+
TITLE_RE+=' [a-zA-Z0-9_-]+(,[[a-zA-Z0-9_]-]+)*:'
21+
TITLE_RE+=' \S.{8,98}\S$'
22+
23+
if [[ ! "${{ github.event.pull_request.title }}" =~ $TITLE_RE ]]; then
24+
echo "::error ::PR title must include issue ID, code area (no extra whitespace),"
25+
echo "::error :: and a short description."
26+
echo "::error ::Issue referenced must be a github issue or YB-internal Jira ID"
27+
echo "::error ::Short description must be less than 100 characters."
28+
echo "::error ::Backport branch should be used only for non-master release branches."
29+
echo "::error ::Examples:"
30+
echo "::error :: [#12345,#12367] area: Cool feature xyz"
31+
echo "::error :: [BACKPORT 2045.1][#12345] area1,area2: Must have fix"
32+
exit 1
33+
fi
34+
35+
- name: Check Backport title
36+
run: |
37+
BACKPORT_RE='^(\[(BACKPORT )?([0-9]+(\.[0-9]+)+)\])'
38+
if [[ "${{ github.base_ref }}" == "master" ]]; then
39+
if [[ "${{ github.event.pull_request.title }}" =~ $BACKPORT_RE ]]; then
40+
echo "::error ::PR title should not specify target branch if targeting master."
41+
exit 1
42+
fi
43+
else
44+
if [[ "${{ github.event.pull_request.title }}" =~ $BACKPORT_RE ]]; then
45+
BACKPORT_BRANCH="${BASH_REMATCH[1]}"
46+
if [[ "${BACKPORT_BRANCH}" != "${{ github.base_ref }}" ]]; then
47+
echo "::error ::Backport branch in title (${BACKPORT_BRANCH}) does not match"
48+
echo "::error :: PR base ref (${{ github.base_ref }})."
49+
exit 1
50+
fi
51+
else
52+
echo "::error ::PR title must begin with target release branch in square brackets."
53+
echo "::error ::Examples:"
54+
echo "::error :: [BACKPORT 2045.1][#12345] area1,area2: Must have fix from master"
55+
echo "::error :: [2045.2][#12345] area: release-specific change"
56+
exit 1
57+
fi
58+
fi
59+
# Check if there is backport commit/PR info in desc.

‎.github/workflows/pull_lint.yml‎

Lines changed: 44 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,44 @@
1+
name: "yb-gh"
2+
3+
on:
4+
pull_request:
5+
types: [opened, synchronize, reopened, edited ]
6+
paths-ignore:
7+
- README.md
8+
- 'docs/**'
9+
10+
jobs:
11+
pr-lint:
12+
runs-on: ubuntu-latest
13+
steps:
14+
- uses: actions/checkout@v6
15+
with:
16+
fetch-depth: 0
17+
fetch-tags: 'false'
18+
19+
# TODO: Create GH custom image with pycodestyle & golint installed
20+
- name: Lint
21+
run: |
22+
docker run \
23+
--user root \
24+
-v "$(pwd):$(pwd)" \
25+
-w "$(pwd)" \
26+
yugabyteci/yb_build_infra_ubuntu2204_x86_64:latest \
27+
bash -c 'chown -R yugabyteci .'
28+
docker run \
29+
--user yugabyteci \
30+
-v "$(pwd):$(pwd)" \
31+
-w "$(pwd)" \
32+
yugabyteci/yb_build_infra_ubuntu2204_x86_64:latest \
33+
bash -c '
34+
set -euo pipefail
35+
# Base reference should be set for PR actions.
36+
set -x
37+
if [[ -n "${{ github.base_ref }}" ]]; then
38+
git fetch origin ${{ github.base_ref }}
39+
./build-support/lint.py --rev origin/${{ github.base_ref }}
40+
else
41+
echo No base reference to compare
42+
exit 1
43+
fi
44+
'

‎.github/workflows/yb-ctl-test.yml‎

Lines changed: 0 additions & 44 deletions
This file was deleted.

‎build-support/jenkins/yb-jenkins-build.sh‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,9 @@ EOT
3939

4040
echo "Jenkins Build script ${BASH_SOURCE[0]} is running"
4141

42+
echo "Intentional build failure"
43+
exit 1
44+
4245
while [ $# -gt 0 ]; do
4346
case "$1" in
4447
-h|--help)

0 commit comments

Comments
 (0)