Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 19:22
    Pixcell opened #1729
  • 17:24
    sophokles73 commented #1672
  • 17:22
    sophokles73 milestoned #1672
  • 16:19
    kaniyan opened #1728
  • 15:12
    kaniyan synchronize #1722
  • 15:07
    ko-j commented #1724
  • 15:01

    dejanb on master

    [#1346] Add missed tag status s… (compare)

  • 14:42
    calohmn commented #1683
  • 14:14

    dejanb on master

    [#1346] Remove event bus from d… (compare)

  • 14:14
    dejanb closed #1672
  • 13:47
    codecov[bot] commented #1672
  • 13:46
    dejanb synchronize #1672
  • 13:19
    kaniyan synchronize #1722
  • 12:47
    dejanb synchronize #1672
  • 12:42
    sophokles73 commented #1722
  • 12:41
    kaniyan synchronize #1722
  • 12:38
    kaniyan commented #1722
  • 12:34
    kaniyan synchronize #1722
  • 11:40
    dejanb synchronize #1672
  • 10:11
    b-abel commented #1724
Jens Reimann
@ctron
Yes, I did that … everyone voted :D
it is today … 16:00 CET
Kai Hudalla
@sophokles73
Except for me, because I didn't follow the issue closely enough?
Jens Reimann
@ctron
sorry, that may be … I only know that Carsten can't join today, he is out of office IIRC
there is also a calendar with the invite
I did however forget to add this to the homepage.
Kai Hudalla
@sophokles73
no worries, found it ...
Kai Hudalla
@sophokles73
@dejanb Hi Dejan, have you been able to take a look at #1647 ?
Dejan Bosanac
@dejanb
@sophokles73 just briefly ... I plan to work on it tomorrow
Kai Hudalla
@sophokles73
cool
Kai Hudalla
@sophokles73
@ctron @dejanb @calohmn It looks like there is only a single place where we still use code from Google Guava: some toString() methods in the Device Registry helper classes. I wonder if we can remove that dependency altogether from Hono ... WDYT?
Jens Reimann
@ctron
hm … while I like the "ToStringHelper" stuff, guava's dependencies are sometimes tricky, so getting rid of it might be a good idea :+1:
that would be a change for 1.1?
Kai Hudalla
@sophokles73
I guess so. Guava is not part of any of our public APIs AFAIK
Dejan Bosanac
@dejanb
+1
Carsten Lohmann
@calohmn
I'm also for removing it then.
Kai Hudalla
@sophokles73
I will create an issue for it then and assign it to @ctron !?
Dejan Bosanac
@dejanb
@sophokles73 how do you wanna proceed with #1647
I have some fixes, but I doubt I can push them to bsinno
Kai Hudalla
@sophokles73
I can push the branch to eclipse/hono as well. We can then work on it together an create a PR from it. Would that work for you?
Dejan Bosanac
@dejanb
yeah ... sounds perfect
Kai Hudalla
@sophokles73
ok, gimme a minute ...
Dejan Bosanac
@dejanb
so basically I have helm chart running (at least getting started looks ok) and systems tests run but I saw a couple of failures
would love to push this to see how it looks in CI and then continue tweaking it
Kai Hudalla
@sophokles73
ok, I have pushed the branch to eclipse/hono ...
Dejan Bosanac
@dejanb
cheers
Dejan Bosanac
@dejanb
new pr is #1651 as I don't think we can edit the old one
I also rebased the branch, so now is mergable
Kai Hudalla
@sophokles73
@ctron would you mind taking a look at #1654 ? It is supposed to be part of 1.0.2 which we want to release this week ...
Jens Reimann
@ctron
done
Kai Hudalla
@sophokles73
@ctron thanks, man :-)
Kai Hudalla
@sophokles73
@ctron @calohmn @dejanb Today is the day we have scheduled 1.1.0-M1 for :-) Anything you got cooking that should be in this milestone in addition to what we have in the "done" column of our 1.1.0 project board?
Carsten Lohmann
@calohmn
From my side that would be #1654. @ctron I've pushed a change to that PR yesterday. Could you have another look?
Jens Reimann
@ctron
@calohmn done … sorry, I assumed you would simply add a comment and merge :grinning:
Dejan Bosanac
@dejanb
@sophokles73 I would like to take a look at the artemis logs
when are you planning to cut it?
Kai Hudalla
@sophokles73
@dejanb No particular plan. What about 5PM?
Dejan Bosanac
@dejanb
@sophokles73 sure ... I'm looking into this now
Kai Hudalla
@sophokles73
I would like to change the way we handled the release notes for milestones in the past. We used to add a heading for the milestone in the release notes, basically treating it as a separate release. However, the Eclipse Foundation doesn't consider a milestone to be a release and neither should we. Base on that, I propose to simply keep on adding text to the release notes for 1.1.0 and include it in the announcement mail I send out for the milestone. This way people still can see what has changed in the milestone but we do not need to create a project board for the milestone(s) which we can reference by URL. WDYT?
Dejan Bosanac
@dejanb
+1
Karthees Kalidass
@kaniyan
+1
Abel Büchner-Mihaljević
@b-abel
+1
Jens Reimann
@ctron
+1
Carsten Lohmann
@calohmn
+1
Trystram Jean-Baptiste
@jbtrystram
+1
Kai Hudalla
@sophokles73
ok everybody, I am now preparing the milestone build and will then push the button ...
Carsten Lohmann
@calohmn
It looks like #1645 takes longer than expected. Therefore I would suggest postponing the 1.0.2 build to end of the week at least. WDYT?
Kai Hudalla
@sophokles73
It seems like a fix to have in 1.0.2 so I am in favor or trying to get it into it. If that means delaying the release a little, I am fine with that ...
Kai Hudalla
@sophokles73
@calohmn are you able to successfully compile and test master locally? I can, but Travis and Jenkins seem to have a problem with it ...
Dejan Bosanac
@dejanb
@sophokles73 I tried it now ... and see the failure