Skip to content
This repository was archived by the owner on Aug 14, 2023. It is now read-only.

Pinnwand deployment#153

Merged
ChrisLovering merged 2 commits into
mainfrom
pinnwand-deployment
Jul 15, 2023
Merged

Pinnwand deployment#153
ChrisLovering merged 2 commits into
mainfrom
pinnwand-deployment

Conversation

@ChrisLovering
Copy link
Copy Markdown
Member

This is not live yet. Currently the deployment exists at paste-new.pythondiscord.com.

There is an open PR upstream to have images built & pushed to ghcr, which will mean we can remove the need for a fork too.

This shouldn't be deployed until we have ensured configuration is exactly as we want it.

@ChrisLovering ChrisLovering requested a review from a team as a code owner June 3, 2023 22:03
@ChrisLovering
Copy link
Copy Markdown
Member Author

Failed lint is due to paste.pythondiscord.com already being used on prod, can safely be ignored as a false flag for now.

@ChrisLovering ChrisLovering force-pushed the pinnwand-deployment branch from bc44453 to c0d746e Compare June 4, 2023 10:53
Comment thread namespaces/default/pinnwand/README.md Outdated
@ChrisLovering ChrisLovering force-pushed the pinnwand-deployment branch 2 times, most recently from 318a8e1 to f15356c Compare June 5, 2023 10:28
Comment thread namespaces/default/pinnwand/deployment.yaml Outdated
@ChrisLovering ChrisLovering force-pushed the pinnwand-deployment branch from f15356c to 9e2cba3 Compare June 7, 2023 13:45
@jchristgit
Copy link
Copy Markdown
Contributor

jchristgit commented Jun 7, 2023 via email

@wookie184
Copy link
Copy Markdown

I think the ratelimits will need configuring, otherwise the bot could be ratelimited (e.g when using the eval command it can create many pastes at once). We can probably afford to be pretty generous.

Also not sure we want the spamscore thing, sending links could potentially be a valid use case so IMO it would make more sense to start with it disabled and enable it if it's an issue.

@wookie184
Copy link
Copy Markdown

Another note, i think a 1 day expiry as a default is a bit short, if we put the config for a month first would it become default?

@ChrisLovering
Copy link
Copy Markdown
Member Author

Another note, i think a 1 day expiry as a default is a bit short, if we put the config for a month first would it become default?

Yes, it does. I've updated and deployed that change.

@ChrisLovering ChrisLovering force-pushed the pinnwand-deployment branch 2 times, most recently from cb11f80 to d716f39 Compare June 8, 2023 08:36
@ChrisLovering ChrisLovering force-pushed the pinnwand-deployment branch 5 times, most recently from 1c5b531 to d618154 Compare July 11, 2023 14:43
@ChrisLovering ChrisLovering merged commit 38c1469 into main Jul 15, 2023
@ChrisLovering ChrisLovering deleted the pinnwand-deployment branch July 15, 2023 13:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants