reviews  

Messages by Thread

• • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Aurora ReviewBot
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Joshua Cohen
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Steve Niemitz
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Aurora ReviewBot
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Steve Niemitz
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Aurora ReviewBot
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Bill Farner
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Steve Niemitz
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Bill Farner
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Steve Niemitz
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Bill Farner
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Steve Niemitz
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Aurora ReviewBot
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Steve Niemitz
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Aurora ReviewBot
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Steve Niemitz
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Aurora ReviewBot
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Steve Niemitz
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Aurora ReviewBot
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Steve Niemitz
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Aurora ReviewBot
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Bill Farner
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Steve Niemitz
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Steve Niemitz
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Aurora ReviewBot
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Steve Niemitz
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Steve Niemitz
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Aurora ReviewBot
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Bill Farner
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Bill Farner
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Aurora ReviewBot
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Bill Farner
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Bill Farner
  • Re: Review Request 31338: Added a command line flag, -global_container_mounts, to allow mounting paths from the slaves into the (docker) containers they run. Aurora ReviewBot
• Review Request 31334: Fixing cron update quota checking. Maxim Khutornenko
  • Re: Review Request 31334: Fixing cron update quota checking. Aurora ReviewBot
  • Re: Review Request 31334: Fixing cron update quota checking. Bill Farner
  • Re: Review Request 31334: Fixing cron update quota checking. Maxim Khutornenko
  • Re: Review Request 31334: Fixing cron update quota checking. Maxim Khutornenko
  • Re: Review Request 31334: Fixing cron update quota checking. Bill Farner
  • Re: Review Request 31334: Fixing cron update quota checking. Aurora ReviewBot
  • Re: Review Request 31334: Fixing cron update quota checking. Kevin Sweeney
• Review Request 31289: Enabling bash -e for vagrant runs Thorhallur Sverrisson
  • Re: Review Request 31289: Enabling bash -e for vagrant runs Bill Farner
  • Re: Review Request 31289: Enabling bash -e for vagrant runs Aurora ReviewBot
• Review Request 31251: Fix swallowed exceptions in health check test. Brian Wickman
  • Re: Review Request 31251: Fix swallowed exceptions in health check test. Aurora ReviewBot
  • Re: Review Request 31251: Fix swallowed exceptions in health check test. Brian Wickman
  • Re: Review Request 31251: Fix swallowed exceptions in health check test. Brian Wickman
  • Re: Review Request 31251: Fix swallowed exceptions in health check test. Zameer Manji
  • Re: Review Request 31251: Fix swallowed exceptions in health check test. Zameer Manji
  • Re: Review Request 31251: Fix swallowed exceptions in health check test. Brian Wickman
  • Re: Review Request 31251: Fix swallowed exceptions in health check test. Joe Smith
  • Re: Review Request 31251: Fix swallowed exceptions in health check test. Aurora ReviewBot
  • Re: Review Request 31251: Fix swallowed exceptions in health check test. Brian Wickman
  • Re: Review Request 31251: Fix swallowed exceptions in health check test. Zameer Manji
• Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Maxim Khutornenko
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Maxim Khutornenko
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Maxim Khutornenko
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Maxim Khutornenko
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Joshua Cohen
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Joshua Cohen
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. David McLaughlin
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Joshua Cohen
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Aurora ReviewBot
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Joshua Cohen
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Maxim Khutornenko
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Joshua Cohen
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Aurora ReviewBot
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Bill Farner
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Joshua Cohen
  • Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. Aurora ReviewBot
• Review Request 31241: Pushing transactions up in QuotaManager. Maxim Khutornenko
  • Re: Review Request 31241: Pushing transactions up in QuotaManager. Bill Farner
  • Re: Review Request 31241: Pushing transactions up in QuotaManager. Maxim Khutornenko
  • Re: Review Request 31241: Pushing transactions up in QuotaManager. Maxim Khutornenko
  • Re: Review Request 31241: Pushing transactions up in QuotaManager. Bill Farner
  • Re: Review Request 31241: Pushing transactions up in QuotaManager. Aurora ReviewBot
  • Re: Review Request 31241: Pushing transactions up in QuotaManager. Maxim Khutornenko
  • Re: Review Request 31241: Pushing transactions up in QuotaManager. Aurora ReviewBot
