-
Notifications
You must be signed in to change notification settings - Fork 123
Promoting a previously deleted test with the same name #48
Comments
Hi! Thanks for raising. Is my understanding correct that the issue you have identified is in the indeedeng/proctor-webapp-library repo ? Additionally if you are able to share more details on a repro for "not possible to promote a test once it's been deleted", we will definitely take a look at it. |
That's correct. I'd love to try it out in the new location but as I mentioned in the OP there does not appear to be any tagged releases. When I clone and follow the instructions on the master branch I get the following error when trying to start the app:
To reproduce the issue (in the archived version) you simply create a new test, promote to qa and production, delete the test from all 3, then recreate and repromote. When you try to promote it will fail because that test name is represented in the git history (albeit with a 'Deleting' commit message) so it tries to update the existing test (which doesn't exist) instead of creating it. It's possible this issue is fixed now because looking at the code in master it's using |
same problem! Did you solve it? |
Not the spring error when starting the app but I was able to work around the issue of recreating the test by doing it outside the proctor webapp. Far from ideal but I just recreated it manually and pushed to the git repo myself. |
Hi there! The issue with tagged releases and this repo lagging several commits behind has been solved. One fix it brings is for the spring error mentioned above. |
We've identified an issue where it's not possible to promote a test once it's been deleted. We believe the issue is here https://github.com/indeedeng/proctor-webapp-library/blob/master/src/main/java/com/indeed/proctor/common/ProctorPromoter.java#L118 where it's finding the last commit for that test even though it was a "Deleting A/B test" message in the git log. We could try to contribute a fix but it seems that this repo has now been integrated with the main proctor repo and there does not appear to be any new releases of that project since 2018. Any recommendations?
The text was updated successfully, but these errors were encountered: