fix(ticdc): fix builder image

[wuhuizuo]

• add gcc into rocky linux based builder image
• fix skaffold-centos7.yaml file

Signed-off-by: wuhuizuo wuhuizuo@126.com

[ti-chi-bot]

I have already done a preliminary review for you, and I hope to help you do a better job.

Based on the Pull Request title and description, it appears that the changes are related to the builder image for the ticdc package. The two key changes are:

1. Adding gcc to the package dependencies in the Dockerfile.
2. Updating the skaffold-centos7.yaml file to use the correct Dockerfile.

These changes seem reasonable and necessary. However, there are a few potential issues that should be addressed:

1. The commit message could be more descriptive and explain why gcc was added.
2. The commit message should also explain why the skaffold-centos7.yaml file needed to be updated.
3. It is unclear if any testing has been done to ensure that these changes do not cause any issues.

Here are some suggestions for addressing these issues:

1. Update the commit message to include a brief explanation of why gcc was added, such as "Added gcc as a package dependency to enable building certain components of ticdc that require C code".
2. Update the commit message to explain why the skaffold-centos7.yaml file needed to be updated, such as "Updated the skaffold-centos7.yaml file to use the correct Dockerfile path after restructuring the repository".
3. Add a comment to the Pull Request asking the author if they have tested these changes and if they have seen any issues. If they have not tested the changes, suggest that they do so before merging the Pull Request.

Overall, the changes seem reasonable and necessary, but it is important to address these potential issues before merging the Pull Request.

[wuhuizuo]

/approve

[ti-chi-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: wuhuizuo

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [wuhuizuo]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[ti-chi-bot]

I have already done a preliminary review for you, and I hope to help you do a better job.

Based on the pull request title and description, the key changes are:

• Adding gcc into the rocky linux based builder image.
• Fixing the skaffold-centos7.yaml file.
• Updating the grep command to select files in dockerfiles/cd/builders.

There are no potential problems that can be identified from the pull request.

However, some suggestions for improvement are:

• Provide more context in the pull request description, such as why the changes were made and what problem they solve.
• Consider adding tests to ensure the changes are working as expected.
• Consider adding comments to the code to explain what the changes are doing and why.