Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Map download not working #12866

Closed
Azredder opened this issue Aug 27, 2024 · 10 comments
Closed

Map download not working #12866

Azredder opened this issue Aug 27, 2024 · 10 comments
Labels
Major Indicates the problem is very impactful, very noticable, no work around Urgent Something in production is not working and is very noticable - urgent mitigation needed

Comments

@Azredder
Copy link

Azredder commented Aug 27, 2024

Map downloads were working this past weekend (Aug 24, 2024). Today, not working. Debug is below.

Aug 26, 2024 8:12:02 PM org.triplea.game.client.HeadedGameRunner lambda$initializeClientSettingAndLogging$0
SEVERE: No enum constant games.strategy.engine.framework.map.download.DownloadFileDescription.MapCategory.Experimental
java.lang.IllegalArgumentException: No enum constant games.strategy.engine.framework.map.download.DownloadFileDescription.MapCategory.Experimental
	at java.base/java.lang.Enum.valueOf(Enum.java:240)
	at games.strategy.engine.framework.map.download.DownloadFileParser.optEnum(DownloadFileParser.java:89)
	at games.strategy.engine.framework.map.download.DownloadFileParser.lambda$parseImpl$0(DownloadFileParser.java:66)
	at java.base/java.util.stream.ForEachOps$ForEachOp$OfRef.accept(ForEachOps.java:183)
	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:195)
	at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1654)
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:484)
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:474)
	at java.base/java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:150)
	at java.base/java.util.stream.ForEachOps$ForEachOp$OfRef.evaluateSequential(ForEachOps.java:173)
	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.base/java.util.stream.ReferencePipeline.forEach(ReferencePipeline.java:497)
	at games.strategy.engine.framework.map.download.DownloadFileParser.parseImpl(DownloadFileParser.java:49)
	at games.strategy.engine.framework.map.download.DownloadFileParser.parse(DownloadFileParser.java:36)
	at games.strategy.engine.framework.map.download.ContentReader.downloadAndApplyAction(ContentReader.java:51)
	at games.strategy.engine.framework.map.download.ContentReader.download(ContentReader.java:39)
	at games.strategy.engine.framework.map.download.DownloadRunnable.download(DownloadRunnable.java:26)
	at games.strategy.engine.framework.map.listing.MapListingFetcher.lambda$getMapDownloadList$1(MapListingFetcher.java:39)
	at java.base/java.util.Optional.orElseGet(Optional.java:369)
	at games.strategy.engine.framework.map.listing.MapListingFetcher.getMapDownloadList(MapListingFetcher.java:39)
	at games.strategy.engine.framework.ui.background.BackgroundTaskRunner$1.doInBackground(BackgroundTaskRunner.java:108)
	at java.desktop/javax.swing.SwingWorker$1.call(SwingWorker.java:304)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.desktop/javax.swing.SwingWorker.run(SwingWorker.java:343)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:834)
@Azredder
Copy link
Author

TripleA version 2.5.22294

@TheDog-GH
Copy link
Contributor

@TheDog-GH TheDog-GH added Major Indicates the problem is very impactful, very noticable, no work around Urgent Something in production is not working and is very noticable - urgent mitigation needed labels Aug 27, 2024
@beelee1
Copy link
Contributor

beelee1 commented Aug 27, 2024

this is the same as #12859 Will leave them both open for now. I wonder if 14862 is when it changed ? https://github.com/triplea-game/triplea/releases/tag/2.7.14862

Maybe revert to 14861 for now ? @frigoref thoughts

@Cernelius should a temporary link be added to lobby taking people to the maps at git ? They can dl direct. Would need instructions too

@TheDog-GH
Copy link
Contributor

2.5 is also broke, so its the map server

@beelee1
Copy link
Contributor

beelee1 commented Aug 27, 2024

Will ping @RoiEXLab then. Idk if he can help or not as i don't think the lobby is his area of expertise but dan said he could restart bots anyway :)

I made a couple posts explaining how to dl directly from git for the site and the A&A one. Maybe someone can add that link to the lobby welcome message. https://www.axisandallies.org/forums/topic/41126/how-to-dl-triplea-maps-directly-from-github

@RoiEXLab
Copy link
Member

I believe this could be just a spelling error that #12869 should fix

@RoiEXLab
Copy link
Member

@beelee1 @Azredder Can you double check it works now?

@beelee1
Copy link
Contributor

beelee1 commented Aug 27, 2024

yay Roi :)

yes I can dl maps in 2.7, cant test 2.5 but i'm guessing so :)

Edit
Oh boy I merged Roger's yaml. I'm guessing that's where it came from. Said no conflicts squash and merge. Hmm ... maybe i shouldn't do those anymore or at least look closer

@beelee1
Copy link
Contributor

beelee1 commented Aug 27, 2024

@RoiEXLab sorry to bug again but should there be some sort of check added for the spelling for the yaml updates so this doesn't happen again ? It was all green when I merged it and the yamls are the only ones I do. Was kind of a mini catastrophe :)

Edit
I see how to fix it now though :) Thanks for saving the day :)

@RoiEXLab
Copy link
Member

@beelee1 Nono you have a point, there should be a check to prevent these kinds of failures

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Major Indicates the problem is very impactful, very noticable, no work around Urgent Something in production is not working and is very noticable - urgent mitigation needed
Projects
None yet
Development

No branches or pull requests

6 participants
@RoiEXLab @beelee1 @TheDog-GH @Azredder and others