Skip to content

Conversation

@yakov-olkhovskiy
Copy link

No description provided.

@google-cla
Copy link

google-cla bot commented Mar 11, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@github-actions
Copy link

yakov-olkhovskiy is a new contributor to projects/clickhouse. The PR must be approved by known contributors before it can be merged. The past contributors are: nickitat, Felixoid, evillique, CurtizJ, nikitamikhaylov, kitaisreal, blinkov, qoega, kyprizel

@nikitamikhaylov
Copy link
Contributor

We ended up just moving the infra to our own CI because we found that to be more convenient! Don't worry, we will continue fuzzing!

@nikitamikhaylov
Copy link
Contributor

@jonathanmetzman Can you please review? 😢

@jonathanmetzman
Copy link
Contributor

Sure :-(
Can you get approval from one of the maintainers listed above?

Is there anything annoying or bad about OSS-Fuzz that prompted the switch? Are you going to be using ClusterFuzzLite?

@nikitamikhaylov
Copy link
Contributor

@jonathanmetzman I was listed in the list of maintainers :)

The past contributors are: nickitat, Felixoid, evillique, CurtizJ, nikitamikhaylov, kitaisreal, blinkov, qoega, kyprizel

nikitamikhaylov <- This is me.

The OSS-Fuzz is not annoying, but it is just inconvinient for us to access its results + the build system is quite different which complicates supporting it. We constantly receive the emails about that the build is broken, while it is working Ok in our CI - possibly because we are switching to the new compiler way faster than the OSSFuzz.

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.

3 participants