Skip to content

FISH-13354: create simple deployment group deployment & redeployment tests (Payara7)#8080

Open
astoica1 wants to merge 19 commits intopayara:mainfrom
astoica1:FISH-13354-Create-Simple-Deployment-Group-Deployment-tests
Open

FISH-13354: create simple deployment group deployment & redeployment tests (Payara7)#8080
astoica1 wants to merge 19 commits intopayara:mainfrom
astoica1:FISH-13354-Create-Simple-Deployment-Group-Deployment-tests

Conversation

@astoica1
Copy link
Copy Markdown
Contributor

Create simple deployment group deployment redeployment tests

Description

This PR adds 4 new tests targeting deployment groups:

  • test_deploy_to_group_appears_on_all_instances
  • test_deploy_to_group_listed_on_group_target
  • test_undeploy_from_group_removes_from_all_instances
  • test_redeploy_to_group_propagates_to_all_instances

Important Info

Blockers

None

Testing

New tests added

  • test_deploy_to_group_appears_on_all_instances
  • test_deploy_to_group_listed_on_group_target
  • test_undeploy_from_group_removes_from_all_instances
  • test_redeploy_to_group_propagates_to_all_instances

Testing Performed

pip install -r requirements.txt 
export PAYARA_HOME=/path/to/payara6
pytest test_deployment_group_deployment.py -v

Testing Environment

MacOS, JDK11, Latest Payara6 Enterprise head

Documentation

See README.md file

Copy link
Copy Markdown
Contributor

@simonladen simonladen left a comment

Choose a reason for hiding this comment

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

Tested locally

Copy link
Copy Markdown
Member

@Pandrex247 Pandrex247 left a comment

Choose a reason for hiding this comment

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

Missing an entry in the Jenkinsfile - this test isn't actually being run

Comment thread appserver/tests/functional/test-apps/clusterjsp.war
@astoica1 astoica1 requested a review from Pandrex247 April 20, 2026 07:52
Comment thread Jenkinsfile
Comment on lines +403 to +405
sh """curl https://bootstrap.pypa.io/get-pip.py -o get-pip.py"""
sh """python3 get-pip.py --user"""
sh """$HOME/.local/bin/pip install -r appserver/tests/functional/deployment-groups/requirements.txt"""
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I dislike this being a part of the Jenkinsfile
It should be a part of the image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This would need another task be created.

Comment thread appserver/tests/functional/deployment-groups/test_deployment_group.py Outdated
Comment thread appserver/tests/functional/deployment-groups/test_deployment_group.py Outdated
Comment thread appserver/tests/functional/deployment-groups/test_deployment_group.py Outdated
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.

4 participants