eclipse-symphony / symphony

Symphony project
MIT License
27 stars 17 forks source link

Pass correct request body in materialize provider #157

Closed RemindD closed 2 months ago

RemindD commented 2 months ago

Fix https://github.com/eclipse-symphony/symphony/issues/155

Tests:

msftcoderdjw commented 2 months ago

https://github.com/eclipse-symphony/symphony/blob/f8d82c7f4480ebe54cd2a6554df5cf3ca7841567/api/pkg/apis/v1alpha1/providers/stage/materialize/materialize.go#L164-L165

I think we need to also fix this line, use catalog, not catalog.Spec in json.Marshal.

RemindD commented 2 months ago

@Haishi2016 gentle ping for review.