Skip to content
This repository was archived by the owner on Mar 10, 2025. It is now read-only.

Commit a100419

Browse files
authored
delete telemetry (#129)
* removes telemetry * update * lint * update * fix
1 parent 06e6b1e commit a100419

File tree

25 files changed

+40
-67
lines changed

25 files changed

+40
-67
lines changed

.github/workflows/ci-mac.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ jobs:
1212
runs-on: macos-latest
1313
strategy:
1414
matrix:
15-
python-version: [3.7, 3.8, 3.9]
15+
python-version: [3.9]
1616

1717
steps:
1818
- uses: actions/checkout@v2

.github/workflows/ci-windows.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ jobs:
1212
runs-on: windows-latest
1313
strategy:
1414
matrix:
15-
python-version: [3.7, 3.8, 3.9]
15+
python-version: [3.9]
1616

1717
steps:
1818
- uses: actions/checkout@v2

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ jobs:
1313
runs-on: ubuntu-latest
1414
strategy:
1515
matrix:
16-
python-version: [3.7, 3.8, 3.9]
16+
python-version: [3.9]
1717

1818
steps:
1919
- uses: actions/checkout@v2

src/soopervisor/abc.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
"""
22
Abstract classes that define the protocol for all exporters
33
"""
4+
45
import click
56
import abc
67
from pathlib import Path

src/soopervisor/airflow/export.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
"""
22
Export a Ploomber DAG to Airflow
33
"""
4+
45
import json
56
import os
67
from pathlib import Path

src/soopervisor/argo/export.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
"""
22
Export to Argo Workflows
33
"""
4+
45
from pathlib import Path
56

67
import click

src/soopervisor/aws/batch.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
"""
22
Running pipelines on AWS Batch
33
"""
4+
45
import re
56
from uuid import uuid4
67
import os

src/soopervisor/aws/lambda_.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
"""
22
Online DAG deployment using AWS Lambda
33
"""
4+
45
from pathlib import Path
56
from ploomber.util import default
67
from ploomber.io._commander import Commander

src/soopervisor/cli.py

Lines changed: 17 additions & 58 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
from soopervisor import config
77
from soopervisor import exporter
88
from soopervisor.enum import Backend, Mode
9-
from soopervisor.telemetry import telemetry
109

1110

1211
@click.group()
@@ -39,34 +38,22 @@ def add(env_name, backend, preset):
3938
"""
4039
backend = Backend(backend)
4140

42-
try:
43-
if Path("soopervisor.yaml").exists():
44-
cfg = yaml.safe_load(Path("soopervisor.yaml").read_text())
45-
if env_name in cfg:
46-
raise click.ClickException(
47-
f"A {env_name!r} section in the "
48-
"soopervisor.yaml configuration file "
49-
"already exists. Choose another name."
50-
)
51-
if Path(env_name).exists():
41+
if Path("soopervisor.yaml").exists():
42+
cfg = yaml.safe_load(Path("soopervisor.yaml").read_text())
43+
if env_name in cfg:
5244
raise click.ClickException(
53-
f"{env_name!r} already exists. Select a different name."
45+
f"A {env_name!r} section in the "
46+
"soopervisor.yaml configuration file "
47+
"already exists. Choose another name."
5448
)
55-
except Exception as e:
56-
telemetry.log_api(
57-
"soopervisor_add_error",
58-
metadata={"type": backend.value, "env_name": env_name, "error": str(e)},
49+
if Path(env_name).exists():
50+
raise click.ClickException(
51+
f"{env_name!r} already exists. Select a different name."
5952
)
60-
raise
6153

6254
Exporter = exporter.for_backend(backend)
6355
Exporter.new("soopervisor.yaml", env_name=env_name, preset=preset).add()
6456

65-
telemetry.log_api(
66-
"soopervisor_add_success",
67-
metadata={"type": backend.value, "env_name": env_name},
68-
)
69-
7057
click.echo(
7158
"Environment added, to export it:\n"
7259
f"\t $ soopervisor export {env_name}\n"
@@ -100,23 +87,8 @@ def export(
10087
"""
10188
Export a target platform for execution/deployment
10289
"""
103-
input_args = {
104-
"env_name": env_name,
105-
"until_build": until_build,
106-
"mode": mode,
107-
"skip_tests": skip_tests,
108-
"skip_docker": skip_docker,
109-
"ignore_git": ignore_git,
110-
"task": task,
111-
}
112-
11390
backend = Backend(config.get_backend(env_name))
11491

115-
telemetry.log_api(
116-
"soopervisor_export_started",
117-
metadata={"type": backend.value, "input_args": input_args},
118-
)
119-
12092
until = None
12193

12294
if until_build:
@@ -129,27 +101,14 @@ def export(
129101

130102
Exporter = exporter.for_backend(backend)
131103

132-
try:
133-
Exporter.load("soopervisor.yaml", env_name=env_name, lazy_import=lazy).export(
134-
mode=mode,
135-
until=until,
136-
skip_tests=skip_tests,
137-
skip_docker=skip_docker,
138-
ignore_git=ignore_git,
139-
lazy_import=lazy,
140-
task_name=task,
141-
)
142-
143-
except Exception as e:
144-
telemetry.log_api(
145-
"soopervisor_export_error",
146-
metadata={"type": backend.value, "input_args": input_args, "error": str(e)},
147-
)
148-
raise
149-
150-
telemetry.log_api(
151-
"soopervisor_export_success",
152-
metadata={"type": backend.value, "input_args": input_args},
104+
Exporter.load("soopervisor.yaml", env_name=env_name, lazy_import=lazy).export(
105+
mode=mode,
106+
until=until,
107+
skip_tests=skip_tests,
108+
skip_docker=skip_docker,
109+
ignore_git=ignore_git,
110+
lazy_import=lazy,
111+
task_name=task,
153112
)
154113

155114

src/soopervisor/commons/dag.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
"""
22
Loading dags
33
"""
4+
45
from pathlib import Path
56

67
from ploomber.constants import TaskStatus

0 commit comments

Comments
 (0)