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

Unable to set maintenance mode - Invalid time value #3811

Closed
2 tasks done
AzureInformatique opened this issue Sep 28, 2023 · 17 comments
Closed
2 tasks done

Unable to set maintenance mode - Invalid time value #3811

AzureInformatique opened this issue Sep 28, 2023 · 17 comments
Labels

Comments

@AzureInformatique
Copy link

⚠️ Please verify that this bug has NOT been raised before.

  • I checked and didn't find similar issue

🛡️ Security Policy

📝 Describe your problem

Hi,

When I try to add maintenance to the system regardless of the value inserted I get an Invalid time value error, do you have a solution?

Thanks a lot

📝 Error Message(s) or Log

Error

🐻 Uptime-Kuma Version

1.23.2

💻 Operating System and Arch

Docker

🌐 Browser

Chrome

🐋 Docker Version

Latest

🟩 NodeJS Version

No response

@JulienAubert9
Copy link

Hello,

We have same issue to create a new maintenance window (error with time value).

But on our side, we don't see the previous configured maintenance anymore, and they are working fine we can see them in the history. Strange...

@sysadminafterdark
Copy link

I can confirm that I am having this issue as well with status.sysadminafterdark.com. I'm running Amazon Linux 2023 with the latest Docker-CE and UptimeKuma release. I can confirm both the host and container time is correct and the timezone is defined as an environmental variable. It is also correct in Settings > General > Timezone. No matter which timezone I select, I receive the error "Invalid Time Value". I'm certain this may be a bug. Any news relating to this? Thanks!

@CommanderStorm
Copy link
Collaborator

We have not been able to reproduce this in any development setting.
That you have set an enviornment variable might be a clue. What have you set there? Looking at the environment variables we offer, I don't see one that interacts with time https://github.com/louislam/uptime-kuma/wiki/Environment-Variables

@sysadminafterdark
Copy link

sysadminafterdark commented Feb 12, 2024

Heya! Thanks for your reply! I have my entire launch script here with instructions to reproduce my environment: https://gitlab.sysadminafterdark.com/sysadminafterdark/afterdark-uptimekuma-launch-script

Can you spin up a quick instance and try to reproduce? I can also grab logs for you if need be, just let me know. Thanks!

@CommanderStorm
Copy link
Collaborator

CommanderStorm commented Feb 12, 2024

I can confirm both the host and container time is correct and the timezone is defined as an environmental variable.

Cannot reproduce this statement. The script you have shown does not have such variables.
What changes have you made afer this script?

@sysadminafterdark
Copy link

Whoops! You’re right. I forgot to push my changes to git last night while fiddling with things. I added

environment:

  • TZ=America/Detroit
    To both containers in my docker compose file because it WAS incorrect. If that is not the correct course of action, I can remove it once home and try again.

I can confirm the configuration hosted currently on my git still exhibits this issue - I was just trying to troubleshoot on my own before posting here. Thanks!

@CommanderStorm
Copy link
Collaborator

CommanderStorm commented Feb 12, 2024

Sorry, but really cannot reproduce..

Without TZ=America/Detroit

image
image

With TZ=America/Detroit

image
image

@CommanderStorm

This comment was marked as off-topic.

@sysadminafterdark
Copy link

Sorry, but really cannot reproduce..

Without TZ=America/Detroit
With TZ=America/Detroit

Let me bang on this a bit once I get home. Worst case, I’ll blow away the server and try anew. Since I did a bit of automation, this should only take me a few minutes to rebuild. Please stand by.

@sysadminafterdark
Copy link

sysadminafterdark commented Feb 12, 2024

Sorry, but really cannot reproduce..
Without TZ=America/Detroit
With TZ=America/Detroit

I just finished blowing away my instance and resetting things up. Everything now works as expected. That is VERY weird. I ran my git script as is without any TZ modification. @AzureInformatique and @JulienAubert9 - if you are still experiencing this issue, perhaps a fresh install is on the books. Please feel free to use my script located above if you need a jumpstart getting back online. I wonder what the common denominator between the three of us was that resulted in that popup. Thanks for your help @CommanderStorm!!!

@Blue-Gamer48

This comment was marked as off-topic.

@CodeMeZone

This comment was marked as resolved.

@CommanderStorm
Copy link
Collaborator

Without being able to reproduce this issue, we can't relally help here..
I don't know where this is coming from or how to get into this state..

Maybe (kind of doubt it, but anyway) related (because of timezone handling): https://github.com/Hexagon/croner/releases/tag/8.0.0

If somebody can somewhat-regularly reproduce this, please record a screencapture via https://gifcap.dev/ and post how you want to add a maintenance in this thread. ^^
Please also include:

  • what the timezone + time is inside/on the docker contaienr/host
  • what the timezone is set to inside the settings

@Hexagon
Copy link

Hexagon commented Feb 17, 2024

Older versions of croner has a bug related to time zones resolving to AKDT/AKST and AST. Could this be it? Fixed in croner 7.x and 8.x.

7.x is for node versions prior to 18.

@CommanderStorm
Copy link
Collaborator

@Hexagon
None of the reports have mentioned using AKDT/AKST/AST, only timezones like America/Detroit which are not affected by said cange

=> This is the reason I stated above that I am not quite convinced that this is the case…
=> recording a screen capture as mentioned above is the best way we can see possible differences in reproduction
=> hopefully making this reproducible

Copy link

We are clearing up our old help-issues and your issue has been open for 60 days with no activity.
If no comment is made and the stale label is not removed, this issue will be closed in 7 days.

@github-actions github-actions bot added the Stale label Apr 18, 2024
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Apr 25, 2024
@sassanix
Copy link

I made post here which I belive is the same issue: #4930

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

8 participants