Page MenuHomeElementl

rexledesma (Rex Ledesma)
UserAdministrator

Projects

User does not belong to any projects.

User Details

User Since
Sep 21 2020, 5:44 PM (29 w, 5 d)
Roles
Administrator

Recent Activity

Fri, Apr 16

rexledesma added a comment to D7478: [Helm] Allow configuration of postgres connection string on storage classes.

@johann yeah I might have went about this a goofy way - it's probably more simpler to just expose a new field postgres_db.db_params in the storage config, and then just append the urlencoded values to the generated postgresql connection string

Fri, Apr 16, 7:45 PM
rexledesma planned changes to D7478: [Helm] Allow configuration of postgres connection string on storage classes.

More complicated than expected - will have to update the k8s run launchers to take in postgres_connection_string_secret rather than postgres_password_secret

Fri, Apr 16, 7:24 PM
rexledesma updated the diff for D7478: [Helm] Allow configuration of postgres connection string on storage classes.

clarify localhost

Fri, Apr 16, 5:52 PM
rexledesma updated the diff for D7478: [Helm] Allow configuration of postgres connection string on storage classes.

fix fixture import

Fri, Apr 16, 5:13 PM
rexledesma published D7478: [Helm] Allow configuration of postgres connection string on storage classes for review.
Fri, Apr 16, 4:59 PM
rexledesma added a comment to D7409: [RFC] [dagster-graphql] Python Client Implementation [1/3].

bateman

Fri, Apr 16, 4:41 PM
rexledesma accepted D7409: [RFC] [dagster-graphql] Python Client Implementation [1/3].
Fri, Apr 16, 4:41 PM

Thu, Apr 15

rexledesma closed D7441: Remove references to #contributors.
Thu, Apr 15, 12:34 AM
rexledesma committed R1:ad9fc9627f38: Remove references to #contributors (authored by rexledesma).
Remove references to #contributors
Thu, Apr 15, 12:34 AM
rexledesma closed D7439: Update slack bot release message.
Thu, Apr 15, 12:34 AM
rexledesma committed R1:addc7ea54108: Update slack bot release message (authored by rexledesma).
Update slack bot release message
Thu, Apr 15, 12:34 AM

Wed, Apr 14

rexledesma requested review of D7441: Remove references to #contributors.
Wed, Apr 14, 10:28 PM
rexledesma requested review of D7439: Update slack bot release message.
Wed, Apr 14, 10:10 PM
rexledesma requested changes to D7421: [dagster-graphql] Dagster GraphQL Python Client [2/3].

This script looks better placed in python_modules/automation since it's generating some persisted artifacts (the query snapshots) - it should probably be invoked as dagster-graphql-client.

Wed, Apr 14, 8:43 PM

Tue, Apr 13

rexledesma requested changes to D7409: [RFC] [dagster-graphql] Python Client Implementation [1/3].

Add to graphql documentation as well? there are directive you can use to make the classes and associated methods show up

Tue, Apr 13, 7:50 PM

Fri, Apr 9

rexledesma added inline comments to D7370: K8sRunLauncher make postgres optional.
Fri, Apr 9, 5:25 PM

Thu, Apr 8

rexledesma accepted D7269: Improve executor/IO manager error message.
Thu, Apr 8, 9:18 PM
rexledesma published D7269: Improve executor/IO manager error message for review.
Thu, Apr 8, 9:18 PM
rexledesma closed D7351: [1/2] Fix re-execution if execution plan is delegated.
Thu, Apr 8, 8:48 PM
rexledesma committed R1:ade122e29572: [1/2] Fix re-execution if execution plan is delegated (authored by rexledesma).
[1/2] Fix re-execution if execution plan is delegated
Thu, Apr 8, 8:48 PM
rexledesma abandoned D7348: [RFC] Add all executable steps to known execution state.

Dropping in favor of inspecting the pipeline run's step_keys_to_execute

Thu, Apr 8, 8:34 PM
rexledesma updated the diff for D7351: [1/2] Fix re-execution if execution plan is delegated.

god bless alex

Thu, Apr 8, 8:24 PM
rexledesma added inline comments to D7351: [1/2] Fix re-execution if execution plan is delegated.
Thu, Apr 8, 6:17 PM
rexledesma updated the diff for D7351: [1/2] Fix re-execution if execution plan is delegated.

comment

Thu, Apr 8, 6:17 PM
rexledesma updated the diff for D7351: [1/2] Fix re-execution if execution plan is delegated.

fix tests

Thu, Apr 8, 6:15 PM
rexledesma published D7351: [1/2] Fix re-execution if execution plan is delegated for review.
Thu, Apr 8, 5:50 PM
rexledesma updated the test plan for D7348: [RFC] Add all executable steps to known execution state.
Thu, Apr 8, 5:20 AM
rexledesma updated the diff for D7348: [RFC] Add all executable steps to known execution state.

add tests

Thu, Apr 8, 5:20 AM
rexledesma requested review of D7348: [RFC] Add all executable steps to known execution state.
Thu, Apr 8, 4:27 AM

Fri, Apr 2

