Skip to content

Commit 994fc7d

Browse files
committed
Merge branch 'main' into 'enterprise'
Sync open source commits See merge request dkinternal/observability/dataops-observability!18
2 parents 1338435 + 079843a commit 994fc7d

31 files changed

+64
-69
lines changed

CONTRIBUTING.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ You can submit a proposal for a new feature by creating a [GitHub Issue](https:/
1212

1313
## Join our Community
1414

15-
Join our [Slack community](https://data-observability.slack.com) for discussions with other Data Observability users.
15+
Join our [Slack community](https://data-observability-slack.datakitchen.io/join) for discussions with other Data Observability users.
1616

1717
## Code Submission
1818

CONTRIBUTORS.md

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# Contributors
2+
3+
Thanks to everyone who has contributed to DataOps Observability!
4+
5+
💻 = code, 🚇 = infrastructure, 📖 = documentation, 📆 = project management, 🤔 = product management, 💬 = user support
6+
7+
- [Samantha Hamilton](https://www.linkedin.com/in/shamilton-darlingdocs/) 📖
8+
- [David Kershaw](https://www.linkedin.com/in/evisioner/) 🤔
9+
- [Khanh Tran](https://www.linkedin.com/in/k-tran) 💻
10+
- [Abhinav Tiwari](https://www.linkedin.com/in/abhinavyogeshtiwari) 💬
11+
- [Javier Giulianetti](https://github.com/JavierGi) 🚇
12+
- [Jimmie Högklint](https://github.com/hogklint) 💻 🚇
13+
- [Davide Cavaliere](https://github.com/microph1) 💻 🚇
14+
- [Brant Watson](https://github.com/induane) 💻
15+
- [Tyler Stubenvoll](https://github.com/tjstub) 💻 📆 📖 🚇
16+
- [Alejandro Fernandez](https://www.linkedin.com/in/afabriciof) 💻
17+
- [Ricardo Boni](https://www.linkedin.com/in/ricardo-boni-9b15744/) 💻 📆 🚇
18+
- [Diogo Basto](https://www.linkedin.com/in/diogo-t-basto/) 💻 🚇
19+
- [Luis Trinidad](https://www.linkedin.com/in/strinidad) 💻 📆 🚇
20+
- [Aarthy Adityan](https://www.linkedin.com/in/aarthyadityan) 🤔 📆 💻 📖
21+
- [Eric Estabrooks](https://www.linkedin.com/in/ericestabrooks/) 🤔
22+
- [Gil Benghiat](https://www.linkedin.com/in/gilbertbenghiat/) 🤔 💬
23+
- [Chris Bergh](https://www.linkedin.com/in/chrisbergh/) 🤔 📖

README.md

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# DataOps Observability
22
![apache 2.0 license Badge](https://img.shields.io/badge/License%20-%20Apache%202.0%20-%20blue) ![PRs Badge](https://img.shields.io/badge/PRs%20-%20Welcome%20-%20green) [![Docker Pulls](https://img.shields.io/badge/dynamic/json?url=https%3A%2F%2Fhub.docker.com%2Fv2%2Frepositories%2Fdatakitchen%2Fdataops-testgen%2F&query=pull_count&style=flat&label=docker%20pulls&color=06A04A)](https://hub.docker.com/r/datakitchen/dataops-observability) [![Documentation](https://img.shields.io/badge/docs-On%20datakitchen.io-06A04A?style=flat)](https://docs.datakitchen.io/articles/#!dataops-observability-help/dataops-observability-help)
33
[![Latest Version](https://img.shields.io/badge/dynamic/json?url=https%3A%2F%2Fhub.docker.com%2Fv2%2Frepositories%2Fdatakitchen%2Fdataops-observability-be%2Ftags%2F&query=results%5B0%5D.name&label=latest%20version&color=06A04A)](https://hub.docker.com/r/datakitchen/dataops-observability-be)
4-
[![Static Badge](https://img.shields.io/badge/Slack-Join%20Discussion-blue?style=flat&logo=slack)](https://data-observability.slack.com)
4+
[![Static Badge](https://img.shields.io/badge/Slack-Join%20Discussion-blue?style=flat&logo=slack)](https://data-observability-slack.datakitchen.io/join)
55

66
*<p style="text-align: center;">DataOps Observability is part of DataKitchen's Open Source Data Observability. DataOps Observability monitors every data journey from data source to customer value, from any team development environment into production, across every tool, team, environment, and customer so that problems are detected, localized, and understood immediately.</p>*
77

@@ -148,6 +148,9 @@ To resolve this error, do two things:
148148
### Getting Started Guide
149149
We recommend you start by going through the [Data Observability Overview Demo](https://docs.datakitchen.io/articles/open-source-data-observability/data-observability-overview).
150150

151+
### Support
152+
For support requests, [join the Data Observability Slack](https://data-observability-slack.datakitchen.io/join) and ask post on #support channel.
153+
151154
### Connect
152155
Talk and Learn with other data practitioners who are building with DataKitchen. Share knowledge, get help, and contribute to our open-source project.
153156

@@ -163,7 +166,7 @@ Join our community here:
163166

164167
* 📚 [Read our blog posts](https://datakitchen.io/blog/)
165168

166-
* 👋 [Join us on Slack](https://data-observability.slack.com)
169+
* 👋 [Join us on Slack](https://data-observability-slack.datakitchen.io/join)
167170

168171
* 🗃 [Sign The DataOps Manifesto](https://DataOpsManifesto.org)
169172

observability_ui/apps/basic-auth/src/app/login/login.component.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ export class LoginComponent {
4545
).subscribe({
4646
next: (url) => this.router.navigateByUrl(url),
4747
error: (error) => {
48-
this.loginError = 'Invalid credentials: Wrong email or password.';
48+
this.loginError = error.error?.error || (error.status === 0 ? 'Unable to reach Observability API.' : error.message);
4949
console.error(error);
5050
},
5151
});

observability_ui/apps/shell/src/app/projects/integrations/tools/abstract-tool.directive.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ export abstract class AbstractTool implements OnInit {
6262
];
6363
protected readonly eventsAgentsCommonEnvs: any[] = [];
6464
protected readonly commonEnvs = [
65-
{ name: 'EVENTS_API_HOST', placeholder: '', required: true },
65+
{ name: 'EVENTS_API_HOST', placeholder: '# the base API URL for Observability', required: true },
6666
{ name: 'EVENTS_API_KEY', placeholder: '# an API key for the Observability project', required: true },
6767
{ name: 'EVENTS_PROJECT_ID', placeholder: '# the ID of the Observability project', required: true },
6868
{ name: 'EXTERNAL_PLUGINS_PATH', placeholder: '# the path where any custom plugins can be accessed', required: true },
@@ -140,7 +140,6 @@ export abstract class AbstractTool implements OnInit {
140140

141141
ngOnInit(): void {
142142
[
143-
this.commonEnvsForm.controls['EVENTS_API_HOST'],
144143
this.commonEnvsForm.controls['EVENTS_PROJECT_ID'],
145144
this.commonEnvsForm.controls['EXTERNAL_PLUGINS_PATH'],
146145
this.commonEnvsForm.controls['ENABLED_PLUGINS'],
@@ -176,7 +175,9 @@ export abstract class AbstractTool implements OnInit {
176175
let dockerExtraEnvVars: string = '';
177176
let kubernetesExtraEnvVars: string = '';
178177
const valuePerVariable: {[name: string]: string} = {
178+
// k8s does not like underscores
179179
target_service: this.name.replaceAll('_', '-'),
180+
deployment_name: `${this.envListForm.controls['AGENT_TYPE']?.value}-${this.envListForm.controls['AGENT_KEY']?.value}`.replaceAll('_', '-'),
180181
docker_image: this.image,
181182
docker_tag: this.envListForm.controls['DOCKER_TAG'].value,
182183
default_deployment_mode: this.envListForm.controls['DEFAULT_DEPLOYMENT_MODE'].value,
@@ -188,7 +189,7 @@ export abstract class AbstractTool implements OnInit {
188189
for (const variable of this.envList) {
189190
if (!variable.hidden && !variable.excluded) {
190191
const slotInTemplate = variable.tpl ?? variable.name.toLowerCase();
191-
let value = this.envListForm.controls[variable.name].value ?? variable.default ?? '';
192+
let value = this.envListForm.controls[variable.name].value || variable.default || '';
192193
if (variable.formatter) {
193194
value = variable.formatter(value);
194195
}

observability_ui/apps/shell/src/app/projects/integrations/tools/agent-script-v2.tpl

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ set -eo pipefail
55
SCRIPT_NAME=$(basename "$0")
66
DOCKER_DEPLOY_FILE="deploy-docker-agent.json"
77
KUBERNETES_DEPLOY_FILE="deploy-kubernetes-agent.json"
8-
DEPLOYMENT_NAME="{{agent_type}}_{{agent_key}}"
8+
DEPLOYMENT_NAME="{{deployment_name}}"
99
SECRET_NAME="observability-agent-secret"
1010
NAMESPACE="datakitchen"
1111

@@ -217,6 +217,9 @@ create_docker_compose_file() {
217217
"$DEPLOYMENT_NAME": {
218218
"image": "$DOCKER_IMAGE:$DOCKER_TAG",
219219
"container_name": "$DEPLOYMENT_NAME",
220+
"extra_hosts": [
221+
"host.docker.internal:host-gateway"
222+
],
220223
"environment": [
221224
"DK_AGENT_TYPE=$DK_AGENT_TYPE",
222225
"DK_AGENT_KEY=$DK_AGENT_KEY",

observability_ui/apps/shell/src/app/projects/integrations/tools/agent-script.tpl

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,6 +117,8 @@ services:
117117
$TARGET_SERVICE:
118118
image: $image
119119
container_name: $TARGET_SERVICE
120+
extra_hosts:
121+
- host.docker.internal:host-gateway
120122
environment:
121123
- EVENTS_API_HOST='$EVENTS_API_HOST'
122124
- EVENTS_API_KEY='$EVENTS_API_KEY'

observability_ui/apps/shell/src/app/projects/integrations/tools/airflow-tool/airflow-tool.component.html

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,6 @@ <h2>Step 1: Prerequisites</h2>
1414
</mat-expansion-panel-header>
1515

1616
<ul>
17-
<li>
18-
Request access to the agent's private Docker repository from your DataKitchen representative.
19-
</li>
2017
<li>
2118
An API key unique to this agent is recommended. Use the form below to create a key.
2219
</li>

observability_ui/apps/shell/src/app/projects/integrations/tools/airflow-tool/airflow-tool.component.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ export class AirflowToolComponent extends AbstractTool {
3939
static override _name = 'airflow';
4040
static override _displayName = 'Airflow';
4141
static override _icon = 'airflow';
42-
static override _image = 'docker.io/datakitchen/observability-agent-trial';
42+
static override _image = 'docker.io/datakitchen/dataops-observability-agents';
4343
static override _version = 'production';
4444

4545
private readonly defaultValues = {
@@ -49,7 +49,7 @@ export class AirflowToolComponent extends AbstractTool {
4949
override readonly envList = [
5050
{ name: 'AGENT_TYPE', tpl: 'agent_type', placeholder: '' },
5151
{ name: 'AGENT_KEY', tpl: 'agent_key', placeholder: '# assigns the identifier of the agent', required: true },
52-
{ name: 'DK_API_URL', tpl: 'observability_base_url', placeholder: '# the URL to the Observability API', required: true },
52+
{ name: 'DK_API_URL', tpl: 'observability_base_url', placeholder: '# the base API URL for Observability', required: true },
5353
{ name: 'DK_API_KEY', tpl: 'observability_service_account_key', placeholder: '# an API key for the Observability project', required: true },
5454
{ name: 'API_URL', tpl: 'airflow_api_url', placeholder: '# the URL to the Airflow API', required: true },
5555
{ name: 'USERNAME', tpl: 'auth_agent_username', placeholder: '# the username for an Airflow account', required: true },

observability_ui/apps/shell/src/app/projects/integrations/tools/aws-sqs-tool/aws-sqs-tool.component.html

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,6 @@ <h2>Step 1: Prerequisites</h2>
1414
</mat-expansion-panel-header>
1515

1616
<ul>
17-
<li>
18-
Request access to the agent's private Docker repository from your DataKitchen representative.
19-
</li>
2017
<li>
2118
An API key unique to this agent is recommended. Use the form below to create a key.
2219
</li>

0 commit comments

Comments
 (0)