Skip to content

Add IB Gateway diagnostics workflow - #26

Merged
Pigbibi merged 1 commit into
mainfrom
codex/ib-gateway-diagnose-workflow
May 10, 2026
Merged

Add IB Gateway diagnostics workflow#26
Pigbibi merged 1 commit into
mainfrom
codex/ib-gateway-diagnose-workflow

Conversation

@Pigbibi

@Pigbibi Pigbibi commented May 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add a manual diagnostics workflow that SSHes to the gateway VM and prints safe runtime state
  • include Docker status, host/container ports, 2FA watcher status, X11 window titles, 2FA bot logs, and Gateway logs
  • avoid reading or printing credentials

Tests

  • Not run; workflow-only diagnostic helper.

@Pigbibi
Pigbibi merged commit b8cd392 into main May 10, 2026
1 check passed
@Pigbibi
Pigbibi deleted the codex/ib-gateway-diagnose-workflow branch May 10, 2026 16:42

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7650840ae7

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".


REMOTE_DIAG_COMMAND=$(cat <<'EOF'
set -euo pipefail
cd __DEPLOY_PATH__

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Continue diagnostics when deploy path is unavailable

The remote script hard-fails on cd __DEPLOY_PATH__ under set -euo pipefail, so if the deploy directory is missing/corrupted (a common failure mode during broken deployments), the workflow exits before collecting host/container status, ports, and logs. That defeats the purpose of a rescue diagnostic run in exactly the scenarios where it is most needed; make the cd conditional (or gather non-path-dependent checks first) so diagnostics still run when the app directory is absent.

Useful? React with 👍 / 👎.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant