Open Super8film87 opened 7 months ago
Sorry for the delay. I'll look at this after I've fixed #236
No worries at all. Thx for the support
Any idea?
Are you still running 2023.05.4 and tcWebhooks 1.2.5?
Yes we do - not sure when we will move the next version.
We have on Dev (2024.03) now
Is this the result of an actual build sending to the /webhooks/endpoint.html
? Or was it from a "test" from the template preview?
on endpoint-viewer:
{ "inpsectionError": "$InspectionStatsE_value", "inpsectionWarnings": "$InspectionStatsW_value" }
no to /webhooks/endpoint.html
I am wondering if inspections are assembled after the build completes, and perhaps are not available at the time that the webhook runs. None of my test builds appear to have inspections on them in the REST API or in webhook payload. I am not using a jetbrains inspections tool. Mine are just standard maven builds.
Hm could be a possible. For unit tests it is working - I'll check with a small example.
Created an issue here
Thanks
From the issue above:
Hello folks,
The value for InspectionStatsE and InspectionStatsW parameters is calculated and published during TeamCity's event buildFinished().
The same listener is used for WebHookListener and other listeners.
It could happen that plugin's buildFinished() is exectuted before TeamCity's processing of InspectionStats* metrics.
The solution could be to implement PositionAware interface in WebHookListener and set constraint to PositionConstraint.last(), so the web hook is executed after all changes made by TeamCity itself. @Net Wolf could you do this change in the plugin?
Kind regards,
KIR
can you provide feedback?
I'm so sorry I missed that comment. I'm looking at this today. I'll get a build ready for you to test by tomorrow.
I have implemented the change as suggested by Kir. Can you please test a build from this branch?
You can just login as guest download the zip.
I've built this from version 2.0.1. Are you still on 1.2.5? If so, I'll need to do a new build for you.
We are using 2.0.1 => will come to with feedback. Thx for this fast feedback.
Sorry for the delay -
using: /plugins/tcWebHooks-2.0.1-build.506.zip
same issue to report. I dont see it correct in the endpointviewer :(
did already add the change proposed by jet brains?
The change ended up being a significant amount of work. Can you please see https://github.com/tcplugins/tcWebHooks/wiki/Waiting-for-Build-Statistics-to-be-published and download the latest build from issue_235-missing_inspections
I have pushed 2.0.2-rc.2 The build is now available at issue_235-missing_inspections
This fixes an issue where it would take an hour to time out when not all build statistics had been received even though the build had completed and everything else had worked correctly.
Now has another setting that allows controlling the buildStatisticsBuildCompletedTimeout
either from the main-config.xml
or by overriding in a webhook parameter.
See updated docs at: Waiting-for-Build-Statistics-to-be-published
I've updated the docs with a diagram to better explain the timeline and which configuration settings affect the execution of the webhook.
Hey,
I've trouble to find a solution to get statics of InspectionStatsE & InspectionStatsW. If I check following url
I see the correct payload:
if I try to collect the data via velocity
but this only responding:
Additional test:
If I check on Preview Template Payload
I see correct payload
on endpoint-viewer:
Can you provide some guidance - #https://javadoc.jetbrains.net/teamcity/openapi/current/jetbrains/buildServer/serverSide/SBuild.html#getFullStatistics() Do I check the wrong commands?
TeamCity Server: TeamCity Enterprise 2023.05.4 (build 129421) Plugin: 1.2.5