Skip to content

feat: pythonic scenario tests (trials)#72

Open
redpanda-f wants to merge 51 commits intomainfrom
feat/redpanda/feat-scenarios-py
Open

feat: pythonic scenario tests (trials)#72
redpanda-f wants to merge 51 commits intomainfrom
feat/redpanda/feat-scenarios-py

Conversation

@redpanda-f
Copy link
Collaborator

No description provided.

@FilOzzy FilOzzy added this to FOC Mar 3, 2026
@github-project-automation github-project-automation bot moved this to 📌 Triage in FOC Mar 3, 2026
@redpanda-f redpanda-f changed the base branch from feat/redpanda/feat-scenarios to main March 3, 2026 09:43
@redpanda-f redpanda-f changed the base branch from main to feat/redpanda/feat-scenarios March 3, 2026 09:44
@redpanda-f redpanda-f self-assigned this Mar 3, 2026
@redpanda-f redpanda-f moved this from 📌 Triage to ⌨️ In Progress in FOC Mar 3, 2026
@redpanda-f
Copy link
Collaborator Author

Todo:

  • && (github.event_name == 'schedule' || github.event_name == 'workflow_dispatch') needs to be added, so issues are not created with PRs
  • REPORT_ON_SUCCESS: 'true' so we do not have issues on success

before merge

@rjan90 rjan90 added this to the M4.1: mainnet ready milestone Mar 5, 2026
@redpanda-f redpanda-f changed the base branch from feat/redpanda/feat-scenarios to main March 5, 2026 17:14
Copilot AI review requested due to automatic review settings March 5, 2026 17:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@rjan90 rjan90 linked an issue Mar 6, 2026 that may be closed by this pull request
@redpanda-f redpanda-f force-pushed the feat/redpanda/feat-scenarios-py branch from ba99563 to 721355e Compare March 6, 2026 14:12
Copy link
Contributor

@BigLep BigLep left a comment

Choose a reason for hiding this comment

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

A couple of things when looking at #75

  1. Can we include the synapse commit that was used as well?
  2. (nit) Status has a period after it "Status." We can remove the period
  3. For versions info, at the minimum can we remove the prefix that is in the log messages (e.g., "�[32m INFO�[0m �[2msrc/main_app/version.rs�[0m�[2m:�[0m�[2m19:�[0m "). (Even better is just to get it to a simpler bullet list or table, but that's not essential.)
  4. CI run link: can we link directly do the corresponding "job" rather than just to the "run". For example, link to the stability job.
  5. For the test summaries, can we link to the corresponding step as well? For example,I would expect to see a URL with an anchor like #step:39:26 (e.g., https://github.com/FilOzone/foc-devnet/actions/runs/22752589844/job/65990079825#step:39:27)

@BigLep
Copy link
Contributor

BigLep commented Mar 6, 2026

A couple of things:

  1. I assume a better PR description will be provided
  2. I was also wondering about how to break this down into smaller units so parts can start getting merged in smaller chunk. I could imagine a PR for the "latesttag" functionality. Maybe another for the githooks and linting, and then the main PR that adds the scenarios?
  3. I assume we need to add some README_ADVANCED docs about "latesttag"?

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

Labels

None yet

Projects

Status: ⌨️ In Progress

Development

Successfully merging this pull request may close these issues.

CI/Nightly End-to-End Validation of FOC as a whole

4 participants