Pack to integrate OpenAI API to Stackstorm #179

dalarsen · 2023-05-15T14:59:32Z

Pack that adds two actions (chat completion and image generation) through the OpenAI API. Hopefully I've included most of the necessary stuff for this to be correct but please let me know of any changes that are needed. Thank you!

CLAassistant · 2023-05-15T14:59:40Z

All committers have signed the CLA.

arm4b · 2023-08-11T22:31:07Z

Great stuff!
Noticed there's a StackStorm logo as icon.png. Worth replacing it with the official OpenAI logo.

dalarsen · 2023-08-11T22:41:24Z

Great stuff! Noticed there's a StackStorm logo as icon.png. Worth replacing it with the official OpenAI logo.

Nice catch, thanks; i've replaced the logo.

arm4b

Awesome addition to the StackStorm Exchange. Thanks for the contribution!

Looking at other packs, please also include the following to make it complete:

CHANGES.md (example)
openai.yaml.example (like this) - just an example of pack config with some dummy data
.github/CODEOWNERS (example) and include yourself as a maintainer + @StackStorm-Exchange/tsc, - this way on every PR and issue we'll get notifications
Add yourself as a Maintainer in the README footer (example format)

Some basic /tests/ would be nice if you can add them too, but not required.

I think some ChatOps action aliases would be ideal for such a pack, but we can add them in a later PRs after creating a pack.

arm4b · 2023-08-11T22:44:36Z

openai/requirements.txt

@@ -0,0 +1 @@
+openai


Ideally to pin the version to the current major (interval) in case of the upstream breaking changes

dalarsen · 2023-08-15T17:26:22Z

Thanks for the feedback! I think I've addressed everything, if I've missed anything please let me know. Thank you!

openai/.github/CODEOWNERS

arm4b · 2023-08-15T20:15:40Z

Thanks!
Will ask if @StackStorm-Exchange/tsc has any feedback, but the PR looks great to me!

I'll need to check why CI is failing on this as well before merging.

Copy openai pack into directory

openai/requirements.txt

Co-authored-by: Eugen C. <[email protected]>

arm4b · 2023-08-17T11:36:28Z

openai/requirements.txt

@@ -0,0 +1 @@
+openai>=0.27.0,<1.0


Ah, I see now what's going on.
https://pypi.org/project/openai/ Requires minimum Python >=3.7.1, while the tests are ran under the python 3.6

The technical debt of maintaining the Python versions follows us everywhere: StackStorm/community#103

Ahh, that explains it! As a workaround would it be an option for me to just modify the github action that does the tests to use a later Python version (and/or change the image being used to one that's noted to support a later Python version)? Or is there more to it than that?

Yeah, I think it worth trying with Python 3.8 👍 as that's the other version supported by StackStorm.

After pack is promoted to Exchange, we use a common GHA for the builds: https://github.com/StackStorm-Exchange/stackstorm-jira/blob/master/.github/workflows/build_and_test.yaml which is ran under the py3.6
But I just discovered some packs (https://github.com/StackStorm-Exchange/stackstorm-vault/blob/master/.github/workflows/build_and_test.yaml) have this workflow overridden, so it should be fine to do that as well in this new pack.

I changed the workflow to use 3.8 and it looks like it's passing now, though I'm not sure if I did it in a way that it would be kept if it were to be moved to the Exchange.

arm4b

Nice! Thanks @dalarsen!

@cognifloyd I'm going to bootstrap the pack next, following the #172 you've built and see what happens 🤞 Do you foresee any potential issues by switching the CI workflow to py3.8 in this PR?

arm4b · 2023-09-05T11:35:15Z

!bootstrap pack

github-actions · 2023-09-05T11:35:39Z

🚀 Hold onto your hats! Now bootstrapping a new pack repo...

Bootstrapping will:
(1) extract details about the new pack;
(2) create the pack repo and repo metadata;
(3) copy this PR into the transfer branch in the new repo;
(4) use transfer branch for the pack's first PR.

Details: https://github.com/StackStorm-Exchange/exchange-incubator/actions/runs/6084026976

arm4b · 2023-09-05T12:54:18Z

!bootstrap pack

github-actions · 2023-09-05T12:54:41Z

🚀 Hold onto your hats! Now bootstrapping a new pack repo...

Bootstrapping will:
(1) extract details about the new pack;
(2) create the pack repo and repo metadata;
(3) copy this PR into the transfer branch in the new repo;
(4) use transfer branch for the pack's first PR.

Details: https://github.com/StackStorm-Exchange/exchange-incubator/actions/runs/6084873982

arm4b · 2023-09-05T12:57:41Z

!bootstrap pack

github-actions · 2023-09-05T12:58:06Z

🚀 Hold onto your hats! Now bootstrapping a new pack repo...

Bootstrapping will:
(1) extract details about the new pack;
(2) create the pack repo and repo metadata;
(3) copy this PR into the transfer branch in the new repo;
(4) use transfer branch for the pack's first PR.

Details: https://github.com/StackStorm-Exchange/exchange-incubator/actions/runs/6084916981

github-actions · 2023-09-05T12:59:07Z

🎉 Hoooray! Here is the pack's first PR: StackStorm-Exchange/stackstorm-openai#1

Please do the following:
(1) make sure everything looks correct in the PR;
(2) merge it!
(3) make sure a TSC Senior Maintainer has setup group and user access;
(4) wait for the next exchange index update (monitor updates here)
(5) once the index has updated, check out:
https://exchange.stackstorm.com/#openai
(6) Close WITHOUT merging this PR. DO NOT MERGE THIS PR.

arm4b · 2023-09-05T13:40:28Z

@dalarsen We're getting closer!
The new pack repo was bootstrapped.
Need a bit more help from you in the StackStorm-Exchange/stackstorm-openai#1

arm4b · 2023-09-15T20:18:42Z

The pack is live!
https://exchange.stackstorm.org/#openai

@dalarsen Thanks for the contribution!
Please also accept Github invite so you'll have write access to the https://github.com/StackStorm-Exchange/stackstorm-openai repository.

dalarsen force-pushed the master branch from 492f0bf to a156582 Compare August 11, 2023 19:28

arm4b suggested changes Aug 11, 2023

View reviewed changes

dalarsen force-pushed the master branch from f73e74e to a67c2f6 Compare August 15, 2023 17:26

arm4b reviewed Aug 15, 2023

View reviewed changes

openai/.github/CODEOWNERS Outdated Show resolved Hide resolved

arm4b mentioned this pull request Aug 15, 2023

Fix py3.6 build by pinning the container to Ubuntu 20.04 #180

Merged

dalarsen and others added 4 commits August 16, 2023 14:02

Copy openai pack into directory

9fe78b7

Copy openai pack into directory

Replace icon.png image with OpenAI logo

d0ae2da

Adding chatops aliases, unit tests, misc corrections

661126c

Update openai/.github/CODEOWNERS

419621d

arm4b force-pushed the master branch from 52ca51b to 419621d Compare August 16, 2023 13:02

arm4b reviewed Aug 16, 2023

View reviewed changes

openai/requirements.txt Outdated Show resolved Hide resolved

dalarsen and others added 2 commits August 16, 2023 13:31

Update openai/requirements.txt

0312326

Co-authored-by: Eugen C. <[email protected]>

Update requirements.txt

7db2e49

arm4b reviewed Aug 17, 2023

View reviewed changes

Try updating to py3.8

a449081

arm4b approved these changes Sep 1, 2023

View reviewed changes

github-actions bot added the bootstrap:in-progress label Sep 5, 2023

github-actions bot removed the bootstrap:in-progress label Sep 5, 2023

stackstorm-neptr added a commit to StackStorm-Exchange/stackstorm-openai that referenced this pull request Sep 5, 2023

Merge StackStorm-Exchange/exchange-incubator#179

d40a83c

stackstorm-neptr mentioned this pull request Sep 5, 2023

Transfer OpenAI pack from Incubator StackStorm-Exchange/stackstorm-openai#1

Merged

github-actions bot added the bootstrap:complete label Sep 5, 2023

arm4b closed this Sep 15, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Pack to integrate OpenAI API to Stackstorm #179

Pack to integrate OpenAI API to Stackstorm #179

dalarsen commented May 15, 2023

CLAassistant commented May 15, 2023 •

edited

Loading

arm4b commented Aug 11, 2023

dalarsen commented Aug 11, 2023

arm4b left a comment

arm4b Aug 11, 2023

dalarsen commented Aug 15, 2023

arm4b commented Aug 15, 2023

arm4b Aug 17, 2023 •

edited

Loading

dalarsen Aug 17, 2023

arm4b Aug 23, 2023

dalarsen Sep 1, 2023

arm4b left a comment •

edited

Loading

arm4b commented Sep 5, 2023

github-actions bot commented Sep 5, 2023

arm4b commented Sep 5, 2023

github-actions bot commented Sep 5, 2023

arm4b commented Sep 5, 2023

github-actions bot commented Sep 5, 2023

github-actions bot commented Sep 5, 2023

arm4b commented Sep 5, 2023 •

edited

Loading

arm4b commented Sep 15, 2023

		@@ -0,0 +1 @@
		openai>=0.27.0,<1.0

Pack to integrate OpenAI API to Stackstorm #179

Pack to integrate OpenAI API to Stackstorm #179

Conversation

dalarsen commented May 15, 2023

CLAassistant commented May 15, 2023 • edited Loading

arm4b commented Aug 11, 2023

dalarsen commented Aug 11, 2023

arm4b left a comment

Choose a reason for hiding this comment

arm4b Aug 11, 2023

Choose a reason for hiding this comment

dalarsen commented Aug 15, 2023

arm4b commented Aug 15, 2023

arm4b Aug 17, 2023 • edited Loading

Choose a reason for hiding this comment

dalarsen Aug 17, 2023

Choose a reason for hiding this comment

arm4b Aug 23, 2023

Choose a reason for hiding this comment

dalarsen Sep 1, 2023

Choose a reason for hiding this comment

arm4b left a comment • edited Loading

Choose a reason for hiding this comment

arm4b commented Sep 5, 2023

github-actions bot commented Sep 5, 2023

arm4b commented Sep 5, 2023

github-actions bot commented Sep 5, 2023

arm4b commented Sep 5, 2023

github-actions bot commented Sep 5, 2023

github-actions bot commented Sep 5, 2023

arm4b commented Sep 5, 2023 • edited Loading

arm4b commented Sep 15, 2023

CLAassistant commented May 15, 2023 •

edited

Loading

arm4b Aug 17, 2023 •

edited

Loading

arm4b left a comment •

edited

Loading

arm4b commented Sep 5, 2023 •

edited

Loading