rexledesma closed D7272: Update Helm for 0.11.3.
Fri, Apr 2, 4:36 PM
rexledesma closed D7257: Merge Helm repo index when packaging release.
Fri, Apr 2, 4:36 PM
rexledesma committed R1:bb6242f13df6: Update Helm for 0.11.3 (authored by rexledesma).
Update Helm for 0.11.3
Fri, Apr 2, 4:36 PM
rexledesma committed R1:11473be76308: Merge Helm repo index when packaging release (authored by rexledesma).
Merge Helm repo index when packaging release
Fri, Apr 2, 4:36 PM
rexledesma published D7257: Merge Helm repo index when packaging release for review.
Fri, Apr 2, 4:20 PM

Thu, Apr 1

rexledesma requested review of D7272: Update Helm for 0.11.3.
Thu, Apr 1, 11:28 PM
rexledesma closed D7270: Add prelease suffix for Helm kubeVersion.
Thu, Apr 1, 10:57 PM
rexledesma closed D7267: Ensure Chart.lock is in sync.
Thu, Apr 1, 10:57 PM
rexledesma committed R1:b6fc3d14139a: Add prelease suffix for Helm kubeVersion (authored by rexledesma).
Add prelease suffix for Helm kubeVersion
Thu, Apr 1, 10:57 PM
rexledesma committed R1:5e3564223988: Ensure Chart.lock is in sync (authored by rexledesma).
Ensure Chart.lock is in sync
Thu, Apr 1, 10:57 PM
rexledesma requested review of D7270: Add prelease suffix for Helm kubeVersion.
Thu, Apr 1, 10:56 PM
rexledesma added a comment to D7267: Ensure Chart.lock is in sync.

^ flake

Thu, Apr 1, 10:53 PM
rexledesma requested review of D7267: Ensure Chart.lock is in sync.
Thu, Apr 1, 10:53 PM
rexledesma closed D7268: Fix integration tests.
Thu, Apr 1, 9:24 PM
rexledesma committed R1:090875b8e329: Fix integration tests (authored by rexledesma).
Fix integration tests
Thu, Apr 1, 9:23 PM
rexledesma published D7268: Fix integration tests for review.
Thu, Apr 1, 9:17 PM
rexledesma closed D7265: Update chart lock.
Thu, Apr 1, 8:59 PM
rexledesma committed R1:446d022b1b63: Update chart lock (authored by rexledesma).
Update chart lock
Thu, Apr 1, 8:59 PM
rexledesma published D7265: Update chart lock for review.
Thu, Apr 1, 8:58 PM
rexledesma closed D7255: Update Chart.yaml for Artifacthub.
Thu, Apr 1, 8:41 PM
rexledesma committed R1:316dcaf3526d: Update Chart.yaml for Artifacthub (authored by rexledesma).
Update Chart.yaml for Artifacthub
Thu, Apr 1, 8:41 PM
rexledesma updated the diff for D7255: Update Chart.yaml for Artifacthub.

rebase

Thu, Apr 1, 8:40 PM
rexledesma closed D7253: [helm] Allow creation of workspace from dagit values.
Thu, Apr 1, 8:40 PM
rexledesma committed R1:7be48957aa4f: [helm] Allow creation of workspace from dagit values (authored by rexledesma).
[helm] Allow creation of workspace from dagit values
Thu, Apr 1, 8:40 PM
rexledesma closed D7254: Mark Helm values schema as generated.
Thu, Apr 1, 8:40 PM
rexledesma closed D7252: Reorganize helm user deployment template tests.
Thu, Apr 1, 8:40 PM
rexledesma committed R1:5b317e4ffae1: Reorganize helm user deployment template tests (authored by rexledesma).
Reorganize helm user deployment template tests
Thu, Apr 1, 8:39 PM
rexledesma committed R1:8da397e87e1d: Mark Helm values schema as generated (authored by rexledesma).
Mark Helm values schema as generated
Thu, Apr 1, 8:39 PM
rexledesma added a comment to D7253: [helm] Allow creation of workspace from dagit values.

I think from the introductory line in https://docs.dagster.io/concepts/repositories-workspaces/workspaces#workspaces, it should make sense that workspace lives under dagit.

Thu, Apr 1, 7:48 PM
rexledesma updated the diff for D7253: [helm] Allow creation of workspace from dagit values.

clarifying comment on why dagit needs a workspace

Thu, Apr 1, 7:47 PM
rexledesma added a comment to D7253: [helm] Allow creation of workspace from dagit values.

@johann I'm cautious around putting this under dagster-user-deployments since that would generate a manifest used by the parent chart, using values from the subchart. That's a bit confusing. We already do that with dagster-user-deployments.enabled, which is not good.

Thu, Apr 1, 7:29 PM
rexledesma updated the diff for D7253: [helm] Allow creation of workspace from dagit values.

up

Thu, Apr 1, 7:21 PM
rexledesma updated the diff for D7253: [helm] Allow creation of workspace from dagit values.

+johann

Thu, Apr 1, 7:18 PM
rexledesma requested review of D7255: Update Chart.yaml for Artifacthub.
Thu, Apr 1, 5:11 AM
rexledesma published D7254: Mark Helm values schema as generated for review.
Thu, Apr 1, 1:12 AM
rexledesma requested review of D7253: [helm] Allow creation of workspace from dagit values.
Thu, Apr 1, 12:56 AM
rexledesma requested review of D7252: Reorganize helm user deployment template tests.
Thu, Apr 1, 12:55 AM

Wed, Mar 31

rexledesma closed D7249: Sort docs versions by latest.
Wed, Mar 31, 10:10 PM
rexledesma committed R1:ef27a0366ac4: Sort docs versions by latest (authored by rexledesma).
Sort docs versions by latest
Wed, Mar 31, 10:10 PM
rexledesma updated the diff for D7249: Sort docs versions by latest.

+yuhan

Wed, Mar 31, 10:10 PM
rexledesma published D7249: Sort docs versions by latest for review.
Wed, Mar 31, 9:47 PM

Mon, Mar 29

rexledesma closed D7076: Add check for RabbitMQ in Celery deployment.
Mon, Mar 29, 10:48 PM
rexledesma committed R1:45549362bbc6: Add check for RabbitMQ in Celery deployment (authored by rexledesma).
Add check for RabbitMQ in Celery deployment
Mon, Mar 29, 10:48 PM
rexledesma closed D7195: Add test for rendering postgresql secret.
Mon, Mar 29, 10:47 PM
rexledesma committed R1:4e5e930085e8: Add test for rendering postgresql secret (authored by rexledesma).
Add test for rendering postgresql secret
Mon, Mar 29, 10:47 PM
rexledesma retitled D7076: Add check for RabbitMQ in Celery deployment from Fix integration test flakes to Add check for RabbitMQ in Celery deployment.
Mon, Mar 29, 10:45 PM
rexledesma abandoned D7192: Pin ipykernel.
Mon, Mar 29, 10:23 PM
rexledesma updated the diff for D7192: Pin ipykernel.

back to !=

Mon, Mar 29, 9:53 PM
rexledesma closed D7187: Add twitter logo to docs site.
Mon, Mar 29, 8:19 PM
rexledesma committed R1:88ae6058e594: Add twitter logo to docs site (authored by rexledesma).
Add twitter logo to docs site
Mon, Mar 29, 8:19 PM
rexledesma updated the diff for D7187: Add twitter logo to docs site.

update layout

Mon, Mar 29, 8:18 PM
rexledesma closed D7184: Add development instructions to Helm chart.
Mon, Mar 29, 8:06 PM
rexledesma committed R1:f985b491404c: Add development instructions to Helm chart (authored by rexledesma).
Add development instructions to Helm chart
Mon, Mar 29, 8:06 PM
rexledesma updated the diff for D7184: Add development instructions to Helm chart.

developing -> contributing

Mon, Mar 29, 8:05 PM
rexledesma published D7187: Add twitter logo to docs site for review.
Mon, Mar 29, 8:01 PM
rexledesma updated the diff for D7076: Add check for RabbitMQ in Celery deployment.

test 8

Mon, Mar 29, 7:59 PM
rexledesma updated the diff for D7076: Add check for RabbitMQ in Celery deployment.

test 7

Mon, Mar 29, 7:58 PM
rexledesma updated the diff for D7076: Add check for RabbitMQ in Celery deployment.

test 6

Mon, Mar 29, 7:58 PM
rexledesma updated the diff for D7076: Add check for RabbitMQ in Celery deployment.

test 5

Mon, Mar 29, 7:58 PM
rexledesma updated the diff for D7076: Add check for RabbitMQ in Celery deployment.

test 4

Mon, Mar 29, 7:58 PM
rexledesma updated the diff for D7076: Add check for RabbitMQ in Celery deployment.

test 3

Mon, Mar 29, 7:58 PM
rexledesma updated the diff for D7076: Add check for RabbitMQ in Celery deployment.

test 2

Mon, Mar 29, 7:58 PM
rexledesma updated the diff for D7076: Add check for RabbitMQ in Celery deployment.

update rabbitmq hostname

Mon, Mar 29, 7:26 PM
rexledesma published D7184: Add development instructions to Helm chart for review.
Mon, Mar 29, 7:14 PM
rexledesma updated the diff for D7076: Add check for RabbitMQ in Celery deployment.

+alex

Mon, Mar 29, 5:47 PM
rexledesma accepted D7176: refine pyyaml req.
Mon, Mar 29, 3:53 PM

Fri, Mar 26

rexledesma published D7076: Add check for RabbitMQ in Celery deployment for review.
Fri, Mar 26, 9:34 PM
rexledesma closed D7154: Update changelog 0.11.2.
Fri, Mar 26, 2:46 AM
rexledesma committed R1:b56d827d8f6e: Update changelog 0.11.2 (authored by rexledesma).
Update changelog 0.11.2
Fri, Mar 26, 2:46 AM
rexledesma committed R1:e8a72818646e: 0.11.2 (authored by rexledesma).
0.11.2
Fri, Mar 26, 2:06 AM
rexledesma committed R1:b7b750afa7d5: 0.11.2.pre0 (authored by rexledesma).
0.11.2.pre0
Fri, Mar 26, 1:38 AM