Skip to content

Conversation

@zhangyue1818
Copy link
Contributor

@zhangyue1818 zhangyue1818 commented Dec 24, 2025

The gpcheckresgroupv2impl script failed on segments when running gpconfig -c gp_resource_manager -v "group-v2".

Root cause:
The validation script tried to connect to localhost:5432 on each host to retrieve gp_resource_group_cgroup_parent. However, segment hosts don't run the master database - causing "Connection refused" errors.

Fix:

  • Retrieve gp_resource_group_cgroup_parent from master database in gpresgroup.py before dispatching validation commands;
  • Pass the cgroup_parent value to gpcheckresgroupv2impl via command line argument (--cgroup-parent);
  • Remove database connection logic from gpcheckresgroupv2impl;

Fixes #ISSUE_Number

What does this PR do?

Type of Change

  • Bug fix (non-breaking change)
  • New feature (non-breaking change)
  • Breaking change (fix or feature with breaking changes)
  • Documentation update

Breaking Changes

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Passed make installcheck
  • Passed make -C src/test installcheck-cbdb-parallel

Impact

Performance:

User-facing changes:

Dependencies:

Checklist

Additional Context

CI Skip Instructions


The gpcheckresgroupv2impl script failed on segments when running
`gpconfig -c gp_resource_manager -v "group-v2"`.

Root cause:
The validation script tried to connect to localhost:5432 on each host
to retrieve gp_resource_group_cgroup_parent. However, segment hosts
don't run the master database - causing "Connection refused" errors.

Fix:
- Retrieve gp_resource_group_cgroup_parent from master database in
  gpresgroup.py before dispatching validation commands;
- Pass the cgroup_parent value to gpcheckresgroupv2impl via command
  line argument (--cgroup-parent);
- Remove database connection logic from gpcheckresgroupv2impl;
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.

2 participants