These are chat archives for bluefin-octopusdeploy/chrome-extension

27th
Sep 2016
Matt Richardson
@matt-richardson
Sep 27 2016 04:17 UTC
@davidroberts63 - hope the merge wasn't too painful!
David Roberts
@davidroberts63
Sep 27 2016 04:18 UTC
Merged just fine. Actually just finished verifying it all works as intended.
Matt Richardson
@matt-richardson
Sep 27 2016 04:18 UTC
Cool.
it seems happy?
David Roberts
@davidroberts63
Sep 27 2016 04:18 UTC
Aslo, I figured out how to get multiple versions of Octopus Deploy onto a single VM.
Matt Richardson
@matt-richardson
Sep 27 2016 04:18 UTC
Use docker? ;)
David Roberts
@davidroberts63
Sep 27 2016 04:18 UTC
Yes. 3.3.0 and 3.49 both check out.
Nope.
MSIs, registry exports and a diff viewer.
It's hacky, but I basically make msiexec think that Octopus has not been installed.
Matt Richardson
@matt-richardson
Sep 27 2016 04:19 UTC
Cool
David Roberts
@davidroberts63
Sep 27 2016 04:20 UTC
So I could put as many as I can fit on a single machine if I want.
Matt Richardson
@matt-richardson
Sep 27 2016 04:20 UTC
Whats the latest on merging in the ui tests?
David Roberts
@davidroberts63
Sep 27 2016 04:20 UTC
The Tentacle will be a bit more touchy but since they are all basically the same thing jsut with different Calimari I think it'll be okay.
I'll merge those in I hope this week. Vacation two weeks ago, and prepping for upgrade to Octopus 3 this weekend at work. So a bit busy.
Matt Richardson
@matt-richardson
Sep 27 2016 04:21 UTC
heh - understandable
David Roberts
@davidroberts63
Sep 27 2016 04:22 UTC
I'm looking at some PRs from July and I feel kind of bad because they are simple ones.
Matt Richardson
@matt-richardson
Sep 27 2016 04:22 UTC
As I was getting these fixes in, I was really thinking that they needed UI tests to ensure they actually work on multiple versions
David Roberts
@davidroberts63
Sep 27 2016 04:22 UTC
But at the moment focusing on the fixes you made and probably the template library since that'll be changing soon I imagine.
Right.
Matt Richardson
@matt-richardson
Sep 27 2016 04:23 UTC
Yeah - template library will be a big change
I think it might be a if (version < X) return; one again
in theory, it'll do everything that the BlueFin one does
(Haven't seen the latest for a bit)
David Roberts
@davidroberts63
Sep 27 2016 04:24 UTC
I'm likely going to switch to Atlantic.net. Looks like I can get a workable server for about $30/mo if on 24/7 or just under a nickel an hour.
OH. That? I thought it was just an API change on your website.
So no, I didn't hear about it being put inside Octopus Deploy.
Matt Richardson
@matt-richardson
Sep 27 2016 04:25 UTC
Yeah, full on integration - bring library steps front and center
David Roberts
@davidroberts63
Sep 27 2016 04:26 UTC
That. Would be awesome.
The one thing I haven't added (which I'll now just wait) is search within that list.
Matt Richardson
@matt-richardson
Sep 27 2016 04:28 UTC
You going to have time to look at merging bluefin-octopusdeploy/chrome-extension#73 as well?
David Roberts
@davidroberts63
Sep 27 2016 04:30 UTC
Yeah. Looking at it right now.
Matt Richardson
@matt-richardson
Sep 27 2016 04:30 UTC
cool
(just reworded it - didn't want to come across as pushy :smile:)
I'm probably going to tackle bluefin-octopusdeploy/chrome-extension#37 next
David Roberts
@davidroberts63
Sep 27 2016 04:33 UTC

One thing I was going to suggest (not a big deal just a 'functional' idea) was with the commonpygmy.isNewerVersionThan function. Splitting into two such that:

commonpygmy.isOctopusVersion("x.y.z").newerThan("3.4.0")

Reads a bit better and splits the parsing from the comparison.

You'll have to excuse me, I've been on a 'functional programming' kick for a while.
Matt Richardson
@matt-richardson
Sep 27 2016 04:36 UTC
Would be nice - agreed.
I was actually thinking that it would be better to just keep the current version in a central spot anyway
rather than passing it around everywhere
David Roberts
@davidroberts63
Sep 27 2016 04:37 UTC
Yeah. I like that idea. Then just do the isNewerThan("1.2.3") and it checks against the central one.
And the analytics could use that instead of what it's doing now. How it's doing it is just...messy. Your's is much nicer.
And yeah, we need that icon. With th enumber of people mistaking Bluefin stuff for Octopus stuff lately.
I want to reach out to a designer for a refresh on the logo.
Matt Richardson
@matt-richardson
Sep 27 2016 04:43 UTC
I might see if Jess (our in house designer) can assist
David Roberts
@davidroberts63
Sep 27 2016 04:43 UTC
O.O
I would appreciate that. I have no problem paying for it. Just need to find someone I think will do a good job of it. Had been waiting on a local fellow but they have not gotten back to me about it.
Matt Richardson
@matt-richardson
Sep 27 2016 04:56 UTC
next time I see her online, I'll ping her
doubt it would be a problem
David Roberts
@davidroberts63
Sep 27 2016 04:56 UTC
Cool.
Hey, is 'Deployed to channel' the right term to use in #73 ?
Matt Richardson
@matt-richardson
Sep 27 2016 04:58 UTC
I think so...?
I did play around with a few other ways of displaying it, but this one seemed to work best
David Roberts
@davidroberts63
Sep 27 2016 04:59 UTC
I know I'm being pendantic. But it's not 'deployed to a channel' right? The process of channel X gets deployed.
Yeah. I'm not having luck coming up with something that works better.
Matt Richardson
@matt-richardson
Sep 27 2016 05:00 UTC
Deployed to channel: Beta
Deployed to channel: 2.0
works alright i think
lemme double check
David Roberts
@davidroberts63
Sep 27 2016 05:01 UTC
Maybe just:
'Deployment process for release 0.0.2 channel beta'
Or just 'Channel: beta' or nothing if no channel applies.
Matt Richardson
@matt-richardson
Sep 27 2016 05:04 UTC
that last option would be my preference if it is to change
David Roberts
@davidroberts63
Sep 27 2016 05:05 UTC
Yeah. Let's do that.
It works great. Just been testing it.
Matt Richardson
@matt-richardson
Sep 27 2016 05:06 UTC
I had had vague intentions of putting it into past tense
so that it emphasises its the state at that time
rather than current
but...
the release could have been created
but not deployed
so its not actually accurate
so your suggestion is probably the best bet
one thing to note... the step numbers do get out of sequential order, but I decided that was a feature rather than a bug
shows that steps were skipped
David Roberts
@davidroberts63
Sep 27 2016 05:09 UTC
Oh, yeah. Makes sense.
LIkely used to it anyway if they skip steps or have steps only applying to a particular environment anyway.
Matt Richardson
@matt-richardson
Sep 27 2016 05:10 UTC
briefly pondered whether to just gray out the step instead, but decided to leave it as it is now
David Roberts
@davidroberts63
Sep 27 2016 05:11 UTC
Yeah I'd leave it as removed.
Okay, heading to bed. I'll merge #73 tomorrow and publish. Thanks again for your help Matt.
Matt Richardson
@matt-richardson
Sep 27 2016 05:19 UTC
Cool. Thanks for merging!