Skip to content

Conversation

iNFJZ
Copy link
Owner

@iNFJZ iNFJZ commented Jul 29, 2025

… stage to stop and remove all containers, networks, and volume

… stage to stop and remove all containers, networks, and volume
@iNFJZ iNFJZ merged commit c7aad21 into master Jul 29, 2025
2 checks passed
@iNFJZ iNFJZ deleted the fix/jenkinsfile branch July 29, 2025 03:48
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Bugbot free trial expires on August 1, 2025
Learn more in the Cursor dashboard.

) VALUES
(gen_random_uuid(), 'admin', 'Admin Root', '[email protected]', '+84901110001', 1, TRUE, '2025-07-17 08:54:20.754+07', '1990-01-01', 'Hà Nội', 'Super admin', 'Local', NULL, '2025-07-17 08:53:17.542+07', '2025-07-17 08:54:20.754+07', NULL),
(gen_random_uuid(), 'admin', 'Admin Root', '[email protected]', '0901110001', 1, TRUE, '2025-07-17 08:54:20.754+07', '1990-01-01', 'Hà Nội', 'Super admin', 'Local', NULL, '2025-07-17 08:53:17.542+07', '2025-07-17 08:54:20.754+07', NULL),
(gen_random_uuid(), 'user', 'Tien Dung', '[email protected]', '0901110001', 1, TRUE, '2025-07-17 08:54:20.754+07', '1990-01-01', 'Hà Nội', 'Super user', 'Local', NULL, '2025-07-17 08:53:17.542+07', '2025-07-17 08:54:20.754+07', NULL),
Copy link

Choose a reason for hiding this comment

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

Bug: Duplicate Phone Numbers in User Records

The 'admin' and 'user' ('Tien Dung') records are inserted with the same phone number '0901110001'. This creates a data integrity issue and could lead to unique constraint violations if the PhoneNumber field is unique.

Locations (1)
Fix in Cursor Fix in Web

// Remove dangling images
sh 'docker image prune -f || true'

echo "✅ Post-cleanup completed: All containers, networks, volumes, and dangling images removed"
Copy link

Choose a reason for hiding this comment

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

Bug: Jenkinsfile Cleans Up All Docker Resources

The Jenkinsfile uses overly broad Docker cleanup commands (docker stop $(docker ps -aq), docker rm $(docker ps -aq), docker network prune -f, docker volume prune -f, docker image prune -f) in the Pre-Cleanup, post-always, post-failure, and post-cleanup stages. These commands affect all Docker resources on the Jenkins agent, not just those related to the current project. This can cause service disruptions, outages, and potential data loss for other applications running on the same host.

Locations (2)
Fix in Cursor Fix in Web

iNFJZ added a commit that referenced this pull request Jul 29, 2025
Enhance Jenkinsfile for improved Docker management: Added pre-cleanup…
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