Skip to content

E2e add kafka fix #4

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
ariWeinberg merged 41 commits into from
Mar 30, 2025
Merged

E2e add kafka fix #4

ariWeinberg merged 41 commits into from
Mar 30, 2025

Conversation

@ariWeinberg
Copy link
Collaborator

@ariWeinberg ariWeinberg commented Mar 30, 2025

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

  • I sign off on contributing this submission to open-source
  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

@ariWeinberg
feat: introduce Kafka and Redis broadcast settings for improved conta…
686a121 
…iner configuration
@ariWeinberg
feat: refactor test container imports and update Redis broadcast cont…
0a16663 
…ainer with URL method
@ariWeinberg
feat: implement Kafka and Zookeeper container classes for improved te…
bfb7947 
…sting setup
@ariWeinberg
feat: simplify KafkaUIContainer implementation and enhance environmen…
299727f 
…t variable configuration
@ariWeinberg
feat: refactor KafkaBroadcastContainer to simplify implementation and…
5bb78ba 
… enhance environment configuration
@ariWeinberg
feat: refactor ZookeeperContainer to simplify initialization and enha…
fdb97f6 
…nce environment configuration
@ariWeinberg
feat: refactor Kafka and OPA container imports, enhance logging, and …
4e9ac89 
…improve container management
@ariWeinberg
feat: add OPA settings and enhance broadcaster configuration for impr…
4e8125f 
…oved testing
@ariWeinberg
feat: remove deprecated .env.example, introduce SupportedPolicyRepo e…
72fd0e4 
…num, and update policy repo handling in tests
@ariWeinberg
feat: update repo_providers and broadcasters in PyTestSessionSettings…
bef0d99 
… for enhanced testing support
@ariWeinberg
feat: enhance temp_dir fixture to create a persistent temporary direc…
8f6f2ed 
…tory with proper permissions
@ariWeinberg
feat: update GithubPolicyRepoSettings to utilize session matrix for d…
72fede2 
…ynamic configuration
@ariWeinberg
feat: add .env.example for environment variable configuration in tests
a1c4bd3
@ariWeinberg
fix: update SSH key path handling to correctly expand user directory
3aeee0d
@ariWeinberg
feat: refactor create_github_policy_repo_settings to use session matr…
46207fc 
…ix for dynamic repo configuration
@ariWeinberg
feat: add Gitea support to repo_providers and include Redis in broadc…
3467931 
…asters
@ariWeinberg
feat: update test workflow and refactor OPAL server/client configurat…
a616465 
…ions to use session matrix
@ariWeinberg
feat: update test workflows to trigger on master branch and add pytes…
7513058 
…t testing workflow
@ariWeinberg
feat: update Python version in pytest workflow to 3.10.12
fe84864
@ariWeinberg
fix: correct sleep duration in GitHub Actions to 30 seconds
b29e7d2
@ariWeinberg
fix: exclude ddtrace plugins from pytest execution in run.sh
430f07d
@ariWeinberg
feat: add environment specification for OPAL_pytest in pytest workflow
5507847
@ariWeinberg
refactor: remove unused return statement in create_gitea_policy_repo_…
eaa91d7 
…settings function
@ariWeinberg
feat: enhance wait_sometime function to provide countdown during slee…
5287be8 
…p in GitHub Actions
@ariWeinberg
refactor: return SSH keys as a dictionary in generate_ssh_key_pair fu…
a66d1e4 
…nction
@ariWeinberg
fix: add return statement in GithubPolicyRepo to ensure proper flow a…
e940dea 
…fter generating SSH keys
@ariWeinberg
feat: add SSH key support for OPAL policy repository in test settings
ea92dd6
@ariWeinberg
chore: update pytest logging level to DEBUG for enhanced output
3e2bb9b
@ariWeinberg
feat: update source_repo_owner environment variable to 'ariWeinberg' …
5088797 
…in multiple settings files and add logging for test session initialization
@ariWeinberg
feat: update setup fixture to load environment variables and add a du…
642a599 
…mmy test for pytest execution
@ariWeinberg
feat: update setup fixture to include opal_clients and remove dummy test
d0ca91f
@ariWeinberg
feat: add str2bool utility function for string to boolean conversion
013f00b
@ariWeinberg
feat: remove GITEA support from repo_providers and redis from broadca…
66f2a4a 
…sters in PyTestSessionSettings
@ariWeinberg
feat: add logging for policy repo creation and session matrix details
ab568bf
@ariWeinberg
feat: update logger name in policy_repos fixture for clarity
a263d08
@ariWeinberg
feat: add SSH keys to environment variables in pytest workflow
7427d3b
@ariWeinberg
feat: add additional environment variables for pytest workflow and up…
5717770 
…date .env.example
@ariWeinberg
feat: refactor GithubPolicyRepo by removing SSH key handling and clea…
15984ae 
…nup methods
@ariWeinberg
feat: refactor GithubPolicyRepo cleanup method and improve webhook se…
d0bfbb9 
…tup structure
@ariWeinberg ariWeinberg merged commit 88c99a4 into master Mar 30, 2025
1 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@ariWeinberg ariWeinberg

@izzywdev izzywdev

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ariWeinberg @izzywdev