• Review Request 31240: Remove render partial job key method. Zameer Manji
  • Re: Review Request 31240: Remove single caller methods from AuroraCommandContext Zameer Manji
  • Re: Review Request 31240: Remove single caller methods from AuroraCommandContext Bill Farner
  • Re: Review Request 31240: Remove single caller methods from AuroraCommandContext Maxim Khutornenko
• Review Request 31235: Refactoring CronJobManager interface. Maxim Khutornenko
  • Re: Review Request 31235: Refactoring CronJobManager interface. Aurora ReviewBot
  • Re: Review Request 31235: Refactoring CronJobManager interface. Maxim Khutornenko
  • Re: Review Request 31235: Refactoring CronJobManager interface. Bill Farner
  • Re: Review Request 31235: Refactoring CronJobManager interface. Maxim Khutornenko
  • Re: Review Request 31235: Refactoring CronJobManager interface. Aurora ReviewBot
  • Re: Review Request 31235: Refactoring CronJobManager interface. Maxim Khutornenko
  • Re: Review Request 31235: Refactoring CronJobManager interface. Aurora ReviewBot
  • Re: Review Request 31235: Refactoring CronJobManager interface. Kevin Sweeney
  • Re: Review Request 31235: Refactoring CronJobManager interface. Maxim Khutornenko
  • Re: Review Request 31235: Refactoring CronJobManager interface. Kevin Sweeney
• Review Request 31176: Split out ReadOnlySchedulerImplTest. Kevin Sweeney
  • Re: Review Request 31176: Split out ReadOnlySchedulerImplTest. Aurora ReviewBot
  • Re: Review Request 31176: Split out ReadOnlySchedulerImplTest. Kevin Sweeney
  • Re: Review Request 31176: Split out ReadOnlySchedulerImplTest. Kevin Sweeney
  • Re: Review Request 31176: Split out ReadOnlySchedulerImplTest. Kevin Sweeney
  • Re: Review Request 31176: Split out ReadOnlySchedulerImplTest. Bill Farner
  • Re: Review Request 31176: Split out ReadOnlySchedulerImplTest. Kevin Sweeney
  • Re: Review Request 31176: Split out ReadOnlySchedulerImplTest. Aurora ReviewBot
  • Re: Review Request 31176: Split out ReadOnlySchedulerImplTest. Kevin Sweeney
  • Re: Review Request 31176: Split out ReadOnlySchedulerImplTest. Aurora ReviewBot
• Review Request 31171: Saving backups asynchronously. Maxim Khutornenko
  • Re: Review Request 31171: Saving backups asynchronously. Aurora ReviewBot
  • Re: Review Request 31171: Saving backups asynchronously. Kevin Sweeney
  • Re: Review Request 31171: Saving backups asynchronously. Bill Farner
  • Re: Review Request 31171: Saving backups asynchronously. Maxim Khutornenko
  • Re: Review Request 31171: Saving backups asynchronously. Maxim Khutornenko
  • Re: Review Request 31171: Saving backups asynchronously. Bill Farner
  • Re: Review Request 31171: Saving backups asynchronously. Bill Farner
  • Re: Review Request 31171: Saving backups asynchronously. Maxim Khutornenko
  • Re: Review Request 31171: Saving backups asynchronously. Aurora ReviewBot
  • Re: Review Request 31171: Saving backups asynchronously. Joshua Cohen
  • Re: Review Request 31171: Saving backups asynchronously. Bill Farner
  • Re: Review Request 31171: Saving backups asynchronously. Maxim Khutornenko
  • Re: Review Request 31171: Saving backups asynchronously. Bill Farner
  • Re: Review Request 31171: Saving backups asynchronously. Maxim Khutornenko
  • Re: Review Request 31171: Saving backups asynchronously. Joshua Cohen
  • Re: Review Request 31171: Saving backups asynchronously. Bill Farner
  • Re: Review Request 31171: Saving backups asynchronously. Aurora ReviewBot
• Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Bill Farner
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Aurora ReviewBot
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Zameer Manji
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Bill Farner
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Maxim Khutornenko
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Bill Farner
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Zameer Manji
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Bill Farner
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Zameer Manji
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Bill Farner
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Aurora ReviewBot
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Zameer Manji
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Zameer Manji
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Maxim Khutornenko
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Bill Farner
  • Re: Review Request 31170: Refactor existing write APIs for job updates to use IJobUpdateKey. Bill Farner
• Review Request 31165: Add lifecycle endpoints to Thermos Observer Joe Smith
  • Re: Review Request 31165: Add lifecycle endpoints to Thermos Observer Aurora ReviewBot
• Review Request 31144: Renaming OfferQueue into OfferManager. Maxim Khutornenko
  • Re: Review Request 31144: Renaming OfferQueue into OfferManager. Kevin Sweeney
  • Re: Review Request 31144: Renaming OfferQueue into OfferManager. Bill Farner
  • Re: Review Request 31144: Renaming OfferQueue into OfferManager. Aurora ReviewBot
• Review Request 31138: Add ability to pass configurable options to pytest. Joshua Cohen
  • Re: Review Request 31138: Add ability to pass configurable options to pytest. Joshua Cohen
  • Re: Review Request 31138: Add ability to pass configurable options to pytest. Bill Farner
  • Re: Review Request 31138: Add ability to pass configurable options to pytest. Brian Wickman
• Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Bill Farner
  • Re: Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Kevin Sweeney
  • Re: Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Bill Farner
  • Re: Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Bill Farner
  • Re: Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Aurora ReviewBot
  • Re: Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Bill Farner
  • Re: Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Kevin Sweeney
  • Re: Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Maxim Khutornenko
  • Re: Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Bill Farner
  • Re: Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Bill Farner
  • Re: Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Maxim Khutornenko
  • Re: Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Bill Farner
  • Re: Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Bill Farner
  • Re: Review Request 31136: Uniquely identify job updates in the database by JobUpdateKey. Aurora ReviewBot
• Review Request 31137: Update aurora to commons 0.3.3, unflake tests using ThreadedClock Brian Wickman
  • Re: Review Request 31137: Update aurora to commons 0.3.3, unflake tests using ThreadedClock Joe Smith
  • Re: Review Request 31137: Update aurora to commons 0.3.3, unflake tests using ThreadedClock Joshua Cohen
  • Re: Review Request 31137: Update aurora to commons 0.3.3, unflake tests using ThreadedClock Aurora ReviewBot
• Review Request 31135: Pin protobuf dependency to 2.6.1 Zameer Manji
  • Re: Review Request 31135: Pin protobuf dependency to 2.6.1 Kevin Sweeney
  • Re: Review Request 31135: Pin protobuf dependency to 2.6.1 Joe Smith
• Review Request 31124: Fix build break - remove unused imports. Bill Farner
  • Re: Review Request 31124: Fix build break - remove unused imports. Zameer Manji
  • Re: Review Request 31124: Fix build break - remove unused imports. Brian Wickman
• Review Request 31123: Enable checkstyle indentation check, fix violations. Bill Farner
  • Re: Review Request 31123: Enable checkstyle indentation check, fix violations. Zameer Manji
  • Re: Review Request 31123: Enable checkstyle indentation check, fix violations. Aurora ReviewBot
  • Re: Review Request 31123: Enable checkstyle indentation check, fix violations. Kevin Sweeney
  • Re: Review Request 31123: Enable checkstyle indentation check, fix violations. Joshua Cohen
  • Re: Review Request 31123: Enable checkstyle indentation check, fix violations. Bill Farner
  • Re: Review Request 31123: Enable checkstyle indentation check, fix violations. Joshua Cohen
  • Re: Review Request 31123: Enable checkstyle indentation check, fix violations. Bill Farner
• Review Request 31104: task-executor: TASK_RUNNING after first health check Moses Nakamura
  • Re: Review Request 31104: task-executor: TASK_RUNNING after first health check Aurora ReviewBot
  • Re: Review Request 31104: task-executor: TASK_RUNNING after first health check Zameer Manji
  • Re: Review Request 31104: task-executor: TASK_RUNNING after first health check Bill Farner
  • Re: Review Request 31104: task-executor: TASK_RUNNING after first health check Moses Nakamura

• Earlier messages
• Later messages