yelirekim (Mike Riley)
Overloaded Operator

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Aug 25 2012, 1:10 AM (256 w, 5 d)
Availability
Available

I work at Uber ATC and we kinda like Phabricator.

Recent Activity

Fri, Jun 30

yelirekim awarded D18177: Use the correct "completed" time in Harbormaster display UI a Like token.
Fri, Jun 30, 2:47 PM

May 24 2017

Pawka awarded T11518: Make arc feature much more fancy and fun a Like token.
May 24 2017, 2:31 PM · Restricted Project, Feature Request

Apr 17 2017

yelirekim added a comment to T12232: Phortune "add payment method" erroring out on submission.

Yeah I just wanted to buy myself a badge.

Apr 17 2017, 2:45 PM · Phortune, Bug Report
yelirekim backed I2 Donate to Phacility with $13.37 USD.
Apr 17 2017, 2:42 PM · Phacility
yelirekim added a comment to T12232: Phortune "add payment method" erroring out on submission.

No nothing like that but FWIW, it just worked when I tried to reproduce it again.

Apr 17 2017, 2:41 PM · Phortune, Bug Report
yelirekim added a comment to T12232: Phortune "add payment method" erroring out on submission.

Yeah.

Apr 17 2017, 2:38 PM · Phortune, Bug Report
yelirekim reopened T12232: Phortune "add payment method" erroring out on submission as "Open".

Just tried again.

Apr 17 2017, 2:34 PM · Phortune, Bug Report

Mar 29 2017

yelirekim added a comment to T11443: T-shirts with new logo.

I have many other excellent ideas for the Phacility online swag store, these are my favorite:

Mar 29 2017, 9:34 PM · Public & Media Relations

Mar 25 2017

Mnkras awarded T11443: T-shirts with new logo a Party Time token.
Mar 25 2017, 9:30 PM · Public & Media Relations
thoughtpolice awarded T11443: T-shirts with new logo a Love token.
Mar 25 2017, 8:15 PM · Public & Media Relations

Feb 16 2017

michaeljs1990 awarded T9525: Provide a reasonable general use logo a Like token.
Feb 16 2017, 10:05 PM · Public & Media Relations

Feb 10 2017

yelirekim assigned T12243: Repository cluster engine throws InvalidArgument exception when failing to acquire a write lock to jcox.
Feb 10 2017, 5:19 PM · Diffusion, Bug Report
yelirekim created T12243: Repository cluster engine throws InvalidArgument exception when failing to acquire a write lock.
Feb 10 2017, 5:19 PM · Diffusion, Bug Report

Feb 8 2017

yelirekim added a comment to T12232: Phortune "add payment method" erroring out on submission.

Yeah I clicked "Delete File" and can't see it anywhere now.

Feb 8 2017, 5:16 PM · Phortune, Bug Report
yelirekim added a comment to T12232: Phortune "add payment method" erroring out on submission.

Edited to not give out my credit card number on the internet.

Feb 8 2017, 5:12 PM · Phortune, Bug Report
yelirekim updated the task description for T12232: Phortune "add payment method" erroring out on submission.
Feb 8 2017, 5:11 PM · Phortune, Bug Report
yelirekim created T12232: Phortune "add payment method" erroring out on submission.
Feb 8 2017, 5:10 PM · Phortune, Bug Report
yelirekim added a comment to D17324: Add back the motivator panel.

FWIW, this is the thing that I point someone at when I want to show them how to contribute changes to Phabricator, like just go through the workflow.

Feb 8 2017, 4:53 PM
yelirekim added a comment to D17324: Add back the motivator panel.

Feb 8 2017, 4:43 PM

Feb 7 2017

yelirekim awarded D17233: Remove motivator panel a Heartbreak token.
Feb 7 2017, 8:01 PM

Jan 4 2017

yelirekim added auditors to rPef05bf335d8e: Allow Harbormaster builds to publish to a different object: jcox.

This might break our stuff, test async diff before deploying upstream updates.

Jan 4 2017, 10:38 PM
yelirekim updated subscribers of T9515: Build a "quorum" mechanism and associated UI.
Jan 4 2017, 6:04 PM · Restricted Project, Notifications, Phacility, Auth
yelirekim added a comment to T9515: Build a "quorum" mechanism and associated UI.

Jan 4 2017, 6:04 PM · Restricted Project, Notifications, Phacility, Auth

Dec 9 2016

yelirekim added a comment to D17018: Add a `title` field to the `maniphest.query` conduit method.

At worst we can just add another API method to our install, like, maniphest.reallysearchtitle

Dec 9 2016, 9:07 PM
yelirekim added a comment to D17018: Add a `title` field to the `maniphest.query` conduit method.

I think we are doing this all downstream.

Dec 9 2016, 9:06 PM
yelirekim added a comment to D17018: Add a `title` field to the `maniphest.query` conduit method.

I assume everyone is fully aware of the caveats and just being lazy and not wanting to store the identifier they get back when they create the task? I will inquire further.

Dec 9 2016, 8:46 PM
yelirekim added a comment to D17018: Add a `title` field to the `maniphest.query` conduit method.

Is Phabricator not a suitable replacement for a Jiffy Lube inventory terminal?

Dec 9 2016, 8:42 PM
yelirekim added a comment to D17018: Add a `title` field to the `maniphest.query` conduit method.

Because people are filing tasks from external systems in some systematic way (consistent titles) not saving the ID, and then wanting to reference the task later.

Dec 9 2016, 8:39 PM

Dec 5 2016

yelirekim added a comment to T11951: Make conduit more secure.

This seems like a good plan to me, we're just checking in that this isn't totally outside the realm of possibility while we plan some super secret changes to our instance.

Dec 5 2016, 6:33 PM · Feature Request

Nov 23 2016

yelirekim awarded D16857: Rebuild "arc branch" on new "hardpoint" infrastructure a Party Time token.
Nov 23 2016, 1:20 AM

Nov 22 2016

yelirekim awarded E1216: Errands a Baby Tequila token.
Nov 22 2016, 10:04 PM

Nov 16 2016

yelirekim awarded D16876: Support application partitioning across multiple masters a Grey Medal token.
Nov 16 2016, 1:23 AM

Nov 10 2016

yelirekim awarded T11845: Media with "autoplay" should possibly not autoplay in news feed a Party Time token.
Nov 10 2016, 7:51 PM · Prioritized, Remarkup

Oct 13 2016

yelirekim committed rP8247edff98e8: Modularize Owners package transactions (authored by yelirekim).
Modularize Owners package transactions
Oct 13 2016, 9:07 PM
yelirekim closed D16651: Modularize Owners package transactions by committing rP8247edff98e8: Modularize Owners package transactions.
Oct 13 2016, 9:07 PM
yelirekim updated the diff for D16651: Modularize Owners package transactions.

use array() instead of inheriting validation errors

Oct 13 2016, 8:56 PM
yelirekim committed rP8759f7e6ec62: Expose Drydock blueprints via Conduit (authored by yelirekim).
Expose Drydock blueprints via Conduit
Oct 13 2016, 8:48 PM
yelirekim closed D16593: Expose Drydock blueprints via Conduit by committing rP8759f7e6ec62: Expose Drydock blueprints via Conduit.
Oct 13 2016, 8:48 PM
yelirekim added a commit to T11694: Allow clients to generally reason about Drydock leases over the API: rP8759f7e6ec62: Expose Drydock blueprints via Conduit.
Oct 13 2016, 8:48 PM · Restricted Project, Drydock
yelirekim updated the diff for D16593: Expose Drydock blueprints via Conduit.

remove class from conduit fields

Oct 13 2016, 8:47 PM

Oct 11 2016

yelirekim committed rPfa90f8bef413: Expose Drydock authorizations via Conduit (authored by yelirekim).
Expose Drydock authorizations via Conduit
Oct 11 2016, 7:55 PM
yelirekim added a commit to T11694: Allow clients to generally reason about Drydock leases over the API: rPfa90f8bef413: Expose Drydock authorizations via Conduit.
Oct 11 2016, 7:55 PM · Restricted Project, Drydock
yelirekim closed D16592: Expose Drydock authorizations via Conduit by committing rPfa90f8bef413: Expose Drydock authorizations via Conduit.
Oct 11 2016, 7:55 PM

Oct 7 2016

yelirekim added a comment to D16651: Modularize Owners package transactions.

This seems to work afaik. Still need to do custom fields and try this in conjunction.

Oct 7 2016, 7:21 AM
yelirekim retitled D16651: Modularize Owners package transactions from wip modular transactions for owners to Modularize Owners package transactions.
Oct 7 2016, 7:14 AM
yelirekim updated the diff for D16651: Modularize Owners package transactions.
  • Fix display of quoted transaction values
  • Remove unneeded empty checks for create transactions
  • Remove getRequiredHandlePHIDs and transactionHasEffect methods
  • Use generateOldValue instead of getOldValue when applying external effects
Oct 7 2016, 6:59 AM

Oct 3 2016

yelirekim added inline comments to D16651: Modularize Owners package transactions.
Oct 3 2016, 7:49 PM
yelirekim added a comment to D16651: Modularize Owners package transactions.

I still have stylistic stuff to do throughout, I just focused on faithfully moving functionality out of the editor/transaction into the modular transactions here. There are a few things that don't work, haven't even looked at custom fields yet.

Oct 3 2016, 7:16 PM
yelirekim updated the diff for D16651: Modularize Owners package transactions.

finishing the gruntwork

Oct 3 2016, 7:13 PM
yelirekim planned changes to D16651: Modularize Owners package transactions.
Oct 3 2016, 5:58 AM
yelirekim retitled D16651: Modularize Owners package transactions from to wip modular transactions for owners.
Oct 3 2016, 5:58 AM

Sep 28 2016

yelirekim awarded D16624: Remove a flaky test which can fail under load a Y So Serious token.
Sep 28 2016, 11:46 PM

Sep 26 2016

yelirekim awarded D16597: Remarkup rule to embed images a Pterodactyl token.
Sep 26 2016, 9:57 PM
yelirekim added inline comments to D16594: Expose Drydock leases via Conduit.
Sep 26 2016, 5:52 PM
yelirekim added a comment to T11693: Make drydock command interfaces accessible via SSH workflows.

(not building locally anymore, eg, any time I want to do anything with a built binary at all, i'm actually doing it with a binary on a remote server because I can get the build results faster)

Sep 26 2016, 4:44 PM · Restricted Project, Drydock
yelirekim added a comment to T11693: Make drydock command interfaces accessible via SSH workflows.

Well one facet of this is that remote servers tend to be significantly more powerful than developer machines, and so since I've implemented this downstream, I have basically taken to not building locally anymore.

Sep 26 2016, 4:43 PM · Restricted Project, Drydock

Sep 24 2016

yelirekim updated the diff for D16594: Expose Drydock leases via Conduit.

Don't yet presume that users will be lease owners

Sep 24 2016, 7:28 AM
yelirekim retitled D16594: Expose Drydock leases via Conduit from to Expose Drydock leases via Conduit.
Sep 24 2016, 7:19 AM
yelirekim added a revision to T11694: Allow clients to generally reason about Drydock leases over the API: D16594: Expose Drydock leases via Conduit.
Sep 24 2016, 7:19 AM · Restricted Project, Drydock
yelirekim updated D16593: Expose Drydock blueprints via Conduit.
Sep 24 2016, 7:09 AM
yelirekim added a revision to T11694: Allow clients to generally reason about Drydock leases over the API: D16593: Expose Drydock blueprints via Conduit.
Sep 24 2016, 7:09 AM · Restricted Project, Drydock
yelirekim retitled D16593: Expose Drydock blueprints via Conduit from to Expose Drydock blueprints via Conduit.
Sep 24 2016, 7:00 AM
yelirekim retitled D16592: Expose Drydock authorizations via Conduit from to Expose Drydock authorizations via Conduit.
Sep 24 2016, 6:49 AM
yelirekim added a revision to T11694: Allow clients to generally reason about Drydock leases over the API: D16592: Expose Drydock authorizations via Conduit.
Sep 24 2016, 6:49 AM · Restricted Project, Drydock
yelirekim abandoned D16565: Expose Drydock lease management from conduit.
Sep 24 2016, 5:42 AM
yelirekim updated the diff for D16565: Expose Drydock lease management from conduit.

Removing all of the *.search stuff and leaving the rest to make it easier on future diff archaeologists

Sep 24 2016, 5:41 AM
yelirekim renamed T11693: Make drydock command interfaces accessible via SSH workflows from Make drydock command interfaces accessible over SSH to Make drydock command interfaces accessible via SSH workflows.
Sep 24 2016, 5:17 AM · Restricted Project, Drydock
yelirekim abandoned D16565: Expose Drydock lease management from conduit.
Sep 24 2016, 5:16 AM
yelirekim added a comment to T11693: Make drydock command interfaces accessible via SSH workflows.

Our actual internal use case is as follows:

Sep 24 2016, 4:38 AM · Restricted Project, Drydock
yelirekim created T11694: Allow clients to generally reason about Drydock leases over the API.
Sep 24 2016, 4:16 AM · Restricted Project, Drydock
yelirekim created T11693: Make drydock command interfaces accessible via SSH workflows.
Sep 24 2016, 4:12 AM · Restricted Project, Drydock

Sep 21 2016

yelirekim updated subscribers of D16565: Expose Drydock lease management from conduit.

@epriestley / @chad do you have pending comments on this? I can split it up into "obvious improvements", like bundling together search engine / dao / API methods per-dao. And then leave drydock.createlease and drydock.destroylease in their own diffs since I imagine they are the most contentious.

Sep 21 2016, 3:14 AM

Sep 20 2016

yelirekim awarded rP25eea83bc05f: (stable) Stop doing an excessive amount of work in `diffusion.rawdiffquery` a Baby Tequila token.
Sep 20 2016, 4:59 PM

Sep 19 2016

yelirekim awarded D16573: Update Phurl to use EditEngine a Piece of Eight token.
Sep 19 2016, 8:42 PM
yelirekim added a comment to D16565: Expose Drydock lease management from conduit.

I may have misunderstood you, were you advocating for application capabilities instead of authorizations?

Sep 19 2016, 8:07 PM
yelirekim added a comment to D16573: Update Phurl to use EditEngine.

Edit engine is deceitfully magical, and you may have indeed done everything you needed to here.

Sep 19 2016, 6:12 PM
yelirekim added inline comments to D16566: Manage Drydock leases from arcanist.
Sep 19 2016, 5:22 PM
yelirekim accepted D16569: Added a 'name' field to the results for harbormaster.build.search endpoint.
Sep 19 2016, 12:34 PM

Sep 18 2016

yelirekim added a comment to D16565: Expose Drydock lease management from conduit.

From my perspective, it makes sense for it to be the user requesting authorization, for our use cases a user (or a bot user in the case of other services consuming this) should be the "thing" that is requesting authorization.

Sep 18 2016, 3:40 AM
hach-que awarded D16566: Manage Drydock leases from arcanist a Love token.
Sep 18 2016, 3:33 AM
hach-que awarded D16565: Expose Drydock lease management from conduit a Love token.
Sep 18 2016, 3:32 AM

Sep 17 2016

yelirekim added a comment to D16565: Expose Drydock lease management from conduit.

@epriestley couple thoughts:

Sep 17 2016, 9:24 PM
yelirekim updated the diff for D16566: Manage Drydock leases from arcanist.

fix lint problem

Sep 17 2016, 1:58 AM
yelirekim added a dependent revision for D16565: Expose Drydock lease management from conduit: D16566: Manage Drydock leases from arcanist.
Sep 17 2016, 1:57 AM
yelirekim retitled D16566: Manage Drydock leases from arcanist from to Manage Drydock leases from arcanist.
Sep 17 2016, 1:57 AM
yelirekim retitled D16565: Expose Drydock lease management from conduit from to Expose Drydock lease management from conduit.
Sep 17 2016, 1:13 AM

Sep 12 2016

yelirekim added a comment to Q489: What awful and unforetold security problems am I unleashing by allowing users to proxy arbitrary commands to drydock machines? (Answer 457).

The bash workflow is just loaded the same way all of the other ssh workflows are loaded, I think that the problem is one of security policy / segmentation.

Sep 12 2016, 9:13 PM
yelirekim added a comment to Q489: What awful and unforetold security problems am I unleashing by allowing users to proxy arbitrary commands to drydock machines? (Answer 457).

Maybe make it so that we can host an arbitrary number of ssh daemons on different ports with different entry scripts? I suppose we can kinda do that already by just copying the existing script, minus the part where we have different groups of ssh keys, but could probably sub in passphrase as the backing for authentication there.

Sep 12 2016, 8:42 PM
yelirekim updated subscribers of Q489: What awful and unforetold security problems am I unleashing by allowing users to proxy arbitrary commands to drydock machines? (Answer 457).

Oh nice, I was expecting the "this is awful" upstream response for this :) I can put together use cases and either @jcox or I can work on upstreaming it.

Sep 12 2016, 8:37 PM
yelirekim updated Q489: What awful and unforetold security problems am I unleashing by allowing users to proxy arbitrary commands to drydock machines? from to What awful and unforetold security problems am I unleashing by allowing users to proxy arbitrary commands to drydock machines?.
Sep 12 2016, 7:05 PM · Drydock

Sep 2 2016

michaeljs1990 awarded D16485: Provide a workflow to restart Harbormaster builds a Dat Boi token.
Sep 2 2016, 4:10 PM
yelirekim committed rP403073c989b4: Provide a workflow to restart Harbormaster builds (authored by yelirekim).
Provide a workflow to restart Harbormaster builds
Sep 2 2016, 1:31 PM
yelirekim closed D16485: Provide a workflow to restart Harbormaster builds by committing rP403073c989b4: Provide a workflow to restart Harbormaster builds.
Sep 2 2016, 1:31 PM
yelirekim added a commit to T10867: Version daemons more clearly in daemon console so it's clear when `phd reload` has taken effect: rP403073c989b4: Provide a workflow to restart Harbormaster builds.
Sep 2 2016, 1:31 PM · Daemons, Feature Request
yelirekim added a comment to D16485: Provide a workflow to restart Harbormaster builds.

In theory we could make it so that steps resume correctly, but in practice I have very little control over the contents of the scripts that get run. People tend to wrap all of the stuff up that their build is supposed to do into a single script, and assume they're starting fresh each time it's executed.

Sep 2 2016, 1:31 PM
yelirekim updated the diff for D16485: Provide a workflow to restart Harbormaster builds.

use pretty numbers when displaying build count

Sep 2 2016, 12:46 PM
yelirekim committed rPHUc8b76485ef84: Use new format for docblock specials when parsing symbols (authored by yelirekim).
Use new format for docblock specials when parsing symbols
Sep 2 2016, 12:05 PM
yelirekim closed T11575: Make PhutilDocblockParser always return arrays when parsing "@doc-tags" as Resolved by committing rPHUc8b76485ef84: Use new format for docblock specials when parsing symbols.
Sep 2 2016, 12:05 PM · Bug Report
yelirekim closed D16484: Use new format for docblock specials when parsing symbols by committing rPHUc8b76485ef84: Use new format for docblock specials when parsing symbols.
Sep 2 2016, 12:05 PM
yelirekim updated D16485: Provide a workflow to restart Harbormaster builds.
Sep 2 2016, 6:10 AM