Skip to content

CI failing on PRs #768

Open
Open

Description

@lionello

which is why dependabot PRs are not getting auto-merged "on green".

Activity

  1. defangdevs commented on Oct 30, 2026

    @defangdevs
    Contributor

    Findings (investigated, no code change made):

    Asks for @lionello:

    1. Add the DEPENDABOT_AUTOMERGE_TOKEN secret, then merge chore(ci): merge all Dependabot updates after CI passes #764.
    2. Say if you want me to look at the mastra defang ps exit-5 failure as a separate issue.
  2. lionello commented on Oct 30, 2026

    @lionello
    MemberAuthor
    1. Elsewhere we had auto-merge "on green" working without dealing with tokens. How? I expected the same.
    2. yes, fix it now as part of this issue.
      @defangdevs
  3. defangdevs commented on Oct 30, 2026

    @defangdevs
    Contributor
    1. No token is needed for the existing workflow. Elsewhere (defang, portal, etc.) it is the same dependabot-automerge.yml: GITHUB_TOKEN approves and runs gh pr merge --auto, with repo auto-merge enabled. The samples repo already has that workflow, and it works. Single patch/minor bumps merged this way (chore(deps): bump crass from 1.0.6 to 1.0.7 in /samples/rails/app #749-chore(deps): bump body-parser from 2.3.0 to 2.4.0 in /samples/self-improving/agent #762, merged by github-actions). The open Dependabot PRs are not stuck on a token. chore(deps): bump body-parser and sails in /samples/sailsjs-postgres/app #763 and chore(deps): bump body-parser and express in /starter-sample/app #759 are grouped updates, where fetch-metadata returns update-type: null (the run log shows it), so the approve and auto-merge steps are skipped. chore(deps): bump zod from 3.22.4 to 4.6.6 in /samples/vllm/ui #761, chore(deps): bump next from 14.1.1 to 15.5.28 in /samples/vllm/ui #758, chore(deps-dev): bump vite and @vitejs/plugin-vue in /samples/vuejs/app #757 and chore(deps-dev): bump vite, @sveltejs/kit and @sveltejs/vite-plugin-svelte in /samples/sveltekit/sveltekit #756 are major bumps, which the workflow skips by design. So chore(ci): merge all Dependabot updates after CI passes #764's DEPENDABOT_AUTOMERGE_TOKEN is not required for this; I would close it. If you want grouped PRs merged too, the small change is to treat a null update-type like patch/minor. Say if you want that as a PR.

    2. Fixed in ci(samples): skip BYOC-only mastra-extended in deploy check #770. mastra-extended is not deployable on the Playground (2 GiB model services over the 1 GiB quota, managed Postgres/Redis unsupported). compose up died in about 3s, and the harness's follow-up defang ps --json then exited 5 and hid the cause. ci(samples): skip BYOC-only mastra-extended in deploy check #770 skips it in the deploy check, like self-improving.

    Asks for @lionello:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions