Skip to content

Add test for user-defined env var persistence in APIcast operator#1052

Open
cathal-bailey wants to merge 2 commits into
3scale-qe:mainfrom
cathal-bailey:env-var-persistence-test
Open

Add test for user-defined env var persistence in APIcast operator#1052
cathal-bailey wants to merge 2 commits into
3scale-qe:mainfrom
cathal-bailey:env-var-persistence-test

Conversation

@cathal-bailey

@cathal-bailey cathal-bailey commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator
  • Adding a new test to check that user-defined environment variables remain after operator reconciliation cycles.
  • This test validates fix for https://issues.redhat.com/browse/THREESCALE-12224
  • APIcast operator update included in 2.16.4 GA
  • Test forces APIcast operator to trigger reconciliation by changing a supported env var (APICAST_LOG_LEVEL)

Comment thread testsuite/tests/apicast_operator/test_user_env_var.py Outdated
Comment thread testsuite/tests/apicast_operator/test_user_env_var.py Outdated
Comment thread testsuite/tests/apicast_operator/test_user_env_var.py Outdated
@mdujava mdujava added the to_backport Commits to be backported to stable branches label Jun 16, 2026
@cathal-bailey
cathal-bailey marked this pull request as ready for review June 23, 2026 07:52
Comment thread testsuite/tests/apicast_operator/test_user_env_var.py Outdated
Comment thread testsuite/tests/apicast_operator/test_user_env_var.py Outdated
…main after operator reconciliation cycles. This test validates fix for THREESCALE-12224

Removed nonpersistence, seperated test setup into pytest fixture, replacing sleep(30) with forced reconciliation trigger

wait_for() added to explicitly wait for operator reconciliation. Unnecessary logs removed.
@cathal-bailey
cathal-bailey force-pushed the env-var-persistence-test branch from e235b21 to ddacd25 Compare June 24, 2026 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

to_backport Commits to be backported to stable branches

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants