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

fix statistics creation when creationTime is provided #357

Merged
merged 5 commits into from
May 6, 2022

Conversation

emilpaw
Copy link
Contributor

@emilpaw emilpaw commented Apr 24, 2022

This fixes the bug where the request to save yorc statistics fails when
a creationTime was provided in the generator-jhipster part of the
entry payload. The issue is that Instant.parse() was called with a
timestamp while it only accepts an ISO formatted instant.

Fix jhipster/generator-jhipster#18238

@emilpaw
Copy link
Contributor Author

emilpaw commented Apr 24, 2022

I haven't been able to make the tests work on my local machine yet. Could you approve the workflow so that I can use the pipeline for testing?

@emilpaw emilpaw force-pushed the fix-yorc-statistics-creation branch from 760d296 to 091c2cd Compare April 25, 2022 10:57
@emilpaw emilpaw changed the title add creationTimestamp field to yorc in test fix statistics creation when creationTime is provided Apr 25, 2022
This fixes the bug where the request to save yorc statistics fails when
a `creationTime` was provided in the `generator-jhipster` part of the
entry payload. The issue is that `Instant.parse()` was called with a
timestamp while it only accepts an ISO formatted instant.

Fix jhipster/generator-jhipster#18238
@emilpaw emilpaw force-pushed the fix-yorc-statistics-creation branch from 091c2cd to 256c781 Compare April 25, 2022 11:00
@emilpaw emilpaw marked this pull request as ready for review April 25, 2022 11:01
@atomfrede
Copy link
Member

@DanielFran @pascalgrimaud Should we merge this? Will it be auto deployed to jhipster online (guess not)?

@pascalgrimaud
Copy link
Member

the deploy is done only when there is a new tag
I didn't follow closely what is done here, so feel free to merge it :p

@DanielFran
Copy link
Member

I believe @pascalgrimaud needs to deploy a new version

@emilpaw
Copy link
Contributor Author

emilpaw commented May 6, 2022

@DanielFran This can be merged, right?

Then @pascalgrimaud could make a new tag to deploy it.

@DanielFran DanielFran merged commit 85222f0 into jhipster:main May 6, 2022
@DanielFran
Copy link
Member

@emilpaw Done.

Thanks for contribution and do not forget to claim the bug bounty

@emilpaw emilpaw deleted the fix-yorc-statistics-creation branch May 6, 2022 16:45
@emilpaw
Copy link
Contributor Author

emilpaw commented May 15, 2022

@pascalgrimaud Could you make a new tag? Since the statistics are not collected without this fix I think it's good to deploy it soon.

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

Successfully merging this pull request may close these issues.

JHipster Statistics have had a significant drop
4 participants