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.

Sign up for GitHub

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

Jump to bottom

Migrate e2e test for InfluxDB to Go #3237

Closed
Tracked by #2737
tomkerkhove opened this issue Jun 22, 2022 · 8 comments · Fixed by #3451
Closed
Tracked by #2737

Migrate e2e test for InfluxDB to Go #3237

tomkerkhove opened this issue Jun 22, 2022 · 8 comments · Fixed by #3451
Assignees
@Ritikaa96
Labels
enhancement New feature or request help wanted Looking for support from community testing

Comments

@tomkerkhove
Copy link
Member

Proposal

Relates to #2357, #2737.

Use-Case

No response

Anything else?

No response
@tomkerkhove tomkerkhove added needs-discussion feature-request All issues for new features that have not been committed to labels Jun 22, 2022
@tomkerkhove tomkerkhove moved this to Proposed in Roadmap - KEDA Core Jun 22, 2022
@tomkerkhove tomkerkhove mentioned this issue Jun 22, 2022
Closed
48 tasks
@tomkerkhove tomkerkhove added testing enhancement New feature or request help wanted Looking for support from community and removed needs-discussion feature-request All issues for new features that have not been committed to labels Jun 22, 2022
@tomkerkhove tomkerkhove moved this from Proposed to To Do in Roadmap - KEDA Core Jun 22, 2022
@Ritikaa96
Copy link
Contributor

Hi @tomkerkhove, I'll take this up if no one's working on it already

@JorTurFer JorTurFer self-assigned this Jul 19, 2022
@JorTurFer
Copy link
Member

@Ritikaa96 , are you on this? if not, I'll tackle it

@JorTurFer JorTurFer removed their assignment Jul 19, 2022
@Ritikaa96
Copy link
Contributor

Ritikaa96 commented Jul 26, 2022 •
edited
Loading

@Ritikaa96 , are you on this? if not, I'll tackle it

Hey @JorTurFer , Sorry for missing out on your comment, yes I'm working on this one.

@JorTurFer
Copy link
Member

Do you think that you could add the activation feature too? We plan to do a release aiming to the end of next week and we'd like to have that feature implemented. If you don't have time, it's not a problem at all, no pressure from our side, we can tackle it before the release if next week you haven't had time 😄

@Ritikaa96
Copy link
Contributor

are you talking about this one: #2800 ? I might need a little reference but I'll definitely try adding this feature. if you have any other references please let me know.

@JorTurFer
Copy link
Member

hey @Ritikaa96 ,
Sure, these are some samples:
#3408
#3405
#3402

@JorTurFer
Copy link
Member

Once you have migrated the e2e test, the changes are quite easy :)

@Ritikaa96
Copy link
Contributor

Once you have migrated the e2e test, the changes are quite easy :)

Thanks @JorTurFer , i'm on it.

@Ritikaa96 Ritikaa96 mentioned this issue Jul 30, 2022
Merged
2 tasks
Repository owner moved this from To Do to Ready To Ship in Roadmap - KEDA Core Aug 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ritikaa96 Ritikaa96

Labels
enhancement New feature or request help wanted Looking for support from community testing
Projects
Roadmap - KEDA Core
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

migrating influxdb e2e to go Ritikaa96/keda
3 participants
@tomkerkhove @JorTurFer @Ritikaa96