search

sevntu-checkstyle / methods-distance

GNU Lesser General Public License v3.0
5 stars 13 forks source link

infra: setup manual trigger for heroku deploy #75

Closed nrmancuso closed 1 year ago

nrmancuso commented 1 year ago

From https://github.com/sevntu-checkstyle/methods-distance/pull/74#issuecomment-1469933987:

@nrmancuso , it would be awesome. better to release when required, not on commit, as some commits are not functional changes for user.

nrmancuso commented 1 year ago

Direct link to deployment README: https://github.com/sevntu-checkstyle/methods-distance/blob/7068a3dba6944712d65e4271b4c36d4c1e761596/.circleci/README.md

nrmancuso commented 1 year ago

Link to workflow triggered manually as proof that changes work: https://app.circleci.com/pipelines/github/sevntu-checkstyle/methods-distance/10/workflows/8531b380-48f2-40d6-8495-7a3af45a115d

We still need to set API key in environment variables

romani commented 1 year ago

it is not good that CI fails, deploy should not be triggered in PR.

nrmancuso commented 1 year ago

@romani see https://github.com/sevntu-checkstyle/methods-distance/pull/75#issuecomment-1469965967, CI is failing from missing API key. I triggered workflow manually (the only way to do it now), do you still want branch filter?

romani commented 1 year ago

We did not trigger it to let it fail. Or it is feature of circleci to show all builds for commit? even they are manual

nrmancuso commented 1 year ago

@romani builds are on commit, and show up in PR if commit exists there

I triggered the build by instructions in PR

romani commented 1 year ago

key is regenerated at https://dashboard.heroku.com/account and deploy is passed .

romani commented 1 year ago

there is no free tier any more.

all requests, endup in error, in Heroku logs: 2023-04-15T14:26:48.541486+00:00 heroku[router]: at=error code=H14 desc="No web processes running" method=GET path="/dsm?source_url=https://raw.githubusercontent.com/checkstyle/checkstyle/b4e884c2ff3bef182b045692b59c1aceae3cb892/src/main/java/com/puppycrawl/tools/checkstyle/Checker.java" host=methods-distance.herokuapp.com request_id=a5c18a6e-248a-4578-9033-0d48f1b860a7 fwd="76.103.49.161" dyno= connect= service= status=503 bytes= protocol=https

~/java/github/sevntu-checkstyle/methods-distance [master|✔] 
07:46 $ heroku ps:scale web=1 -a methods-distance
 ›   Warning: heroku update available from 7.69.1 to 8.0.4.
Scaling dynos... !
 ▸    Item Could not be Updated: The app owner has to subscribe to Eco
   to scale your dynos. Learn more at https://blog.heroku.com/new-low-cost-plans

I opened ticket on how to stay free - https://help.heroku.com/tickets/1240314?n=1

links: https://help.heroku.com/8GD2AVBW/why-am-i-seeing-an-application-error-after-the-termination-of-free-plans

romani commented 1 year ago

https://help.heroku.com/RSBRUH58/removal-of-heroku-free-product-plans-faq

Open source programs: If you are a maintainer on an open-source project, and would like to request Heroku support for your project, contact the Salesforce Open Source Program office at ospo-heroku-credits@salesforce.com.

I sent request to it.