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

[revogi] Initial contribution - Resubmitted for OH3 #8534

Merged
merged 38 commits into from Nov 3, 2020

Conversation

andibraeu
Copy link
Contributor

Signed-off-by: Andreas Bräu ab@andi95.de

Revogi Binding

Due to the move to OH3 I transformed the binding, and it's now based on the OH3 main branch. The previous PR was #7029

This PR will add a new binding for Revogi's Smart Strip (SOW019). I added a thread at https://community.openhab.org/t/binding-for-revogi-smart-strip-sow019/92923

Description

If you use the Smart Strip with the Revogi Home App, you always need internet access to use this device. With this binding you can control the Smart Strip without the need of a cloud based solution in the internet.

The binding supports to switch every 6 plugs separately, retrieve statistics for each plug or switch all 6 plugs together. All of this works by sending UDP packets.

Testing

Let's wait for a first build, then I can provide a link to the built jar!

Signed-off-by: Andreas Bräu <ab@andi95.de>
@TravisBuddy
Copy link

TravisBuddy commented Sep 21, 2020

Hey @andibraeu,
Something went wrong with the build.

TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.

View build log

TravisBuddy Request Identifier: 8da1e560-14ac-11eb-aff7-dd8cbeb067cc

Signed-off-by: Andreas Bräu <ab@andi95.de>
@andibraeu
Copy link
Contributor Author

BTW: the Jenkins build doesn't seem to checkout the PR branch. It builds for another commit and these changes aren't included. The build is failing because of another binding.

@kaikreuzer, I'm not sure if you're the right person to address this.

@kaikreuzer kaikreuzer added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Sep 22, 2020
@kaikreuzer
Copy link
Member

All checks green!

@andibraeu
Copy link
Contributor Author

Thank you!
But in the jenkins build my binding doesn't appear: https://ci.openhab.org/job/PR-openHAB-Addons/222/
Is there anything configured wrong on my side?

@kaikreuzer
Copy link
Member

That's indeed weird. I have changed the build configuration now in the hope that it works as expected - let's have a try and see!

@kaikreuzer kaikreuzer added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Sep 22, 2020
@andibraeu
Copy link
Contributor Author

nice, now it worked!

@kaikreuzer
Copy link
Member

Are you sure? Why did it fail with static analysis errors on Astro tests...?

@andibraeu
Copy link
Contributor Author

oh I see.. maybe I was a bit too fast.

will my PR be applied to the HEAD of this branch during the build? Or is it build based on the commit I used when I created my branch?

@kaikreuzer
Copy link
Member

I've now cleaned the workspace of the build job and will try again.

will my PR be applied to the HEAD of this branch during the build?

Yes, it is merged with the head of the main branch.

@kaikreuzer kaikreuzer added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Sep 22, 2020
@Hilbrand Hilbrand added the new binding If someone has started to work on a binding. For a new binding PR. label Sep 22, 2020
@andibraeu
Copy link
Contributor Author

Unfortunately it still fails because of the Astro Binding Integration tests

@kaikreuzer
Copy link
Member

On other PRs, this has meanwhile disappeared. I'll try one more...

@kaikreuzer kaikreuzer added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Sep 23, 2020
@andibraeu
Copy link
Contributor Author

Now we've got a successful build, but it was aborted in the end:

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  02:56 h
[INFO] Finished at: 2020-09-23T17:54:11Z
[INFO] ------------------------------------------------------------------------
Build timed out (after 180 minutes). Marking the build as aborted.

@Hilbrand
Copy link
Member

On the older pr build @wborn had recently increased the timeout to 4 hours because a lot of pr builds did timeout.

@wborn wborn added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Sep 24, 2020
@wborn
Copy link
Member

wborn commented Sep 24, 2020

I've increased it on this build job as well.

@fwolter fwolter self-assigned this Oct 4, 2020
Signed-off-by: Andreas Bräu <ab@andi95.de>
Signed-off-by: Andreas Bräu <ab@andi95.de>
Copy link
Member

@fwolter fwolter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some checkstyle warnings left. You could take a look at target/code-analysis/report.html.

andibraeu and others added 8 commits October 26, 2020 21:24
Signed-off-by: Andreas Bräu <ab@andi95.de>
Signed-off-by: Andreas Bräu <ab@andi95.de>
Signed-off-by: Andreas Bräu <ab@andi95.de>
Signed-off-by: Andreas Bräu <ab@andi95.de>
Signed-off-by: Andreas Bräu <ab@andi95.de>
Signed-off-by: Andreas Bräu <ab@andi95.de>
Signed-off-by: Andreas Bräu <ab@andi95.de>
Copy link
Member

@fwolter fwolter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Signed-off-by: Andreas Bräu <ab@andi95.de>
@andibraeu
Copy link
Contributor Author

Signed-off-by: Andreas Bräu <ab@andi95.de>
Copy link
Contributor

@cpmeister cpmeister left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only a few minor changes and I think we are good to go. I'll merge as soon as it is done.

Signed-off-by: Andreas Bräu <ab@andi95.de>
Signed-off-by: Andreas Bräu <ab@andi95.de>
Signed-off-by: Andreas Bräu <ab@andi95.de>
Signed-off-by: Andreas Bräu <ab@andi95.de>
@cpmeister cpmeister merged commit 1c93eb9 into openhab:main Nov 3, 2020
@cpmeister cpmeister added this to the 3.0.0.M3 milestone Nov 3, 2020
@andibraeu andibraeu deleted the revogismartstrip-oh3 branch November 3, 2020 21:20
nowaterman pushed a commit to nowaterman/openhab-addons that referenced this pull request Jan 19, 2021
boehan pushed a commit to boehan/openhab-addons that referenced this pull request Apr 12, 2021
* Rename binding and resubmit to OH3

Signed-off-by: Andreas Bräu <ab@andi95.de>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
* Rename binding and resubmit to OH3

Signed-off-by: Andreas Bräu <ab@andi95.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new binding If someone has started to work on a binding. For a new binding PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants