Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update files generated from the k6 Extension Registry #1751

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

github-actions[bot]
Copy link
Contributor

The k6 Extension Registry has changed.
This pull request contains the files generated from the k6 Extension Registry.

@CLAassistant
Copy link

CLAassistant commented Sep 30, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions bot force-pushed the extension-registry-changed branch from a344303 to 5537e4b Compare September 30, 2024 16:47
@heitortsergent
Copy link
Collaborator

@szkiba something doesn't look right here. 🤔 Do you know what the script was trying to do?

@szkiba
Copy link
Contributor

szkiba commented Sep 30, 2024

Ooops, I just accepted some last minute change proposal on PR from @jdbaldry and it seem some of them has a syntax problem...
(https://github.com/grafana/k6-docs/actions/runs/11110541005/job/30868471419)
I'll check it tomorrow...

@jdbaldry
Copy link
Member

#1755

@github-actions github-actions bot force-pushed the extension-registry-changed branch from 5537e4b to 48849bd Compare September 30, 2024 18:11
@jdbaldry
Copy link
Member

I re-ran the workflow manually in https://github.com/grafana/k6-docs/actions/runs/11111741710

@szkiba
Copy link
Contributor

szkiba commented Sep 30, 2024

Much better now :)
I'll add review to each change

@@ -51,10 +47,6 @@ Use the table to explore the many extensions. Questions? Feel free to join the d
<h4>xk6-cache</h4>
<p>Enable vendoring remote HTTP modules to a single source-control-friendly file</p>
</a>
<a href="https://github.com/fornfrey/xk6-celery" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extension does not meet the registry requirements.
fornfrey/xk6-celery#7
fornfrey/xk6-celery#6

@@ -35,10 +35,6 @@ Use the table to explore the many extensions. Questions? Feel free to join the d
<h4>xk6-ansible-vault</h4>
<p>Encrypt and decrypt Ansible Vault</p>
</a>
<a href="https://github.com/joanlopez/xk6-aws" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extension does not meet the registry requirements.
joanlopez/xk6-aws#2
joanlopez/xk6-aws#1

@@ -95,14 +87,6 @@ Use the table to explore the many extensions. Questions? Feel free to join the d
<h4>xk6-dotenv</h4>
<p>Load env vars from a .env file</p>
</a>
<a href="https://github.com/szkiba/xk6-ts" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extension owner changed this is why name changed from "szkiba/xk6-ts" to "grafana/xk6-ts"

@@ -323,6 +275,10 @@ Use the table to explore the many extensions. Questions? Feel free to join the d
<h4>xk6-tracetest</h4>
<p>Support for Tracetest test execution and tracing client</p>
</a>
<a href="https://github.com/grafana/xk6-ts" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extension owner changed this is why name changed from "szkiba/xk6-ts" to "grafana/xk6-ts"

<h4>xk6-ts</h4>
<p>Add TypeScript support for k6</p>
</a>
<a href="https://github.com/BarthV/xk6-es" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extension does not meet the registry requirements
BarthV/xk6-es#7
BarthV/xk6-es#6

@@ -135,14 +119,6 @@ Use the table to explore the many extensions. Questions? Feel free to join the d
<h4>xk6-git</h4>
<p>Clone Git repositories from tests</p>
</a>
<a href="https://github.com/AckeeCZ/xk6-google-iap" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extension does not meet the registry requirements
AckeeCZ/xk6-google-iap#10
AckeeCZ/xk6-google-iap#9
AckeeCZ/xk6-google-iap#8

<h4>xk6-google-iap</h4>
<p>Provides access to Google Auth token</p>
</a>
<a href="https://github.com/heww/xk6-harbor" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extension does not meet the registry requirements
goharbor/xk6-harbor#16
goharbor/xk6-harbor#15

@@ -185,15 +161,7 @@ Use the table to explore the many extensions. Questions? Feel free to join the d
</a>
<a href="https://github.com/pmalhaire/xk6-mqtt" target="_blank" class="nav-cards__item nav-cards__item--guide">
<h4>xk6-mqtt</h4>
<p>mqtt extension</p>
</a>
<a href="https://github.com/ydarias/xk6-nats" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extension does not meet the registry requirements
ydarias/xk6-nats#15
ydarias/xk6-nats#14

<h4>xk6-nats</h4>
<p>Provides NATS support</p>
</a>
<a href="https://github.com/patrick-janeiro/xk6-neo4j" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extension does not meet the registry requirements
patrick-janeiro/xk6-neo4j#5

<a href="https://github.com/patrick-janeiro/xk6-neo4j" target="_blank" class="nav-cards__item nav-cards__item--guide">
<h4>xk6-neo4j</h4>
<p>Interact with Neo4J graph databases</p>
<p>MQTT extension</p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MQTT is an abbreviation

@@ -215,10 +183,6 @@ Use the table to explore the many extensions. Questions? Feel free to join the d
<h4>xk6-output-clickhouse</h4>
<p>Export results to ClickHouse</p>
</a>
<a href="https://github.com/dynatrace/xk6-output-dynatrace" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -255,14 +219,10 @@ Use the table to explore the many extensions. Questions? Feel free to join the d
<h4>xk6-output-timestream</h4>
<p>Export results to AWS Timestream</p>
</a>
<a href="https://github.com/wosp-io/xk6-playwright" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

owner changed

@@ -255,14 +219,10 @@ Use the table to explore the many extensions. Questions? Feel free to join the d
<h4>xk6-output-timestream</h4>
<p>Export results to AWS Timestream</p>
</a>
<a href="https://github.com/wosp-io/xk6-playwright" target="_blank" class="nav-cards__item nav-cards__item--guide">
<a href="https://github.com/nicholasvuono/xk6-playwright" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

owner changed

<h4>xk6-playwright</h4>
<p>Browser automation and end-to-end web testing using Playwright</p>
</a>
<a href="https://github.com/mcosta74/xk6-plist" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extension does not meet the registry requirements
mcosta74/xk6-plist#2

@@ -287,10 +247,6 @@ Use the table to explore the many extensions. Questions? Feel free to join the d
<h4>xk6-sql</h4>
<p>Load-test SQL Servers</p>
</a>
<a href="https://github.com/mridehalgh/xk6-sqs" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extension does not meet the registry requirements
mridehalgh/xk6-sqs#8

@@ -303,11 +259,7 @@ Use the table to explore the many extensions. Questions? Feel free to join the d
<h4>xk6-stomp</h4>
<p>Client for STOMP protocol</p>
</a>
<a href="https://github.com/NAlexandrov/xk6-tcp" target="_blank" class="nav-cards__item nav-cards__item--guide">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extension does not meet the registry requirements
NAlexandrov/xk6-tcp#7

Copy link
Contributor

@szkiba szkiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
@heitortsergent , I reviewed all the changes and wrote a justification for them.
Would you please look it over and if you consider the justifications appropriate, then approve it.

In most cases, the extension does not meet the requirements of the registry. I opened issues about these in the extensions repository about 2 months ago and received no response. They don't seem to be maintained.

@heitortsergent heitortsergent self-assigned this Sep 30, 2024
@heitortsergent heitortsergent added Area:extensions Issues about the extension ecosystem type/docs labels Sep 30, 2024
@heitortsergent
Copy link
Collaborator

@dgzlopes I added you as a reviewer here. @szkiba this is related to the extensions registry project, @szkiba implemented a way for us to keep the information between the extensions registry and the docs in sync.

There are a few extensions that would be removed from our docs list because they don't meet our extensions requirements.

I'm wondering, is it okay to remove all of the ones listed in this PR, or are there any of them that might still be useful for users, even if they don't meet all of our requirements? 🤔

@github-actions github-actions bot force-pushed the extension-registry-changed branch from 48849bd to 606f6e0 Compare October 1, 2024 12:54
@github-actions github-actions bot force-pushed the extension-registry-changed branch from 606f6e0 to 85c4c7d Compare October 3, 2024 08:46
@dgzlopes
Copy link
Member

dgzlopes commented Oct 3, 2024

@heitortsergent, I see that Ivan reached out to all Extension developers asking them to update their extensions... To be honest, I'm a bit sad to see a few of these go away (e.g., Harbor), but unless we fork them (& we don't have resources to maintain more stuff), I'm not sure there is anything that can be done.

@heitortsergent
Copy link
Collaborator

Ok, I guess I will merge this for now. I haven't kept up with the conversations around the extensions registry, but maybe there's some way for us to have a list of useful extensions that might not be "recommended" somewhere? 🤔

@heitortsergent heitortsergent merged commit 01fa95d into main Oct 3, 2024
1 check was pending
@heitortsergent heitortsergent deleted the extension-registry-changed branch October 3, 2024 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area:extensions Issues about the extension ecosystem type/docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants