WISE-Developers / WISE_Application

WISE_Application formerly known as WISE
GNU Affero General Public License v3.0
7 stars 2 forks source link

[WISE Bug]: Build Fails compiling protobuf in FBP #53

Closed spydmobile closed 5 months ago

spydmobile commented 6 months ago

Contact Details

spydmobile@gmail.com

What happened?

The bad news is that after updating the expired personal access token for the automated build process, WISE is having problems compiling protobuf in FBP and so the build is failing

image

image

Version

(Ubuntu 2022) v1.0.0-beta

What component are you seeing the problem on?

Other (indicate in "What happened" above)

Relevant log output

see:
https://github.com/WISE-Developers/WISE_Application/actions/runs/8214421182/job/22466899027

Approvals Process

spydmobile commented 6 months ago

@BadgerOnABike @nealmcloughlin this is our current show stopper....

spydmobile commented 5 months ago

@RobBryce @tredpath double check this is not simply your PAT for HSS expired.

spydmobile commented 5 months ago

@tredpath @RobBryce ok, now the problems have moved up the list: https://github.com/WISE-Developers/WISE_Application/actions/runs/8298852792/job/22713190655#step:86:150

fixing this issue like this will take forever.

I am authorizing HSS to go ahead and merge your own PRs to fix this issue, once you have this building we can go back authorizing your PRs please.

tredpath commented 5 months ago

The build runs now. I had to downgrade the Paho C++ version to 1.2.0 (really it's the same version that WISE was released with previously). This is further back than I wanted to go, I had hoped going to 1.3.2 would be enough as it removes eclipse/paho.mqtt.cpp#374 which breaks the build here but there were further issues on Windows caused by the changes to message.h in this commit.

@spydmobile, do you know what permissions the PAT you used when updating the WISE_PAT variable had? The release stage isn't able to push tags to other projects. If it is a permissions issue the release stage of the last action run should be able to be rerun without redoing the entire build after it's fixed.

tredpath commented 5 months ago

Also, I updated the version to 1.0.6-beta.6 which hopefully works as a release version.

spydmobile commented 5 months ago

i have tried everything, i cant fix this. I have opened a ticket with Github.

https://support.github.com/ticket/personal/0/2678132

spydmobile commented 5 months ago

ok @tredpath @ @RobBryce , heard back from github, this is because of new stricter PAT policies, I made a CLASSIC token instead with full powers and no expire and made that the new WISE_PAT in Wise application, and reran the release and it went, closing this.