Re: Pullrequest required checks

Quick addendum, it looks like you would have to update the master branch as well as develop. As the master branch<https://github.com/apache/geode/blob/master/.asf.yaml#L39> has more required checks for reason. -michael From: Michael Oleske Sent:

Re: Pullrequest required checks

I believe the required checks are setup in the .asf.yaml file here. I would hope just deleting those lines would remove the existing checks and in your migrate to github actions pr you could change the required checks. I would hope

Re: https://concourse.apachegeode-ci.info/ is shutting down

views on GF actions. If you can find another project that has already done this, it’s a lot easier. Anthony > On Oct 7, 2022, at 8:24 AM, Mario Salazar de Torres > wrote: > > ⚠ External Email > > Hi everyone, > > Regarding this, some time ago Michael Oleske and myse

Rescued Geode Protobuf

Hi Geode Friends! Since I didn't see any movement on rescuing Geode Protobuf into another repository, I went ahead and did that. It seemed like a better idea to me than trying to search commit history or old branches with git. It is located at https://github.com/moleske/geode-protobuf I'm

State of https://github.com/apache/geode-dotnet-core-client

Hi Geode Dev Friends! I'm starting a personal little side project and was thinking of using Geode as my backend (no real reason, certainly doesn't need the capabilities of Geode, but thought it'd be fun.) I'm writing my application in .NET 5 (which is core based) and was going to try and use

Re: [PROPOSAL] Postpone Geode 1.14

@geode.apache.org Subject: Re: [PROPOSAL] Postpone Geode 1.14 +1 to Michael's suggestion. On 7/31/20, 5:38 PM, "Michael Oleske" wrote: Is there plans as a community to do a postmortem or retrospective around why the release is taking so long? If there is an understanding of events that

Re: [PROPOSAL] Postpone Geode 1.14

Is there plans as a community to do a postmortem or retrospective around why the release is taking so long? If there is an understanding of events that occurred to cause the long delay of Geode 1.13 to be released, that would help inform decisions if processes should change or if the release

Re: [DISCUSSION] Stop using the Geode Repository for Feature/WIP Branches

> Suppose I want to commit to another contributor's fork. How can they grant > me permission to do so? (This is a common predicament for me when I'm > reviewing doc PRs.) Just as a note: if there is a PR and it is from a fork, the person who opens the PR can check a box that says "Allow

Re: Over usage of @SuppressWarnings

For context, here is an example of PR that added warnings as error https://github.com/apache/geode/pull/4816. Here is the JIRA https://issues.apache.org/jira/projects/GEODE/issues/GEODE-7869 -michael On Fri, May 8, 2020 at 11:45 AM Kirk Lund wrote: > I'm reviewing lots of PRs which are >

Re: GitHub Project Board Applications API

ttempt hence not being > discussed at this point in time as we don't have all the information. > > Regards > Naba > > > On Fri, May 1, 2020 at 4:00 PM Michael Oleske wrote: > > > Hi Devs! > > > > I saw on github that the main geode project now has a pro

GitHub Project Board Applications API

Hi Devs! I saw on github that the main geode project now has a project board ( https://github.com/apache/geode/projects/1). I don't remember it being discussed on the dev list. Are some folks using it now for organization? I'm curious how that flow works since our Jira could also be doing that

Re: [DISCUSS] Adding Google Analytics to our website

What things are we looking to learn? Without knowing what we are interested in learning I would be hesitant to add anything. If we know what we want to learn then a conversation about analytics would be more fruitful (to me at least) -michael On Tue, Apr 7, 2020 at 3:31 PM Alexander Murmann

Re: RFC: Add C Bindings to Geode Native Client

+1 Would certainly be nice since the protobuf work is still mostly experimental (and then I can continue my goal of Geode-ing all the languages) -michael On Wed, Mar 25, 2020 at 9:03 AM Dan Smith wrote: > +1 > > Great idea! Hey, it's also easy to call into C libraries from Java - maybe > we

Re: [VOTE] Using Github issues and wiki for geode-kafka-connector project

+1 !! -michael On Mon, Mar 23, 2020 at 8:33 AM Aaron Lindsey wrote: > YES +1 > > > On Mar 23, 2020, at 7:42 AM, Bruce Schuchardt > wrote: > > > > +1 > > > > On 3/21/20, 5:17 PM, "Nabarun Nag" wrote: > > > >Hello team, > > > >We are planning to experiment with

[Polite Request] Clean up branches on Geode

There's a lot of branches on Geode that look like they can be deleted because they have either been merged or closed. Below is a list. Please consider deleting your branch if you have created one of the below. Thanks! -michael The following branches on Geode have been merged

Re: privacy protection

Something to consider is what scenarios do you want to protect against. Full disk encryption protects against a drive pull attack, such as an attack walks into a data storage room full disks, pulls a disk from the room, and then makes a run for it. Since the full disk is encrypted, the attacker

Re: [VOTE] Apache Geode 1.11.0.RC4

+1 - ran `./gradlew build` on Geode 1.11.0.RC4 on Mac and Windows - built Geode Native 1.11.0.RC4 against Geode 1.11.0.RC4 on Mac and Windows - ran `ctest -L STABLE -C Debug` on cppcache/integration-test on Mac and Windows - ran `ctest` on cppcache/integration/test on Mac and Windows (Did not

Re: PR Checks issues?

r feedback) from someone else in the community who will > be using these LGTM alerts regularly. > > Best, > Joey > > On Fri, Dec 13, 2019 at 11:31 AM Michael Oleske > wrote: > > > Hi Geode Dev! > > > > This PR https://github.com/apache/geode/pull/4406 is a

PR Checks issues?

Hi Geode Dev! This PR https://github.com/apache/geode/pull/4406 is a change to the readme. However it took 3 empty commits to get it to go green enough to be allowed to be merged. This seems odd, especially with just a readme change. Is there something going with how CI works for PRs? This

Re: DistributionArchUnitTest OutOfMemoryError

I've hit the same error as Kirk. My lazy solution was to just rerun `./gradlew build` cause that would pass on the second time. -michael On Wed, Nov 20, 2019 at 4:53 PM Dan Smith wrote: > We recently added this test. It passes for me, but I will look into it. It > is scanning classes, so that

Re: [VOTE] Apache Geode 1.9.2.RC1

+1 built and ran tests for native client on windows. -michael On Wed, Oct 16, 2019 at 2:21 PM Blake Bender wrote: > Okay I'm convinced we're okay, so switching to a +1. Standard PEBKAC error > here, I managed to produce a native client build which was missing the jar > file we use for a

Re: [DISCUSS] GEODE-7241 - make Jar not War?

For context, these are the docs for running Pulse as stand alone https://geode.apache.org/docs/guide/11/tools_modules/pulse/quickstart.html#topic_795C97B46B9843528961A094EE520782 The instructions always seemed odd since we tell folks to go "tools/pulse" to copy the pulse.war file to their

Re: Propose including GEODE-7178 in 1.10

us though - is the native client behaving like an older versions > of the java client, or is this totally unique behavior for the native > client? Is there some integration test that we are missing here? > > -Dan > > On Thu, Sep 12, 2019 at 11:52 AM Michael Oleske > wrote:

Re: Propose including GEODE-7178 in 1.10

o merge? > > Regards > > -Dick > > > On Thu, Sep 12, 2019 at 10:23 AM Anthony Baker wrote: > > > +1 yes please! > > > > > On Sep 12, 2019, at 10:11 AM, Michael Oleske > wrote: > > > > > > Hi Geode Devs! > > > > &

Propose including GEODE-7178 in 1.10

Hi Geode Devs! I'd like to propose including the fix for GEODE-7178. This resolves an issue that Ivan (https://markmail.org/message/dwwac42xmpo4xb2e) ran into in 1.10 RC1. SHA: 91176d61df64bf1390cdba7b1cdc2b40cdfaba3a Link to GitHub:

Re: Gradle daemon crashed in PR precheckin

Not sure what the exact issue in this particular case, but I suspect we might not want to run with the Gradle daemon in CI. The Gradle daemon is really meant to help local building (see https://docs.gradle.org/current/userguide/gradle_daemon.html). That link explicitly calls out that you may not

Updating geode-native-build docker image

Hi Geode Devs! Geode Native merged https://github.com/apache/geode-native/pull/509 this morning since our docker image was using an old Geode version. What is the proper way to update docker hub ( https://hub.docker.com/r/apachegeode/geode-native-build) with the new image? Is that something

Re: ./gradlew test no longer runs/reruns tests?

For more information/background on gradle and running tests, I suggest reading this blog post from the gradle team https://blog.gradle.org/stop-rerunning-tests -michael On Wed, Aug 7, 2019 at 11:58 AM Jacob Barrett wrote: > Gradle has, for as long as I can recall, always had this behavior for

Re: New release branch for Apache Geode 1.10.0

Naba's concern was this PR: https://github.com/apache/geode/pull/3880 which has been merged to develop -michael On Fri, Aug 2, 2019 at 4:33 PM Owen Nichols wrote: > Hi Naba, thank you for bringing your concern. > > Our current process < >

Re: [DISCUSS] Add a test dependency to geode-core - ArchUnit

Seems alright to me given the context that you're trying to separate code into modules (and want to make sure you're not including packages you shouldn't). Looking forward to see how the progress goes! -michael On Thu, Jun 20, 2019 at 4:35 PM Dan Smith wrote: > We don't have a membership

Re: [DISCUSS] is it time to make Windows tests gating?

Agreed on making them gating if we want to say we run on Windows. If we don't want to say we run on Windows, delete the jobs. -michael On Thursday, May 16, 2019, Anilkumar Gingade wrote: > >> around 5 hours, vs 2 hours for Linux tests). > May be a good time to look at reducing/optimizing this.

Pulse - Non-Embedded Mode

Hi Geode Community! Some colleagues and I were looking at GEODE-6683 ( https://issues.apache.org/jira/browse/GEODE-6683) and noticed that we do not have test coverage for running Pulse in non-embedded mode. We were wondering what our strategy is around Pulse in non-embedded mode. In order to

Re: [VOTE] Apache Geode 1.9.0 RC4

+1 Built geode-native with clang against geode build from source. Ran geode-native integration test suite. Built and ran C++ examples. All done on clang. -michael On Wed, Apr 24, 2019 at 2:29 PM Charlie Black wrote: > +1 Geode Native client built on my Mac successfully. > > On Wed, Apr 24,

Re: How to run PulseAuthorizationTest

These test require you to have the Chrome Driver from Selenium. You can find it here https://github.com/SeleniumHQ/selenium/wiki/ChromeDriver If you are on a Mac, you can probably (haven't tested this in awhile) use brew with this command brew cask install chromedriver -michael On Mon, Apr

Re: [Geode Build] Are the idea and eclipse plugins being used by anyone?

I tend to import into Intellij the way you have described. I've never used ./gradlew idea to configure. -michael On Wed, Apr 3, 2019 at 2:24 PM Patrick Rhomberg wrote: > I'm eyeballing the ide.gradle file we have and am wondering if it is cruft > from many iterations of both Gradle and

Bug Numbers and Trac Numbers in comments

Hey Geode Dev Friends! I was reviewing a PR (this one https://github.com/apache/geode/pull/3197) and made a note that maybe we should remove comments that make references to bug and trac numbers that people cannot reach (like me for one). Kirk mentioned that some people (like him) have access to

Re: Geode 1.9 Release Manager

I have merged https://github.com/apache/geode/pull/3195 just now. -michael On Fri, Feb 15, 2019 at 8:27 AM Sai Boorlagadda wrote: > BTW, Is PR #3195 is planned to merge today? > > On Fri, Feb 15, 2019 at 7:53 AM Sai Boorlagadda > > wrote: > > > I am planning to cut the1.9 release branch today

Re: Should Geode stats conform to backwards compatibility constraints?

Here's a PR for a revert of the offending commits. https://github.com/apache/geode/pull/3195 Seems valuable to revert for now and decide a path forward rather than rush to get in 1.9 -michael On Wed, Feb 13, 2019 at 9:51 PM Jacob Barrett wrote: > How about mod MAX_INT? It would wrap back to

Re: Spotless Check failing on Windows?

For those who like closure here's a PR <https://github.com/apache/geode/pull/2873> and Geode Issue <https://issues.apache.org/jira/browse/GEODE-6073> for a fix to the issue -michael On Mon, Nov 12, 2018 at 6:10 PM Michael Oleske wrote: > I'll look to make time to play with i

Re: Spotless Check failing on Windows?

build on Windows, but with -x spotlessCheck > > > > Someone (maybe whoever’s touched th spotless rules most recently) might > > know what to do. I tend to agree that the same source code ought to be > able > > to pass spotless on both Linux and windows; and our pipelin

Spotless Check failing on Windows?

Hi Geode Dev Friends! I was building the latest off of develop (hash 75d7ed8a49d6) in PowerShell on my Windows box and noticed during build that the spotless check failed. It strangely fails on line endings (\r\n fun times) on what seems to be all (or at least 34 of them) build.gradle files.

Re: Geode Native & Apache Geode 1.8 Release

Plus 1 for source release. Exciting times we live in! For verifying, plus one to a pipeline that's not just travis. Though they're instructions in the repo about how to run tests to get that baseline confidence. -michael On Wednesday, October 10, 2018, Anilkumar Gingade wrote: > Good work

Re: Instructions for Setting Up IntelliJ

+1 for the repo as source of truth. Wiki always seemed better as a place to show examples rather than how to build the project. I also like having the readme about how to use the product and the building about how to manually build so one could contribute (especially since Geode already provided

GEODE Jira Access

Hi all, I'd like to be able to update GEODE Jira to reflect what I'm working on, so was hoping to get access. Thanks! michael Michael Oleske Software Engineer Pivotal - Santa Monica