golang-cloud-google split & review requests

[Mikel Olasagasti]

Hi all,

I created 17 review requests and a PR to split current
golang-cloud-google into modules. This is required in order to update
other packages required by Google's SDK.

The plan is:

1) have all this reviewed & approved
2) build all modules in bootstrap mode to avoid cyclic dependencies
and major issues in other packages. Building in bootstrap mode will
make no package to be affected by a module not being available.
3) build all modules in non-bootstrap mode
4) fix issues

Required reviews:

- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250108
golang-cloud-google-bigquery
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250109
golang-cloud-google-compute-metadata
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250110
golang-cloud-google-compute
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250111
golang-cloud-google-datacatalog
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250113
golang-cloud-google-datastore
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250114
golang-cloud-google-firestore
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250115
golang-cloud-google-iam
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250116
golang-cloud-google-kms
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250117
golang-cloud-google-logging
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250118
golang-cloud-google-longrunning
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250119
golang-cloud-google-monitoring
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250120
golang-cloud-google-osconfig
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250121
golang-cloud-google-pubsub
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250122
golang-cloud-google-secretmanager
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250123
golang-cloud-google-spanner
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250124
golang-cloud-google-storage
- https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2250125
golang-cloud-google-trace
- https://src.fedoraproject.org/rpms/golang-cloud-google/pull-request/4

I've been testing these specs and packages depending on them in the
following COPR:

https://copr.fedorainfracloud.org/coprs/mikelo2/google-cloud-ng/packages/

Current issues are:

- All the modules but compute & datastore build fine, these require
changes in genproto packages.
- golang-contrib-opencensus-exporter-stackdriver fails because genproto.
- golang-github-google-trillian requires
golang-contrib-opencensus-exporter-stackdriver to be fixed
- golang-google-grpc requires
golang-contrib-opencensus-exporter-stackdriver to be fixed
- golang-github-googlecloudplatform-guest-logging fails because genproto
- golang-gocloud fails because genproto.
- google-osconfig-agent fails because genproto
- golang-github-acme-lego FTBFS
- google-guest-agent FTBFS

I would like to push this asap *only* to rawhide and keep fixing /
updating required packages like the genproto stack.

Comments are welcome.

Regards,
Mikel

[W. Michael Petullo]

>  I created 17 review requests and a PR to split current
>  golang-cloud-google into modules. This is required in order to update
>  other packages required by Google's SDK.
>  
>  The plan is:
>  
>  1) have all this reviewed & approved
>  2) build all modules in bootstrap mode to avoid cyclic dependencies
>  and major issues in other packages. Building in bootstrap mode will
>  make no package to be affected by a module not being available.
>  3) build all modules in non-bootstrap mode
>  4) fix issues 
I reviewed and approved the packages, so I think you can soon make
progress on this.

-- 
Mike

:wq

[Mikel Olasagasti]

Hau idatzi du W. Michael Petullo (mike(a)flyn.org) erabiltzaileak (2023
aza. 18(a), lr. (21:24)):
>  > I created 17 review requests and a PR to split current
>  > golang-cloud-google into modules. This is required in order to update
>  > other packages required by Google's SDK.
>  >
>  > The plan is:
>  >
>  > 1) have all this reviewed & approved
>  > 2) build all modules in bootstrap mode to avoid cyclic dependencies
>  > and major issues in other packages. Building in bootstrap mode will
>  > make no package to be affected by a module not being available.
>  > 3) build all modules in non-bootstrap mode
>  > 4) fix issues
>
>  I reviewed and approved the packages, so I think you can soon make
>  progress on this. 
Thanks a lot Mike!

I just created all the repos but will proceed with imports and builds
tomorrow or during Monday.

Mikel

Thank you both very much for taking care of this!
I'm sorry I was not available a week ago to assist

On 18/11/2023 15:50, Mikel Olasagasti <mikel(a)olasagasti.info&gt; wrote:
> Hau idatzi du W. Michael Petullo (mike(a)flyn.org) erabiltzaileak (2023
> aza. 18(a), lr. (21:24)):
> >
> >> I created 17 review requests and a PR to split current
> >> golang-cloud-google into modules. This is required in order to update
> >> other packages required by Google's SDK.
> >>
> >> The plan is:
> >>
> >> 1) have all this reviewed & approved
> >> 2) build all modules in bootstrap mode to avoid cyclic dependencies
> >> and major issues in other packages. Building in bootstrap mode will
> >> make no package to be affected by a module not being available.
> >> 3) build all modules in non-bootstrap mode
> >> 4) fix issues
> >
> > I reviewed and approved the packages, so I think you can soon make
> > progress on this.
>
> Thanks a lot Mike!
>
> I just created all the repos but will proceed with imports and builds
> tomorrow or during Monday.
>
> Mikel
> --
> _______________________________________________
> golang mailing list -- golang(a)lists.fedoraproject.org
> To unsubscribe send an email to golang-leave(a)lists.fedoraproject.org
> Fedora Code of Conduct:
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives:
https://lists.fedoraproject.org/archives/list/[e-mail]
> Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
>