meshery / meshery-osm

Meshery Adapter for Open Service Mesh
https://layer5.io/meshery
Apache License 2.0
42 stars 47 forks source link

[CI] Update Multi Platform build and Release workflow #187

Closed theBeginner86 closed 2 years ago

theBeginner86 commented 2 years ago

Current Behavior

Currently, the multiplatform.yml file is sending unwanted build arguments to docker build process because of which release tag and commit_sha are not being passed over to the image and hence the version of meshery adapter are not visible on-hovering adapters chips on Meshery UI Dashboard.

Desired Behavior

This would be fixed when correct build args are passed to the docker build step inside multiplatform workfow. Required Args:

Check this commit for reference: https://github.com/meshery/meshery-consul/commit/446ad170c938c9bfe22e6e8710459d63395b9148

related to: https://github.com/meshery/meshery/issues/6245

Environment


Contributor Guides and Resources

welcome[bot] commented 2 years ago

Thanks for opening this issue. A contributor will be by to give feedback soon. In the meantime, please review the Layer5 Contributors' Welcome Guide and sure to join the community Slack.

keenson commented 2 years ago

Hi @theBeginner86, Please assign this issue to me. I would like to do this.

kamalbuilds commented 2 years ago

here you go @keenson βœ”οΈ

acald-creator commented 2 years ago

This issue can be closed at this time, due to this commit https://github.com/meshery/meshery-osm/commit/0a5bc4cfefb9ddd5e828a12e0b3e51600f5ede96