Skip to content

[2025.2][DEVOPS-3735] ci: github actions for PR static check#31526

Closed
svarnau wants to merge 1 commit into
yugabyte:2025.2from
svarnau:gh2025.2
Closed

[2025.2][DEVOPS-3735] ci: github actions for PR static check#31526
svarnau wants to merge 1 commit into
yugabyte:2025.2from
svarnau:gh2025.2

Conversation

@svarnau
Copy link
Copy Markdown
Member

@svarnau svarnau commented May 8, 2026

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 b6877984608175053fe4139cd715c776ea49ccc1.

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 cd629a60e6853e07ad9ae0086a12e4dba9a0d884.

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

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
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces an intentional build failure in the Jenkins build script by adding an exit command. The review feedback correctly identifies this as leftover debugging code that should be removed to prevent unconditional build failures.

Comment on lines +42 to +44
echo "Intentional build failure"
exit 1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

critical

This code causes an unconditional build failure and appears to be leftover debugging or testing code. It should be removed before merging.

@svarnau svarnau closed this May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant