rqlite automatic pr's via updatecli

NerdyShawn commented 1 month ago

Thank you for wanting to submit a Pull Request to the Civo Kubernetes Marketplace repository!

If your pull request is to submit a new application to the marketplace, please answer the following questions:

[x] Have you followed the guidelines in our Contributing document in setting up the application?
[x] Have you checked that the application your submission is proposing to add does not yet exist on the Marketplace, or as a pull request to be processed?
[x] Have you tested your application on the Civo managed Kubernetes service? Please include a screenshot.
[x] Can you confirm that the software you are submitting licensed for use in a publicly-available Kubernetes marketplace?
[x] Have you included the specific version of the application to be installed?

If your pull request concerns an existing Marketplace application, please make sure you have:

[x] Notified the Maintainer of the application in this pull request so that they are aware of your proposal.
[x] Outlined the changes you are proposing, and the reasons these are required (e.g. stability, compatibility with new versions of Kubernetes implementations, etc).
[x] Tested that the application works with the proposed updates applied (including a screenshot).
[x] Updated the version number of the app if that has changed.

NerdyShawn commented 1 month ago

The dry-run seems fine

⚠ deps: bump rqlite version:
    Source:
        ✔ [app] Get latest rqlite version
        ✔ [chart] Get latest rqlite Helm Chart version
    Target:
        ✔ [install] Update rqlite/install.sh
        ✔ [manifest] Update rqlite/manifest.yaml

Run Summary
===========
Pipeline(s) run:
  * Changed:    1
  * Failed: 0
  * Skipped:    0
  * Succeeded:  18
  * Total:  19

but when I applied my targets step is failing, and I checked pat token permissions.

TARGETS
========

install
-------
✔ - all contents from 'file' and 'files' combined already up to date

⚠ While nothing change in the current pipeline run, according to the git history, some commits must be pushed
ERROR: non-fast-forward update
ERROR: push error: authorization failed

manifest
--------
✔ - no change detected:
    * key "$.version" already set to "8.24.2", from file "rqlite/manifest.yaml"

⚠ While nothing change in the current pipeline run, according to the git history, some commits must be pushed
ERROR: non-fast-forward update
ERROR: push error: authorization failed

ERROR: something went wrong in target "install" : "authorization failed"
ERROR: something went wrong in target "manifest" : "authorization failed"

Pipeline "deps: bump rqlite version" failed
Skipping due to:
    targets stage:  "something went wrong during target execution"

ACTIONS
========

ERROR: 2 target(s) (manifest,install) failed for action "kubernetes-marketplace"
Existing GitHub pull request found: https://github.com/civo/kubernetes-marketplace/pull/746
Existing GitHub pull request found: https://github.com/civo/kubernetes-marketplace/pull/748
Existing GitHub pull request found: https://github.com/civo/kubernetes-marketplace/pull/747
Existing GitHub pull request found: https://github.com/civo/kubernetes-marketplace/pull/754
Existing GitHub pull request found: https://github.com/civo/kubernetes-marketplace/pull/764
Existing GitHub pull request found: https://github.com/civo/kubernetes-marketplace/pull/745
Existing GitHub pull request found: https://github.com/civo/kubernetes-marketplace/pull/755
Existing GitHub pull request found: https://github.com/civo/kubernetes-marketplace/pull/760
Existing GitHub pull request found: https://github.com/civo/kubernetes-marketplace/pull/756
Existing GitHub pull request found: https://github.com/civo/kubernetes-marketplace/pull/753

Run Summary
===========
Pipeline(s) run:
  * Changed:    0
  * Failed: 1
  * Skipped:    0
  * Succeeded:  18
  * Total:  19
ERROR: ✗ 1 over 19 pipeline failed
ERROR: command failed: 1 over 19 pipeline failed

olblak commented 1 month ago

@NerdyShawn Your pullrequest sounds good to me even thought I didn't tested but you can target your own repository by updating https://github.com/civo/kubernetes-marketplace/blob/418d85b86adfd114b9e5319bb49f0d483a2226ab/.github/updatecli/values.d/scm.yaml#L2 (locally)

And then running from your machine

updatecli diff --config .github/updatecli/updatecli.d/rqlite.yaml --values .github/updatecli/values.d/scm.yaml

NerdyShawn commented 1 month ago

Nice, thanks for the example to target an individual app like this. I did change the org and set my token's right and verified against my repo that it is making PR's ok so got confidence now in the merge.

Run Summary
===========
Pipeline(s) run:
  * Changed:    13
  * Failed: 0
  * Skipped:    0
  * Succeeded:  6
  * Total:  19

opened 13 pr's ok Thanks for the help and giving me and mentioning targeting the one app is a bit more helpful too, plus quicker than running through the current 19 of them.

civo / kubernetes-marketplace

rqlite automatic pr's via updatecli #765