-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(apps/prod/tekton/configs): add task and trigger to publish files to fileserver from oci artifact #1302
base: main
Are you sure you want to change the base?
Conversation
… to fileserver from oci artifact Signed-off-by: wuhuizuo <[email protected]>
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Based on the pull request, the following key changes have been made:
There are no apparent potential problems in this pull request. However, here are some suggestions for fixing:
Overall, the changes seem to be fine, and it can be merged after addressing the suggestions. |
…ger names Signed-off-by: wuhuizuo <[email protected]>
The pull request adds a new task and trigger to publish files to a file server from an OCI artifact. The changes include adding a new task definition file, a new trigger template file, and updating the kustomization.yaml files to include these new files. One potential problem is that the default value for publisher-url in the task definition is set to the staging publisher mirror. This may not be appropriate for production use. It would be better to use a parameter or environment variable to set the publisher mirror URL based on the deployment environment. Another potential issue is that the task script uses a hardcoded path to the publisher-cli binary. This may not work if the binary is installed in a different location or if the task is run in a different operating system. It would be better to use a parameter or environment variable to set the path to the publisher-cli binary. To fix these issues, the following changes can be made:
Once these changes are made, the pull request can be approved and merged. |
Signed-off-by: wuhuizuo [email protected]