Seagate/cortx-re

cortx-prvsnr submodule in cortx is pointing to pre-cortx-1.0 branch instead of main branch

Closed this issue · 13 comments

We have submodules configured for all component repositories in cortx repo. All other components are configured for main branch however for cortx-prvsnr it's pre-cortx-1,0 branch.

image

This may be because pre-cortx-1,0 is default branch for cortx-prvsnr repository.

stale commented

This issue/pull request has been marked as needs attention as it has been left pending without new activity for 4 days. Tagging @shailesh-vaidya for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.

@mukul-seagate11 Can we finalize on this.

@ypise, what decision are we planning for the provisioner landing branch as some action has to be taken to switch the same to main branch as its accessible to external community?

stale commented

This issue/pull request has been marked as needs attention as it has been left pending without new activity for 4 days. Tagging @shailesh-vaidya for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.

ypise commented

@mukul-seagate11 , @shailesh-vaidya
The submodule in cortx repo needs to be updated to point to the main branch of cortx-prvsnr.
It's not related to setting in cortx-prvsnr in GitHub.

@ypise, as currently the submodule for cortx-prvsnr is pointing to pre-cortx-1.0 branch as https://github.com/Seagate/cortx-prvsnr/blob/pre-cortx-1.0/.github/workflows/dispatch_submodule_update.yml
where it should point to the default branch i.e. main

ypise commented

@mukul-seagate11
It is main branch as configured in the link mentioned above. main being our actual landing branch, this configuration should hold good.

@shailesh-vaidya, can you comment on the same as seems our expectation is different as stated in thread?

@mukul-seagate11 If provisioner is using pre-cortx-1.0 as validation and main is landing branch then we need to debug and modify submodule configuration for cortx repo to sync main branch codebase.
e.g. Currently cortx-prvsnr main branch is at - 76c6303f86802c6cda7fe407955e71a4d86f02e6
and cortx repo provisioner submodule is poiting to - https://github.com/Seagate/cortx-prvsnr/tree/873969f4fc798e8e2009c60e586a203d49dc3c15

@gauravchaudhari02, can you look & correct accordingly for provisioner wokflow?

stale commented

This issue/pull request has been marked as needs attention as it has been left pending without new activity for 4 days. Tagging @shailesh-vaidya for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.

stale commented

This issue/pull request has been marked as needs attention as it has been left pending without new activity for 4 days. Tagging @shailesh-vaidya for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.

This issue is fixed now.