Need to use `php5enmod` to enable the `MCrypt` PHP module after installing the `php5-mcrypt` `apt` package.


harrd to read XML, but I don't know about a text-field.


commit message is too vague... please clearly explain why you are modifying pkgconfig.  For example:

    pkgconfig: update cflags install path

    We now share a single include installation
    directory for all libyami components.


hmmm... this is a lot more pep8 compliant after consulting the standard but I have a strong personal opinion which goes against this :P (too long have I worked with c++ vectors)
If it's better to do that then I've no objections but I'm going to have to train myself to do it the correct way. I'll make a single commit fixing up a few minor problems with the pep8-ness of my code.


For the host OS to get access to MySQL server on the VM.


I appreciate the TODO message. Sleep is definitely a thing.


Here should be only `in case -push` (no \` around `-push`).


What if age isn't a number? For example, if  I enter Apple. Then what?


This sentence is a bit confusing I think.


What String?


Ack to the approach. I'll do a quick scan of the remaining patches & add the ack label to the pull.


there if `g_list_foreach()` to iterate over a list. If you extract this part into its own function then this logic also gets its own name :)


Why is there an option? Are there some use-cases where users don’t want to install these scripts?


@jyuan please remove these commented out lines of code.  This URL should come from your config file.  That way locally you can set the URL to localhost and on the server we can set the URL to the proper domain.


Add extra Financial type to handle VAT (TAX)


You're probably correct but I was playing with different objects being passed to the exception. I'll fix this but generally given the speed of writing stuff to the stdio streams (and the time taken to throw exceptions).
I generally prefer to play it safe as I've encountered a number of times when I've had to use str rather than relying on just %s in the ganga codebase, although yes it should never be needed.


True


Can anyone familiar to with the code see why the change below breaks the test? @GaelVaroquaux @agramfort ?


Looks like we have yet another instance of FineObjectOrientedMess:
FilterStackObject implements some special handling for ObjectSourcePlayerPredicates with four-argument version of match(). TargetStackObject bypasses that by using two-argument version of the method instead. Result is that your custom predicate is not applied at all.

Other than that:
- Your "opponent controls" check is wrong at least in theory, because it assumes that everyone who is not you is an opponent. This is true now, but will not be if for example 2HG gets implemented. I believe you can just remove the check and add a ControllerPredicate to the filter.
- The target permanent may not be on the battlefield anymore. You need to use getPermanentLKIBattlefield() instead of getPermanent() and probably also null-check the result. Hydromorph Guardian also has this bug, it crashes if you try to counter a spell that targets the Guardian itself.


Add extra Financial type to handle VAT (tax)


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 809](https://teamcity.mesosphere.io/viewLog.html?buildId=58496&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:06:59


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 810](https://teamcity.mesosphere.io/viewLog.html?buildId=58501&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:07:34


Good stuff. Biggest notes: WHERE IS THE TEST FOR THE PRESENTER? Try and write the tests to drive your functional code. That's what TDD is all about. If you're have trouble, watch a ten minute youtube about J-Units and Mockito. Secondly, A lot of the names should be more descriptive. For example TABLE_NAME_2 ... TestActivity ... I should know what you mean by those even out of context, and I don't. Please leave me comments on my code as well, and also please respond to my comments above. If you think I am wrong please say so. There are infinite ways to write any code.


The impact is really short here since we always release core updates together on snapshots, then this part of the code will run during boot, before php-fpm starts. The error @fichtner pointed could be triggered when we release a php upgrade alone for example


Phaseid should have been removed for 335 push


you should probably not change the radius, you should only change the warning distance to INT.MAX_VALUE, and to reset it, you should set it to what the server has as warning distance (find a method to get the server current warning distance)


I've never seen this notation in syslog-ng's code (just for a note).


True, this could just be replaced with this I'm just not used to this particular pep8 style choice yet


@Argimon True. Please use !mustfix, because I will forget otherwise :-)


Is this really needed? I thought `StringUtil.isBlank` is `null` friendly.


Давай блок внутри блока не надо. Рядом поставь


Use stdint types please :). We want to work on 16 bit machines too :)


cc @blizzz @PVince81 I simply pushed to master ... it's working here ;)


use `do...end` for multiline blocks, please


deleted


Could you please add a not about which class you copied initially?


My bad, I just checked in Android Studio and they are not text fields.


I would need to catch and deal with quite a few possible exceptions here if it's not the general handler.
The code in _internal_job... needs to be fixed up to only throw certain exceptions in this case which involves being overly careful with every time we contact Dirac. I'll probably get around to this but we don't want to expose the user what are typically errors related to Dirac communication issues and the handling of the output.


Is that method available from it's original source?


Can we drop the `!` and make error message end with a period `.`?


Age should be int.


What was a purpose of this compatibility-breaking change?

ChefDK is still bundled with Berkshelf 3, and I cannot use latest version of your plugin until ChefDK is updated.
And because Vagrant does not provide any plugin version locking mechanisms, that became quite complex to manage.


Same question


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 811](https://teamcity.mesosphere.io/viewLog.html?buildId=58502&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:03:49


not good


!mustfix no \` around `-push`

OK. Mustfix added. I took it as a comment not review blocker, that's why I haven't used mustfix.


would you please provide a working sample script?!
Thank you.


I suspect that the test was passing only because the diabetes dataset was normalized (and not standardized). You will need to standardize it before passing it to lars_path.

As a side note, the fact that the diabetes dataset was standardized points us to how cultural it is to expect one versus the other. Some people will always be surprised, whatever choice we make.


Wait, wait, I'm talking about functional tests, tests extending `WebTestCase` and calling `static::createClient()`. This is what this is about. These don't work. Plain old unit tests work just fine. This commit removed my unsuccessful tries to create good working functional tests.


Sorry, I forgot about the sharing (and the wiki). I'll update accordingly.


test


nice!


some changes in file name


Should be fixed by c48d080e3b41cadb9f4dd5e5e9e08b63cd1d310e.


Well done, @kdaveed . Congratulations.


mettere nome diverso per ogni file che si crea
ex. /signs/Arena19472Arena_name.yml


@sugashing
test2


おや？マージに失敗したままpushしちゃったみたいだね。


Doesn't this need basic validations?


comprimere tutti i mondi della certella worlds in zipwrolds


If properties don't need validations, do we need this loop at all?


you can use `g_list_free_full()` for this case: https://developer.gnome.org/glib/stable/glib-Doubly-Linked-Lists.html#g-list-free-full


Как правило создание БД в тесты не ложиться


93653630-MultipleYearRunPeriodForTabularReports (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1905 of 1905 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Better to use mustfix more than less ;-) And it really should not be
there...

On Wed, Oct 14, 2015 at 2:28 PM Filip Bures <notifications@github.com>
wrote:

> !mustfix no ` around -push
>
> OK. Mustfix added. I took it as a comment not review blocker, that's why I
> haven't used mustfix.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/salsaflow/salsaflow/commit/aa0016c4512d95aa2a8b31397e0eec3f3ab04a89#commitcomment-13765517>
> .
>


@heimonsy 这里是跟业务强关联的，如果评论也需要 expire 的话，都放在 config 不如放在对应 controller


Oder wir nehmen irgendeine Namenskonvention, bei der die immer unten oder oben im Package ist. So das man unsere wichtigen Klassen spaeter zusammen haben.


this is just for a reminder, you know why :)


Excellent, that worked. Thanks!


Of course sprint doesn't work... They always have slightly different
hardware.
On Oct 13, 2015 12:20, "Drew Withers" <notifications@github.com> wrote:

> This commit appears to have entirely broken bluetooth on jfltespr in the
> 10/13 nightly.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CyanogenMod/android_device_samsung_jf-common/commit/ea58045f4f501355cd5100d039961c449af72974#commitcomment-13746741>
> .
>


[This one](https://github.com/scikit-learn/scikit-learn/commit/c183cf21af244d2e4a8c8a3c21d8f510209f43db#diff-8ff3792c5e86539e762dafe022c04f26R496) breaks as well @GaelVaroquaux . Same reason?


We used to provide options for such things. I'm now of the opinion that installing them unconditionally is probably a good idea unless it e.g. conflicts with a ZSH builtin completion (i.e. `git`)


This create breakage when the window in smaller than 997px.
What does it do? Is it to keep the padding between the two graphs to a minimum?
Will need to change to a class so that media in css can be used if so.


done

On Tue, Oct 13, 2015 at 10:37 PM, Clayton Coleman <notifications@github.com>
wrote:

> and all the other places in strings
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kargakis/origin/commit/6852d907759cba8301a0efde605b06e4447edb03#commitcomment-13751565>
> .
>


Can you explain why you made this change? :-)


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 812](https://teamcity.mesosphere.io/viewLog.html?buildId=58503&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 188 Build time: 00:04:08


Yeah, it was part of original commit, and got reverted together. Fixed. Thanks!


This is enough: ` Copyright (c) 2015 BalaBit` instead of `Copyright (c) 2015 BalaBit IT Ltd, Budapest, Hungary`


i added this (a bit downwards), thanks!


Хз. У меня вот к reversed нет претензий, потому что итератор не равен списку. У меня wat случается из-за противоположных штук, типа `True > False` или `0 > None`. Но в третьем, кажется, это поправили.


@OgreTransporter...
Please send meaningful patches if you feel this is a "problem" and should be fixed.
Note that we need to produce reproducible packages, which work on Windows XP. (=we need all the dependencies built using v120_xp toolchain and be sure about it) and be able to explain it to people (I'm sure you have seen all the "I'm a Windows developer and your build instructions are sure wrong because when I *don't* do what they say, the build breaks" on the forum)


Which one is first on the toolbar? I think we would like `h1, p` looks like the order might be reversed here?


@cbuechler /usr/local/etc/php/extensions.ini is modified when you install / remove any php module package, then it end up out of order


Coming to see you


Sounds good. Been wanting to do something like that for a while. To me this seems like a breaking change so a major version might be best.


Try anything with my code and ask for anything you do not understand.


Слишком много логики в 1-ом методе.
Можно вынести отдельные методы `keys`, `vales`
Метод не поодерижвает update.


#test+branch (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (376 of 376 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Can we rephrase it to the following and reduce some confusion?

`Job {scp} within stage {deploy} within pipeline {ProdDeploy} attempts to fetch artifacts from non existant pipeline {not-found}`.


I suppose, Line can be deleted in next commit.


#test+branch (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (376 of 376 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


See comment above.


`msg.nick` is set to empty to respond to the user that the username is not allowed. IMO it's fine.
Yes, I need to check everytime jinora receives a message, because username can easily be tempered with on the client side.


Ditto.


Thought we said we will be building images as part of the guides.... the motivation to which was to make it more transparent how users can customise the image, i.e. start modifying the code etc.


Just guessing, but it sounds like the warning is from an EXPECT_EQ with integers, like this line?


That was it! Thanks WebSpider, that makes perfect sense now that I"m realizing it was not able to verify the SSL. Unlike one of the errors above, mine doesn't mention anything. Could we maybe modify this error to say "Unable to verify SSL, try disabling SSL verification." or something similar?


REST principle: if you implement a GET method, then you *must* return something. That's the nature of GET.

And self.next will not be followed after a GET (what would be the point?).

However, here you do not actually want to return something - so it can not be a GET. It has to be a POST (because you do indeed *update* the record). So either you make it a POST handler, or...

...you fake it.

If you end up in this method with a GET *and* you have a record ID *and* the method is successful, then you can set r.http = "POST" before returning, and /then/ self.next will be followed.

If the method fails, then you should defer to the list view - and that is also where you should normally return (because that's where you started). r.error will otherwise lead you to the index page, which is daft.

An approve_next I don't see as necessary - but if you implement it, make sure you implement the whole sequence (see S3CRUD.update). However, in such case you could just follow update_next rather than having a separate setting.


set/get можно генерировать сразу исходя из полей. Проблему может решить `attr_accessor`


### makeAllOrder 문제의 해결방식 비교
1. bst로 자리를 찾은 후 shift, 그리고 삽입
2. 원소 하나하나당 비교후 shift or 삽입

- time complexity : 1. O(n), 2. O(n)
- code : 1 is worse; 두 개의 수행을 하는 것
- optimal : 1 is better; 조건문을 덜 사용


This was meant for showcasing that starting multiple containers with the same hostname (`-h`) gives you load balancing via DNS. We should put it back :+1:


Since you've gon this far, you should also include the generated css too.


`True > False`, по-моему, совершенно логично.

А итератор и список должны быть несравнимы, их неравенство не соответствует истине.


Метод делает 2 задачи поиска
Нужно сделать 2 приватных метода и разделить логику


Actually, the order here is irrelevant. It's the order below in richTextEditor.ts:39 that matters. I flipped the order to put the h1 first: https://github.com/SamGraber/TypeScript-Angular-Components/commit/7729e1936e4e079330ed406f29c6e8bc298de60c


经济界


93653630-MultipleYearRunPeriodForTabularReports (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1911 of 1911 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:673](../blob/64dd349e32d8bd60006aac64ca350b00da40aa6f/include/chaiscript/language/chaiscript_parser.hpp#L673) Expression is always false because 'else if' condition matches previous condition at line 671. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:677](../blob/64dd349e32d8bd60006aac64ca350b00da40aa6f/include/chaiscript/language/chaiscript_parser.hpp#L677) Expression is always false because 'else if' condition matches previous condition at line 675. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/64dd349e32d8bd60006aac64ca350b00da40aa6f/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 675. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/64dd349e32d8bd60006aac64ca350b00da40aa6f/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/64dd349e32d8bd60006aac64ca350b00da40aa6f/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:1](../blob/64dd349e32d8bd60006aac64ca350b00da40aa6f/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/64dd349e32d8bd60006aac64ca350b00da40aa6f/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/64dd349e32d8bd60006aac64ca350b00da40aa6f/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/64dd349e32d8bd60006aac64ca350b00da40aa6f/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/64dd349e32d8bd60006aac64ca350b00da40aa6f/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/64dd349e32d8bd60006aac64ca350b00da40aa6f/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


lol, I saw line 39 below and figured that was probably the case but didn't figure you needed two comments on this. Perfect, thanks! :sunglasses:


Can we explain why `a job cannot have ... in its name` - this message may be conflict with the one above `Job name must be alphanumeric and may contain underscores and periods.`


if you do git pull --rebase it doesn't create a merge commit ;)


It's kind of redundant, though, and doesn't makes sense to use the same code for another container. I think we should try to get a better example. It will confuse users, as it confused me.


Change to `...and may contain underscores...`


fixed in [35f6a6e](https://github.com/sdslabs/jinora/commit/35f6a6e68eae04f62337d79efb8fd8c184eeb5d9)


rewrite that to be a valid object format


same


Плохая практика ложить 2 класса в 1 файле


Take a look at the changed files in this PR https://github.com/nodemcu/nodemcu-firmware/issues/545.


I will add a section on that.  I just wanted to get a working example up on the website, as it never worked!


Või pole meie seda veel üles leidnud..


#test+branch (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (376 of 376 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Yeah, :+1: for unconditional install.


Btw - I see this as specialist method, it should not be accessible in the normal approval concept.

In fact, it should not be accessible at all - because it's totally insecure.

It is way too easy to craft a URL that approves records and tricking an authorized approver by sending that URL to him by email. I'm totally against having that as a GET method - all alarm bells ringing. Especially when it is about publishing alerts!! This can easily be used to send out fake alerts and cause panic. I can only strongly recommend to not do that! Don't do it!


Ok ath9k. You are using TCP for iperf. Try play with UDP.


there is also a m3u8 manifest url in the wabpage(ipadUrl).
example:
```javascript
flowplayer("newplayer", {
  src: "/res/video/flowplayer.commercial-3.2.7.swf",
  wmode: "opaque"
}, config).ipad({
  ipadUrl: 'http://media-beta.fc-zenit.ru/getvideo.php?id=338301',
  androidUrl: 'http://media-beta.fc-zenit.ru//resources/3/9651/mp4/338301/tr360.mp4'
});
```


Кому-то WAT вот это:
```python
>>> False == False == False
True
>>> (False == False) == False
False
>>> False == (False == False)
False
```
а для меня это просто забавная фича.


"everywhere"


93653630-MultipleYearRunPeriodForTabularReports (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


IMO this in unnecessarily complicated (but correct).
It should be easier to rotate MAG vector into earth frame, truncate Z axis and calculate cross product with actual north direction ( = error vector). Then just rotate error vector back into body frame.
Error vector must be in Z earth axis ([0,0,e], it is cross-product in x-y plane) and north is assumed on X axis ([1,0,0]), so the calculations will simplify a lot ..


Remember that you don't need to add this top navbar @caiohdf, it's already being added dinamically on html/parts/navbar-top.html


Can we push this into `JobConfig#getTasksOfType(FetchTask.class)`?


#test+branch (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (376 of 376 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


No, it shouldn't be necessary.


No users can create fake alerts here, so not quite possible ;)
However the point that, for all the usual RESTful reasons, GET should not modify data is still correct.
This should be a POST (true not fake)


It's important to use `os.path.isdir` instead of `os.path.exists`, because the latter could return true if a file existed by that name.  Then you can remove the dots and slashes, because `isdir` doesn't need them.


yeah! nice spotted @MarceloBorgesP this commit is from a long time ago, will fix it now


Also there are other references of `task instanceof FetchTask` all over the codebase.


like á þetta copy!


I think it should wait for callbacks when executing from a file (like node.js).


except without concrete exception!

Either catch a specific Exception type, or you *must* log the exception. Swallowing errors is not acceptable.


These files same :).
Strings:

 [CATransaction setDisableActions:YES];
   //    	[CATransaction setValue:@(YES) forKey:kCATransactionDisableActions];

are equivalent.

But I use the tabs to spaces, and you use space button to spaces. That is all.


#test+branch (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (376 of 376 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I don't think I can use a required parameter as I only care about the `permitted` option when `next_node` is called with a block. If `next_node` is called with the name of the next node then the `permitted` option isn't required.


output = {}

(using Storage is deprecated)


Is the deadlock really inside d3d9.dll? Isn't it just waiting on the UI thread?


Good idea. I'd completely forgotten about that method.


It's not a good idea to use kwarg-style filters with hybrid properties. Sometimes it works, sometimes it doesn't:

https://bitbucket.org/zzzeek/sqlalchemy/issues/3308/cannot-use-hybrid_property-in-filter_by


Indeed :) ruby/rubyspec@6209f8228d1d575fff766ffd70aa1701d19d45de
(the squash commit is to avoid replicating the rubyspec history too much in jruby)


Javadoc is not matching a variable name


空白があった場合にエラーになるので、空白削除


This won't work unless you convert `limit_reached` to a hybrid property.


#test+branch (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (376 of 376 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


looks like you need to `composer self-update` and `composer clear-cache`


As it is an open source project, should we use things like 'Zertico' @caiohdf ? Sorry if I'm wrong


please put comment above to explain this expression


but introducing another empty line here :stuck_out_tongue:


I recommend against using this technique. You should be defining the behaviour of `getContent()` and `persist()` on the Spock mock objects instead. Grails in Action has an example in listing 9.4. The Spock documentation on interaction testing provides more examples.

The above syntax is called EMC (ExpandoMetaClass) syntax and allows you to dynamically add behaviour to existing classes. The problem with using it in tests is that any changes remain on the class after a test runs, leaking into other tests. Grails has some builtin support for cleaning up the metaclass, but that's not the case here.

Even outside of testing I tend to discourage this approach because it makes code hard to debug and reason about.


#test+branch (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


please put getRequiredToken above getDefaultTokens to show that amount of tokens is growing in each following method.


Thank you.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 813](https://teamcity.mesosphere.io/viewLog.html?buildId=58512&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:04:00


Thanks for the comments @tap. I read that randomascii post last week, but found it pretty dense. Re-reading this morning helped it soak in more. Did some testing this morning and found that a ULP setting of 6 was needed to make

0.20000000000000018
compare as almost equal to
0.2

I would say that is a sufficient threshold for me. Would you support me changing the default to 6 for now?

BTW I think this might point to the need to test the testing methods themselves. We need some test testing tests.


BTW: It would be great to use functions for vector/quaternion manipulation. It will discard some optimizations, but code will be much easier to follow. And working code may be optimized later...


`awful.ewmh.activate`


I think that's a nice idea! Actually I wrote a simple contrib module that monitors the network and updates the
drupalgap.online variable. Maybe another thing to port to jDrupal: https://github.com/eduoda/network_monitor/blob/master/network_monitor.js

In the field-test (in my cellphone, with phonegap), since I always let the 3g/4g on, I noticed that most errors were a timeout from mobile network operator due to poor coverage. Unfortunately, cordova-plugin-network-information does not deal with this. I also tried cordova-connectivity-monitor, but I could not manage it to work. That's why I've implemented the timeout feature and drop the drupalgap.online test (that was misplaced, since it's a drupalgap thing): https://github.com/eduoda/jDrupal/blob/f105fb57e427291e0d2c6d84c7abcdaeead1d987/bin/jdrupal.js#L1208


please remove this, you do not do the same in visitToken so be consistent.


Could also be `unique=True`. I don't think the same transaction should ever be associated with more than one registration.


`uselist=False`


`index=True, unique=True`


No, the only acceptable way is a POST (and you can easily make the "Publish" button a script button that sends a POST) - otherwise alerts can be published from any clickable URL, and that is way too risky.

And if it is a script button, then you POST to a non-interactive URL (e.g. .json), which returns a JSON message indicating success or failure, and then refresh the datatable without reloading the page (or even just remove the button that triggered the POST).

And of course the button should have a user confirmation dialog attached (which I think it has already).


I'd set `now = now_utc()` first.


Looks like a mistake; checked it out and it could never have really made sense. Of course it must have supposed to have been some 256px default for Gravatar, given the commit where I introduced it, but it wouldn't have done anything there. I'll take it out.


`return registration.transaction is None`?


No, it's size management -- the mark that size_in_blocks is effectively respected, not ignored.


Cool, thanks!


@pedro-mendonca
![aanhalingstekens](https://cloud.githubusercontent.com/assets/11356362/10484251/e10a67b4-727b-11e5-924c-b7c513c5d5cd.jpg)
@johnclause
languages with or without quotes I do not care - It is a link and that is fine


```
else:
    return False
```

more readable IMO since all returns will be on the same level that way


I do not like "ast.getType() != TokenTypes.PARAMETER_DEF" here, as we do this on level above in SWITCH-CASE, I propose to have to two methods invertVariable and insertParameter.  The less IF statements we have the easy to understand the code.


More readable if you add `regform =  self.registration_form` ;)


Like that! :+1:


Agree, will fix


Good.


Full test coverage! [goodman](http://www.reactionface.info/sites/default/files/imagecache/Node_Page/images/feels%20goodman.jpg)


Enum is missed in name, please name method as findFirstUpperNamedBlock


added line surpasses the 80th column boundary. Please adequate to HHVM's coding guidelines:
https://github.com/facebook/hhvm/blob/master/hphp/doc/coding-conventions.md

(I'm not a bot!)


OK, will fix. Plz, audit other sprintfs to be snprintfs too. Thanks you :)


That makes totally sense. It was a late minute change. Sorry.


Well, and evil user who has permission to create alerts can very well place a fake alert with nonsense contents, and then trick an authorized approver to publish it unintentionally by sending them the approve-URL as clickable link per email (e.g. disguised as something on linkedin.com or something).

That would be a totally simple phishing attack, nothing easier than that.

I will not hesitate to make this hole public if you leave it at that.


We do have KDE_PUNT_BOGUS_DEPS, but although it seems fairly robust, I've never been sure about always recommending to use it or not. We could do it automagically based on KDE_HANDBOOK/KDE_TEST, but then we lose the information about which packages we're hacking (if we care).


Don't put a href on it in this case. You'll get the `#` in the URL otherwise.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.664](http://teamcity.monogame.net/viewLog.html?buildId=10583&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


looks fine to me @rluz !


This method is too complicated, and I forget why we case there about LOOPs in so special way.
Is there a way to simplify method(split in two methods)  ? if not Javadoc should be extended with details.


so u mean, hasPriorityOver(...) ?


The "you already registered" check is gone?


The warmup you do change was written by @jexp. It would be interessted to hear his opinion.


OK, I see. Links to files in automake-1.14. That will not work, given that there are different versions of automake and, possibly, different locations.


I was thinking about using vector/quaternion functions for better readability, but dropped the idea in favor of optimizing the code.

After I try my code and if it works, I'll try your suggestion for calculating error vector for mag in earth frame instead of body frame. It indeed should greatly simplify the math.


Hola Víctor, ya no da pete por el tema de las importaciones. Te comento algunas cosas que he visto en el commit:

* Cuando Python instancia una clase llama a __init__, con lo que no es necesario que lo llames. Tampoco hace falta el return para que devuelva el objeto.
* En los condicionales no pongas paréntesis y separa cada parte de la condición con un espacio en blanco.
* Quita los global, puedes hacerlo pasando por parámetro las variables que quieras. Intenta no utilizar esa palabra reservada.
* Desengánchate de Java xD


For me it totally unclear what service calls did. Can you explain this in more detail?


Checkbox is no good, since you are allowed to check both male and female at same time. Maybe radio button?


:clap:


Kaggle username is andbar


I need :
1. List of additional data to log
2. Please check if I logged Etags correctly
I don't want to pull request it yet... (and I am working on the next issue so it can wait a little)


any examples on button sending POST request and handling it?


@johnclause
![niet vertaald](https://cloud.githubusercontent.com/assets/11356362/10484467/21c9bee8-727d-11e5-8c5e-4c37c70edd2d.jpg)

<b>:-)</b>


Also I'm not a fan of using single letters to reference objects


I think the whole point with the separate publish-permission is to make sure that only especially authorized people can publish alerts, not every user who can create alerts.

But if you can trick those authorized people with simple clickable URLs that publish alerts without any further warning, then you make this a very weak concept.


@mikecriggs oh i've reverted the FWB part and forgot to revert vendor part.


its vry grt


:hankey:


Il y avait aussi la bonne vieille commande `yes` :

* pour accepter tous les changements : `yes | python -m patacrep.songs.convert ...` ;
* pour refuser tous les changements : `yes n | python -m patacrep.songs.convert …`.


real


I'm an author! Yay! :)


I see, this way is much more clearer! Yet would still add a comment about how errors are handled.


Is it okey to send them to /review and them have them approve or reject as we have without implementing this clickable URLs?


Didn't feel right leaving you out Dan :)


There is a typo in the AutoPropertiesFixedRecoedAttribute class name


You can just do `segment = US["B3", "B2", "B1"]`.


@johnclause That sounds great :-)


I added this line (which I think is correct) and the tests still pass locally, but Travis is complaining that `prettyPrintType` is not in scope (?)


The tests are commented out?


successful means awaiting payment?!


Excelente, @alineatcosta! Obrigado!


-for


Thanks, Carl! Do you have a link to the exact place in Savannah's procurement policy where it shows that?  If it's a PDF, please add #page=thepagenumber at the end of the url.  e.g.: www.example.com/policy.pdf#page=3


Looks ok, but can you shift the second preg_match inside an else. Running 2 regex checks potentially unnecessarily and these are quite expensive and run every request. You could also have one regex where the id part is optional but perhaps this is better for readability.


approve, -for


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 814](https://teamcity.mesosphere.io/viewLog.html?buildId=58523&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:04:05


Follow us on facebook

Now don't say you could see that this ^ link would publish an alert.


Bitte noch die Hilfe der Argumente erweitern damit man weiß welche Commands es überhaupt gibt. :)


$a and $b. $index would be better if that's what it is. The variable names should convey what their purpose is, generally speaking. This makes it much easier to go through the code.  ;-)

$a appears to be used as an index for the $urls array in the foreach loop. Are you sure it's not needed?


Was just adding a pull request for that! http://savannahga.gov/index.aspx?NID=598
--
Carl V. Lewis
Sent with Airmail

On October 14, 2015 at 9:13:14 AM, Alan Mond (notifications@github.com) wrote:

Thanks, Carl! Do you have a link to the exact place in Savannah's procurement policy where it shows that? If it's a PDF, please add #page=thepagenumber at the end of the url. e.g.: www.example.com/policy.pdf#page=3

—
Reply to this email directly or view it on GitHub.


Ehhh, un abrazo Sebas, este fin de semana me le pongo en línea. ;)


The option **generateCSS** has no effect because, on initial load, **Avatar.setOptions({});** is called, and default value of **generateCSS** is **true**. And the injection has already taken place, before anyone could set **generateCSS** to **false**. What to do?


Shouldn't this look similar to `setValue` in `EnvironmentVariableConfig` (#noob)


lgtm


`StringUtil` is `null` safe, no need for `null` check.


@hrach přepnu do češtiny. Vůbec neřeším upozorňování na chyby, souhlasím s tím, že do stable by neměla věc jít too early (a připouštím, že to občas dělám). Ale vadí mi blahosklonné rady nebo nafukování situace.

Master nebo 2.3@dev používá naprosté minimum lidí, takže vzácně se projevující chyby, jako je tato (tj. projeví se warningem pokud použiješ if uvnitř href `href="{if}..."`), se objeví zpravidla teprve po vydání verze. I přesto, že 2.3@dev průběžně provozuji asi na sedmi projektech.

Měsíc to držet v masteru v podobných případech bohužel nepomáhá. Podobně se i ohromná spousta chyb PHP 7 objeví až po vydání PHP 7.0.0. Samozřejmě šanci to nepatrně zvyšuje, jenže na to navazují další věci, takže je potřeba udělat rozhodnutí. A jelikož taková chyba se stává extrémně zřídka, je to asi celkem schůdná cesta.

Co ale situaci fakt nepomáhá, je eskalovat problém, dokonce až k řečem o tom, že to komplikuje udržovat dokumentaci, což je o to divnější, že se ví, že dokumentaci nikdo neudržuje. Pokud po měsíci práce na open source je výstupem pozitivní tvít a negativní komentář na Githubu, tak je to značně demotivující.


I confirm this is working!


oh, right. It says `noCache`. For some reason I thought it was `cache: true` :)


OK, this fix : https://bugs.limesurvey.org/view.php?id=9939 (I think). But why not add a real button after each form ? Internet web form look always with a save/send button at end of the form.
This is better for usuability ond for development : see http://mdo.github.io/code-guide/#html-javascript


That is something for both of us. It was pointed out previously by silvio that we need to better "filter" the pages we create, things like "novalidate", 'id=" "' and 'action=" "', won't be used on ember


This looks fine but I'm wondering why we don't just eliminate the vendorID variable all together if it's unused?


So... notices have disappeared ?


Bunu kabul edeyim, SmallSimple'a gitmesi için kendim bir PR daha yapayım


Agreed. I thought I had text in there to that affect, but it must have gotten lost in the latest edit.


moveDirection should be member: avoid useless memory management


typo


Do we even need the Full switch? I mean we never do releases that don't have Full anyway and Full generates the _mini part, too?


Did you mean the following?
Copyright (c) 2010-2015 BalaBit

By the way, do we always list all major contributors for every file?


Note to someone: things to revert + fix.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 815](https://teamcity.mesosphere.io/viewLog.html?buildId=58524&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:03:55


I'm not sure. Perhaps on the raft leader for the first range of the table.


Here is the readme, Raquel  @rmcmx


Using regular /review is good and safe, yes - because it's more than just a click, it requires a conscious decision and reflection about what the user is going to do. I'm very much in favor of that.

And considering the consequences that this action is meant to have, it should be more than a click.

However, UX design dictate may take this design decision away from you - I don't know. Try to convince your boss.


Wrong commit message. https://github.com/SoundWarp/sound-warp/issues/3  should be closed instead of https://github.com/SoundWarp/sound-warp/issues/9.


@ihrwein Well, compared to the other case it is a bit different here. You are right in that both should be covered, however the reasoning here was to signal the initialization of a list. Maybe we could add a comment for that too. Maybe not. I don't feel strongly for this line either, but just so you know.


works for me in M2:

```
% /usr/local/scala-2.12.0-M2/bin/scala -Dscala.repl.welcome=hi
hi
```


Thank you very much @Eroppo


use awake instead ?


Whaou!

Je viens d'apprendre quelques chose :smile:


pass here at each loop... should throw excep in Start or awake no ?


Hah, very nice, a chronometer!


Well done! If you don't have any questions, feel free to move on to the next chapter.


When I tried to resolve https://github.com/galaxyproject/tools-iuc/issues/325, I was getting problems with the installation of DESeq2 (and also with EdgeR) using the most recent version of Galaxy. When I ran the created command from the code above into the terminal I saw that it was returning a 1 while this should be 0 because the installation was successful:

```
$ Rscript -e "tryCatch( install.packages(c('BiocGenerics_0.14.0.tar.gz'),lib='/usr/local/src/galaxy/tool-dependencies/deseq2/1.8.2/iuc/package_deseq2_1_8_2/00b14423d00f', repos=NULL, dependencies=FALSE), error = quit(status = 1))"
$ echo $?
1
```
When I remove the ```error = quit```-part I see that the installation was successful:
```
$ Rscript -e "tryCatch( install.packages(c('BiocGenerics_0.14.0.tar.gz'),lib='/usr/local/src/galaxy/tool-dependencies/deseq2/1.8.2/iuc/package_deseq2_1_8_2/00b14423d00f', repos=NULL, dependencies=FALSE))"
* installing *source* package ‘BiocGenerics’ ...
** R
** inst
... truncated ...
** building package indices
** testing if installed package can be loaded
* DONE (BiocGenerics)
```

This might mean that all R <package> installations are currently not working. Can anyone confirm this (@mvdbeek)?
I.e. update the galaxy instance and see if similar behavior appears installing one of these packages:
package_dexseq_1_14
package_deseq_2_8_2
package_r_ggplot2_0_9_3
package_r_edger_3_11_0


I don't think it makes any sense to include a language name between quotes.


Du coup tu peux annuler le commit si tu veux


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.664](http://teamcity.monogame.net/viewLog.html?buildId=10583&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 541, ignored: 6 Build time: 00:20:09


@rjbs Duh, this my $ret/$ret is a leftover from a debugging session, feel free to cut it out, it doesn't add any value.


as they are constants, you could make them final as well


In the bridge class, following code will be generated.
```
    WebResourceRequestHandlerBridge(WebResourceRequestHandlerInternal internal) {
        this.internal = internal;
        reflectionInit();
    }
```
```super()``` will be inserted to this constructor automatically. so the base class(```XWalkHttpAuthHandlerInternal```) should has a default constructor.
For this structure, I think it was designed to generate generalized code for other classes.


no need to use Awake5) in that case : I prefer to use it when some kind of order conflicts can happen


Beyza, MAEyler.java zaten Repo'ya eklendi, sen tekrar koyamazsın

Bunu silip kendi dosyanı ekle, sonra commit ve Pull request -- iş bu kadar


Misalignment?


It's already a member ;)


I removed the summary.. i didnt want it to be there :p :p


Looks great!


We'll need this returned.


Can we rollback this change?


Because this change will need to be separately reverted, I suggest you to go ahead and open a PR in riak_pipe to fix that `exec_option()` type. It's pretty trivial and shouldn't take long to get through.

Meanwhile, if you revert rebar.config, I'll merge this branch into mine and make a call for reviewers to have a look.


You can, and I mentioned that in the updated readme. We didn't want to confuse people that are used to running `bin/buildout`


yeah but you create new vector at each loop


Students/Others/Vacareanu M. Robert Ionut/Chapter1/BasicActions/app/src/main/res/layout/activity_main.xml

This way of linking the click listeners to a button is OK, however, I wouldn't prefer it done like this. It works, yes, however, I would have at least made 2 separate listeners for the two buttons.
The second (and in my opinion more clear/concise) way of doing it would be to of course declare the buttons as variables in your class and then adding on `onClick` listener to them, there.

But you know, the way you did it is not wrong! It's just that I've never seen it been written this way in a released application. Though standards may of course differ.


Everything else looks good. Feel free to move to the next chapter!


Nice catch.  It's supposed to be referring to dataCount instead.  Unrolling the loop manually is typically more performant in cases like this.


@benrudolph fyi this broke UCR a number of different ways since many non-serializable types stopped being converted


Apologies if this is a silly question, but are there any examples in eden on button sending POST request and handling it?


93653630-MultipleYearRunPeriodForTabularReports (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (837 of 837 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-24.4%25-green.svg)


I don't mind setting it to `6`, though my gut feeling is that `8` would be better.  What I would really like is something other than trial-and-error to figure it out, but I'm not sure how to accomplish that.

The thought regarding testing our test system assumptions is also very helpful, as such tests will clarify and codify our working assumptions.


@pedro-mendonca Neither do I.

I someone really wants the quotes then that person can just use CSS or change the string in the settings.


Tankyu


Please remove trailing whitespaces.


The problem is that once you've published an alert, it's not easy to take it back. And it's not meant to be - people shall respond, immediately. Low-lying areas shall be evacuated before the Tsunami is rolling in - and that's just minutes to react. There is no time to put it in question or even think that the alert could have been published by accident because someone has followed a malicious link that was send to him.

That's the whole point of alerting - people shall respond, immediately, in order to save lives. And of course that is disruptive and dangerous in itself, but that risk stands against an imminent disaster.

However, the more important it is to make it a conscious decision whether to publish an alert or not. If course it shall not be needlessly fiddly, but a little more than just a click it should well be - and as the very minimum it should not be possible to publish an alert inadvertently.


Can we add an early exit if `helper_nodes` is empty?


worked for me.

perhaps need to add a paragraph on the documentation, to let people know
they need to add the default entries 'again' by a sample code, that show a
specified list with those entries repeated.

nb : my proposed code, was taking care of that, but your seem more aligned with usual constructor practices, so have the advantage of being much more 'predictible'


Good, good, `Snackbars` are indeed replacing Toasts!


the leader which owns (the beginning of (in case we don't want to pin this to one range)) `system.lease` sounds like a natural candidate. We should avoid having to run a goroutine per table.


Probably easier to just mark THEMEDATA_PATH as not-recommended tbh, since my local LOADOTHER_THEMEDATA option works far better(haven't tested your theme with the latter yet though).


I'd name the page more explicitly with something like AdminMenuEntry or AdminMenuEntryUIX


Without it, clicking refreshes the page.


A link with no href shouldn't do anything :o


SO if everything is clear, good work and feel free to move to the next chapter.


Yeah, I'd have refactored it into its own static local function, but I couldn't think of an adequate name for it at 2 AM.


Great, thanks! @Justoce


I imagine the compiler would unroll it! :game_die:


The title of this commit doesn't make any sense. An unused field is exactly the case where you *don't* create a new version.


What do you think about adding `editable=False` here? Then it shouldn't show up in forms by default.


Things look fine. Good work


The `/` indicates the root directory.


Ops! I copied that line from the debug output.
should be:
```
$EDITOR `cygpath.exe -d "$prm_dir/$argument/start.sh"` && $EDITOR `cygpath.exe -d "$prm_dir/$argument/stop.sh"`
```


Cool. When the Admin App is not installed the menu is not here, that's very cool :)

Now we just need to move the translations into a Translations.xml page to be complete ;) (without loosing existing translations as much as possible)


Thank you :smile:


`self.field.set_versioned_data(field_data)` would be nice (and then move this logic into that method)

Could even be a property (get -> current_version.versioned_data, set -> create or update version)


I meant my copy and pasting the assignment operator everywhere!


Try it out, I bet it doesn't ;D


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | Timeout | Skip
 ---------|---------| ---- | ---- | ------- | ----
 basic_test|__suite_setup__|0|1|0|0
basic_test|_basic/test_create_vm_|0|0|0|1
basic_test|_basic/test_stop_vm_|0|0|0|1
basic_test|_basic/test_start_vm_|0|0|0|1
basic_test|_basic/test_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_volume_|0|0|0|1
basic_test|_basic/test_add_vol_to_stopvm_|0|0|0|1
basic_test|_basic/test_add_volume_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_multi_volumes_|0|0|0|1
basic_test|_basic/test_add_volume_negative_|0|0|0|1
basic_test|_basic/test_crt_temp_from_volume_|0|0|0|1
basic_test|_basic/test_crt_vm_with_volume_|0|0|0|1
basic_test|_basic/test_vm_securitygroup_|0|0|0|1
basic_test|_basic/test_create_iso_vm_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account2_|0|0|0|1
basic_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
basic_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
basic_test|_suite_teardown_|0|0|0|1
virtual_router_test|__suite_setup__|0|1|0|0
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
virtual_router_test|_novlan/test_create_vm_|0|0|0|1
virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_vlan/test_create_vm_|0|0|0|1
virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_snat_create_|0|0|0|1
virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
virtual_router_test|_vlan/test_start_vm_|0|0|0|1
virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
virtual_router_test|_volume/test_add_volume_|0|0|0|1
virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
virtual_router_test|_sg/test_ingress_|0|0|0|1
virtual_router_test|_sg/test_egress_|0|0|0|1
virtual_router_test|_sg/test_icmp_|0|0|0|1
virtual_router_test|_sg/test_udp_|0|0|0|1
virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
virtual_router_test|_sg/test_vm_ops_|0|0|0|1
virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
virtual_router_test|_sg/test_detach_l3_|0|0|0|1
virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
virtual_router_test|_pf/test_attach_pf_|0|0|0|1
virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_multi_rules_|0|0|0|1
virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
virtual_router_test|_pf/test_tcp_|0|0|0|1
virtual_router_test|_pf/test_udp_|0|0|0|1
virtual_router_test|_pf/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_2_eips_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
virtual_router_test|_eip/test_attach_eip_|0|0|0|1
virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
virtual_router_test|_eip/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
virtual_router_test|_lb/test_create_lb_|0|0|0|1
virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_suite_teardown_|0|0|0|1
local_ps_virtual_router_test|__suite_setup__|0|1|0|0
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_ps_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_ps_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_udp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_ps_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_ps_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_ps_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_ps_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_ps_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_udp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_ps_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_ps_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_ps_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_ps_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_ps_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_suite_teardown_|0|0|0|1
local_nfs_virtual_router_test|__suite_setup__|0|1|0|0
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_nfs_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_nfs_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_nfs_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_nfs_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_suite_teardown_|0|0|0|1
Total|304|0|4|0|300


Mmmmm ... I would have thought "None" as a better alternative. Or "Unset".


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 816](https://teamcity.mesosphere.io/viewLog.html?buildId=58532&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:07:23


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 817](https://teamcity.mesosphere.io/viewLog.html?buildId=58533&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:07:33


Missing `not`.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 818](https://teamcity.mesosphere.io/viewLog.html?buildId=58535&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:07:54


Good job.


> Now don't say you could see that this ^ link would publish an alert.

My point was that ALERT_EDITOR role would NOT be given to random people..the chances that someone entrusted with the ALERT_EDITOR role would abuse the system like this seem pretty negligible to me...although I certainly don't disagree that the system should be changed anyway...just think that it's being way overplayed.


Heil grammar!


MenuState should be in its own module.


Yes, script buttons are dead-simple. Instead of a "url" for the action button, you set an "_ajaxurl". This will render the button with data-ajax-url=[the URL], and you can attach whatever script you like.


Care to share a couple of words why?


For now, you can define a cpu_reset pin in the platform file and use it. (CK_RST / C2, seems to be active low).
We'll see for chipkit later.


btw, I'm doing a merge to the current master and I'm solving that conflict. Please do not change this, I'll do it


![image1 1](https://cloud.githubusercontent.com/assets/179599/10485275/8ea40f62-7289-11e5-9501-debf9aecd25b.png)


ok


you are fast!


Do you recommend `success = !!collection` instead? I wouldn't call that clean syntax itself, but I've seen that in syslog-ng's code before. I like to clearly signal intent and it should compile to the same anyway. Also, writing constants on the left is defensive programming. Is this something you dislike about this line?


Pretty cool indeed :)


`$.isFunction`


Thanks anyways....but do you know anyone that can help me
On Oct 13, 2015 3:13 PM, "ChristianB1" <notifications@github.com> wrote:

> I dont know how! please stop emailing me
>
>
> On Tue, Oct 13, 2015 at 3:10 PM, Ninjasifi <notifications@github.com>
> wrote:
>
> > I don't know how. Sorry.
> >
> >
> > On Tue, Oct 13, 2015 at 1:38 PM, Reaper1437 <notifications@github.com>
> > wrote:
> >
> > > plz help me pllllzzzzz
> > > Inquiry Number 1210340863
> > > 10/13(Tues) 13:37
> > > Email is sleepy14378@gmail.com
> > > Really need your help
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/3dshax/ctr/commit/bcb3734b9a26d0fe7ef66f7d3814406fee797303#commitcomment-13747184
> > >
> > > .
> >
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/3dshax/ctr/commit/bcb3734b9a26d0fe7ef66f7d3814406fee797303#commitcomment-13749597
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/3dshax/ctr/commit/bcb3734b9a26d0fe7ef66f7d3814406fee797303#commitcomment-13749636>
> .
>


Removed a foreign character that should not have been there.


Like this: https://github.com/nursix/eden/blob/master/modules/s3db/hrm.py#L7638


@gut as we talked, find a better solution for the line comment that is not aligned. Just to remember.


Be careful about randomly changing Class names. You just broken everyone else's code that expects it to be called `Rect`. I don't think there was any reason for the rename anyway.


Try:
`ctags --language-force=SystemVerilog -o - Units/vera-interface.d/input.vr`


@dean0x7d Tried in https://github.com/cppformat/cppformat/commit/08bcf61fb210dc7023e1a4461b9150140b94fc1b, but to no avail.


Your JS version of update is not going to work in browsers for a long time (`for ... of ...`). I'm trying to keep the emitted code at browser-level JS plus ES6 datatypes that can be brought into the browser. I suggest using `Map.forEach(...)` or calling a function in quark_runtime.js.


I'm using iMPACT, can you keep it?


that's right, we're not using a ppa any more, we're using nodesource's deb repo


Ah oops, maybe this is the better serialize function. Always Unicode the object except if it's None. The other way I guess we're always going to be adding types to the list of instance of


You don't need this line. You need to configure ENVIRONMENT variable. We will discuss about it.


Unless the MixpanelAPI class is thread-safe, you might want to use [mutexes (locks/synchronized) on the reference](http://stackoverflow.com/questions/442564/avoid-synchronizedthis-in-java) just in case.


Yikes! Please, no! If we're crashing on trying to print `undefined`, there is a bug in the code. Better to crash.


Thanks for your help debugging this issue. It was invaluable!


@livecodesebastien Please check version numbers properly, not by doing strange things with multiplying and adding! Please make feature macros obvious, e.g. `HAVE_BUILTIN_UNREACHABLE`.  Otherwise it can be very easy to think that `BUILTIN_UNREACHABLE` is the actual function and can be called... ;-) Example:

````C++
#if defined(__clang__) || (defined(__GNUC__) && (__GNUC__ >  4 || (__GNUC__ == 4 && __GNUC_MINOR__ > 5)))
#   define HAVE_BUILTIN_UNREACHABLE 1
#else
#   define HAVE_BUILTIN_UNREACHABLE 0
#endif
````

On reflection, I don't understand why you decided to implement `__MCUnreachable()` as a regular function, rather than as a macro...?


If you see anything written in triple qutoes (""") it is usually a docstring and *must* be on the first line after a module, class, or function definition. They are special strings that python uses to internally generate code documentation for that module.


Nice! However, I recommend not spending time on stuff like this, as the backends are going to be refactored heavily in the near future. That said, since you already did the work, ship it! :wink:


Of course it should!
It seems I shouldn't push late-night commits without thorough review:P


Please add constructor __init__() .... and declare class variables in constructor.


Nice :) It should however be "fulfill" (American spelling) or "fulfil" (British spelling). Not sure which spelling we want to use in the game btw? Any preferences?


Hi Gustavo,

On 10/14/2015 10:03 AM, Gustavo Serra Scalet wrote:
> added line surpasses the 80th column boundary. Please adequate to HHVM's coding
> guidelines:
> https://github.com/facebook/hhvm/blob/master/hphp/doc/coding-conventions.md
Sure. Fixed already. Thanks for spotting it.

> (I'm not a bot!)
(I am not a bot either!)


Thanks @tstastny looks good. And how did you get the image of techpod? Is that a CAD model?


yep.
Because `_ctype` is private function, I don't test these suites


参照博主教程，今天W530去除白名单成功，感谢博主分享。


I suspect the Guice thingie has state that gets dragged across tests. It is not an issue for us now, but lets follow the recommendations.


@kurimon private func checkPeripheralName
でアクセスコントロールできますお


The comment is no longer relevant.


Right, I've changed it. I didn't know about Spock's syntax for stubbing. Does it look ok now?


Dans le cas d'un unique vote négatif :
Pourcentage Son : 50 + ((0 - 1) / 1) * 50, soit 50 + (-1) * 50 , soit 50 - 50, soit 0,
Du coup si une seule personne vote négatif, et tant que personne ne vote pour re-augmenter le son, le volume sera désactivé? :')


When will the next 1.10->master merge be happening? Anything I can help with?


The last bit, "i.e. without having to specify a `.weave.local` suffix", jars to me because `.weave.local` has not been mentioned up to this point; why would the user even think about a suffix?

Maybe replace the whole paragraph with "other names get resolved via DNS, so if you use weaveDNS it will return the weave IP address of any container in the cluster"


Why do you check death time in singleton function? It is incorrect:)


I think good to be explicit for now until it gets worse


That's cool 82e0177ae5d688181d88d5dcf8c4605f994853b8


`yes` est une commande toute simple dont je sers moins d'une fois par an, mais qui est tout de même très utile… :)


Will do. Actually I also consider afmongodb_dd_connect() a bit long and I've considered spitting it into smaller functions. Introducing a function for iteration here would sadly involve moving the prelude further away from the loop core. Hence, I will need to introduce at least two functions for this.


@minhpl  This code has problems even after I fixed the build issues:

https://github.com/lemire/FastPFor/issues/25


Came here to report the typo " As a brand new parent with a lot of imagination" but you've already fixed it :)


This function now mingles together environment manipulation with AST construction. Can we keep them separate as before?


Nit: use `x` for var nodes.


How can this part of the condition be false? Isn't it an invariant that any named type is in the list?


Nit: don't abbreviate field names.


I don't understand, why do you use here decorator? Why do you need it?


Perhaps add an example with no `param` part.


Please remind me of how Lua will use Java generics...


@kurimon // MARK: - Private -
でxcodeから見たときにジャンプバーの見出しができるので // MARK: の書き方がおすすめですお


hmmmm...


Ah interesting. I apologize for breaking your code, that was not my intention. I was following this advice from semver.org:

> What should I do if I update my own dependencies without changing the public API?
>
> That would be considered compatible since it does not affect the public API. Software that explicitly depends on the same dependencies as your package should have their own dependency specifications and the author will notice any conflicts. Determining whether the change is a patch level or minor level modification depends on whether you updated your dependencies in order to fix a bug or introduce new functionality. I would usually expect additional code for the latter instance, in which case it's obviously a minor level increment.

I would have expected npm to prevent you from upgrading this package or show you a warning if the `peerDependencies` weren't properly met, but perhaps I am misunderstanding how that works. Can you help me understand what steps you took that led to the code breakage?


:metal:


@flavour: Biplov is implementing it as a generic function, so you can hardly argue with the SAMBRO business logic as a securing factor (which is weak, because even non-random people are vulnerable, and can be forced to do malicious things).

Moreover, you can /never/ argue with the business logic of a specific deployment when you generalize things.

If it is to be secured by SAMBRO business rules, then move it out of S3CRUD and into that particular template.

Otherwise - it is insecure and not acceptable.


maybe we could make these elseif for performance


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 819](https://teamcity.mesosphere.io/viewLog.html?buildId=58541&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:03:54


I'd like to see how you're going to pass a List to Lua... Hint: you're not


nice


no point in making 2 more check if we already know they're false after the first was true


Looks good


The worker is garbage collect otherwise


great


@mvdkleijn  You've made this attribute derived.
Be aware: This won't work expectedly if you will override `node['apache']['log_dir']` on some other level or in the wrapper cookbook

Details: https://coderanger.net/derived-attributes/


Mh, we actually use it in Jenkins 1.633 and there are no problems I'll have a look at it....


Will do. Yes, I did consider that alternative before implementing this. However I didn't see it as an improvement because of the extra functional complexity. Maybe I also did something else here at the beginning, but that eroded away.


@sometheycallme

I realized there's a bit o' b0rkedness there.

`/opt/nessus-persist/nessus/var`

should be:

`/opt/nessus-persist/var`

as should any similar references in that script.  I realized that this morning whilst battling the docker demons.


So you not only managed to copy and paste code and forget to change the TYPE, but you also don't use it at all.


Hi, i have question with the file(**LowPassFilter2p.cpp**),
i think
***float delay_element_0 = sample - _delay_element_1 * _a1 - _delay_element_2 * _a2;***and
***float output = delay_element_0 * _b0 + _delay_element_1 * _b1 + _delay_element_2 * _b2;***
should be
***float delay_element_0 = _b0 * sample - _delay_element_1 * _a1 - _delay_element_2 * _a2;*** and
***float output = delay_element_0 + _delay_element_1 * _b1 + _delay_element_2 * _b2;***


You can install [depcheck-es6](https://www.npmjs.com/package/depcheck-es6) which is native supporting JSX.


#! /usr/bin/env node
var fs = require('fs');
var outfile = "startup.txt";
var out = "A startup is a business built to grow rapidly.\n";
fs.writeFileSync(outfile, out);
console.log("Script: " + __filename + "\nWrote: " + out + "To: " + outfile);


ここは、以下のパクリ
http://qiita.com/suin/items/10ce40d0f8478e9a1287


Hi, changing force_username_lowercase from false to true by default breaks the refresh of already imported ldap groups containing upper cases.


And btw - if you trust all ALERT_EDITORS, then why do you need an ALERT_APPROVER?


仮実装


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 820](https://teamcity.mesosphere.io/viewLog.html?buildId=58542&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 188 Build time: 00:06:09


```const events = [
        'job:processing:init',
        'job:processing:processed',
        'job:processing:failed',
        'job:processing:done',
        'job:set-state:init',
        'job:set-state:success',
        'job:set-state:fail',
];


赞赞赞！！！


On a common usage, user will set the period once and just keep changing the duty cycle, for duty cycle it a file descriptor to avoid opening it each time.


Looks wrong keep allocating heap memory when we can use stack memory to temporary things, lets wait for his comment.
According to documentation PATH_MAX is the maximum path length.
http://man7.org/linux/man-pages/man3/realpath.3.html
http://pubs.opengroup.org/onlinepubs/009695399/basedefs/limits.h.html


The "pwmchip%d' is defined on the PWM sysfs API.


Fairly good point, I don't know either why I didn't choose the simple path. Well, time for (another) update


You really need to start writing your demo IN LUA. Then, when you actually try to use this code, you will realize which things you wrote are wrong.


events.emit('job:processing:processed', {job});


maybe `job:run:*`? a bit nicer to write


Why is `setPosition` in UIWrapper?


Unless I'm misunderstanding the TODO on line 11, `prefs.getString(Config.USER_ID);` returns null because [`Config.USER_ID == null`](https://github.com/EmergentOrganization/cell-rpg/blob/4e2da96bc767373650a23b979c4624644df3ef83/core/src/com/emergentorganization/cellrpg/tools/Config.java#L16)?


```events.emit('job:processing:done', {job});```


Awesome


This should be either moved/copied to the generic extractor since that's the default way flowplayer embeds formats.


Initial commit with bare bones README file


What about float type?
Could you please a look at link https://docs.python.org/2/library/functions.html?


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 821](https://teamcity.mesosphere.io/viewLog.html?buildId=58543&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:03:53


Thank you for the bug correlation ! Yep it resolved this bug, and many bugs of this kind. In LS2.0x,  most of the time, form validation is based on custom js... And most of the time, it's based on the event 'button click' rather than 'form submit'.

About the double save buttons :  why note... Holch also asked for it. I'm not so found of double save buttons, but why not...


Resolves #2


@eileenmcnaughton I hope this is what you meant by extracting the guts of the function into another for testing?


You're totally right.  I'm going to roll back the changes and put a build in place which properly resolves without zend-hydrator then put support for that into it's own issue.


I did not see logging lines:). Could you please add their?


bene


@eileenmcnaughton Could you help me out here please? I've tried various ways to extract these two values (for comparision) but I just can't seem to get it right.


I suggest you trace through the code logic again, and/or double check that you're reading `==` versus `!=` correctly.  There are two cases here: normal disconnection, where we terminate the IRC client, set the disconnected status, clear the history and hide the chat UI; and errors where we terminate the IRC client, set the error status *don't* clear the history and *don't* hide the UI until the user manually hits the disconnect button to dismiss the errors and reset to a normal disconnect state.


Legacy; removed to avoid confusions.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 822](https://teamcity.mesosphere.io/viewLog.html?buildId=58544&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:04:01


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.665](http://teamcity.monogame.net/viewLog.html?buildId=10587&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


@timtheisen Do you have an ETA for 8.5.0 binaries and deb repositories?


Annulé aeb10c3c2a6e7459284d894a74fe4ee6318d88ae


@balthisar well it seems better than a 404 I was getting... thanks... seems this happened 10\12 days ago when I think I push something to the gh-pages branch... not sure what now... small page change I
think... but it was some time, days after that because that change was there in a browser test...

However, while I might have the time to troubleshoot, I do not have the knowledge or understanding ;=((

I even tries to set it up locally in my linux in the hope I could debug it that way, on `http://localhost:4000`, but after installing LOTS of things the `$ bundle exec jekyll serve` failed with `Could not find JavaScript runtime`... that was from this page - https://help.github.com/articles/using-jekyll-with-pages/ ... will continue fiddling with that... maybe...

One other thing I do note is that there is a `.nojekyll` file? I read this stops Jekyll processing, and something about directories with a leading underscore??

**Please, help needed!!!**


Potential simplifications. top target and decoy scores are directly accessed via the interface.


Looks like I extended the italics one space too far in front of General Services Administration, which doesn't cause any problems in the WYSIWYG, but Markdown will then continue the italics until it runs into another *.


:+1:

The comment was more for the `success` to be called `processed` and then after everything is finished (i.e. after setState), emit `done`.


Suggest using a final variable to hold '23' since it is used in many places; changing it later, should the need arise, will be easier.


This is using data from the IRC server – there is nothing to sanitize.


Did you have any error messages?
I will check tomorrow morning, but probably not solved before next week.


Also reports the p-values. This is necessary to make the numerical comparison with bioconductor/qvalue.


ERROR: Missing a Javadoc comment.


ERROR: More than 7 parameters (found 8).


ERROR: 'p1' hides a field.


ERROR: 'p2' hides a field.


ERROR: Utility classes should not have a public or default constructor.


ERROR: Class SoundManager should be declared as final.


WARNING: [InflateParams] Avoid passing `null` as the view root (needed to resolve layout parameters on the inflated layout's root element)
 When inflating a layout, avoid passing in null as the parent view, since otherwise any layout parameters on the root of the inflated layout will be ignored.


Te faltan los objetivos de la segunda semana y el enlace al primer hito de las prácticas. La fecha límite de entrega ha terminado ya, así que no te demores mucho si no quieres perder el hilo de la asignatura.


Нужно вынести код счетчика в отдельный файл.
Сделать так, чтобы код счетчика загружался только на сервере.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 824](https://teamcity.mesosphere.io/viewLog.html?buildId=58548&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:04:04


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 823](https://teamcity.mesosphere.io/viewLog.html?buildId=58547&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:04:04


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 825](https://teamcity.mesosphere.io/viewLog.html?buildId=58549&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:04:15


We missed that when refactoring, thanks !


why?


We always want to use the last iteration of scores for the statistical model.


Perhaps use a static file server module for this? No need to reinvent that wheel - and this doesn't appear to be considering _../_ or similar, for instance.


sdsfsdf


Generally I take it for an invalid argument to say that a user role "would only be given to the right people". Even if that was true (how can you ever be sure?), then it's the still the point with technical barriers to bar people from doing something that must not do.

A barrier that can be worked around easily is not more than a reminder - and I highly doubt that you would deploy the entire machinery of authorization if you really trusted all ALERT_EDITORs. If that'd be the case, then you appeal to them to stick with the chain of command and not publish alerts on their own.


saw approval from gordonguthrie
at https://github.com/basho/riak_kv/commit/a23ec2c5cfa89d2a596faa829c068de796b99adb


merging basho/riak_kv/fix/at/return_state_with_error = a23ec2c5 into borshop-integration-1222-fix/at/return_state_with_error


basho/riak_kv/fix/at/return_state_with_error = a23ec2c5 merged ok, testing candidate = 41aed106


The ignore import can be removed now.


This seems to make the biggest difference; implemented according to the R code in qvalue().


Ok then. It is just that it is only compatible with latest kernel versions. Then I leave to lucas to tell if he wants to be only compatible with the latest kernel versions.


Sending to multiple destinations at once will reduce the chances of receiving response if we're behind Address or Port restricted Cone NATs.


But you set 10.10+ in the OpenSC part ?


Build was broken in https://github.com/discourse/discourse/commit/b72b72747c0ab023d7471dd5a059270ea4683324


Avoid commented code.


Effectivement ! :p


Yeah, I'm not sure about the usage of `current_time` and clock skew either. I'm not following your suggestion, though. Are you suggesting to update a `TableDescriptor.modification_time` every time the descriptor is written? I'm not seeing how that would help. Or perhaps you're suggesting that the `expiration` time for a lease should instead be its `creation_time`.


![vertalen](https://cloud.githubusercontent.com/assets/11356362/10485958/ab27dac4-7284-11e5-993b-e8111e0aa027.jpg)
A lot of work and for what, peanuts? :-)


indexInstance  may be null here (might have just been deleted)


could be a unique_ptr for enhanced memory leak protection.


@kurimon なぜここはvarなのだ？無しだと動かない感じですかえ？


Distinct isn't required here, you get different instances, and they will be distinct by definition.
We do want to sort it by the etag, though


std::


The command `pip install tendenci --upgrade` won't work for the T7 currently until we push T7 to PyPi. So the safest way to update is to run `pip install -r requirements.txt --upgrade`.


93653630-MultipleYearRunPeriodForTabularReports (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (1911 of 1911 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 826](https://teamcity.mesosphere.io/viewLog.html?buildId=58554&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:03:43


This `options` is also not required...


For completness.

In the context of the `slackPoster`:
- if it listened only to `job:success`, the `job.state` would be `out of sync` (its already processed, but the state would say `SCHEDULED`).
- if it listened to both `job:success` and `job:set-state:success` it will post 2 message to slack for every processed job.


No, you get me the hashed etag, we need the actual GetMostRecentDocumentEtag, and document count would be nice, too


2B *


A goroutine per table does not sound problematic to me, but also unnecessary here. Actually, it is probably easiest to use the same node that gossips the system config (i.e. the leader for the system range).


What's this package for?


Interesting. I wonder if that's unique to our Markdown rendering engine or if it points to a bigger issue. Definitely worth an issue in 18F/federalist


Ha! Was about to post the same comment saying that `phpro/zf-doctrine-hydration-module` still implements `Zend\Stdlib\Hydrator\HydratorInterface` so this module's factory doesn't work if you switch to `Zend\Hydrator\HydratorInterface`. And then your comment pops up while I'm gathering intel.

A temporary fix is to lock composer to `"zfcampus/zf-apigility-doctrine": "1.1.0"`.


Thank you for the reply! I have installed the drake,and running the LittleDog.m again.It's Successed. May i have more question?  How to simulate it with one gait? Thanks a lot!


Подключать JS нужно в конце страницы. Или использовать асинхронную загрузку.


The techpod image is something I took from my masters students thesis (I used gimp to subtract the ground and our old zivi Jan). Then the CAD is from Thomas Mantel (of course!). Timo had the snapshot of the CAD.


Not related to your changes, so OK to leave it as is for now: In many of the code blocks, the assignment of first variable (like stringArray) is pointless if the check two lines down fails, so it can be moved into the if statement as a minor optimization.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 827](https://teamcity.mesosphere.io/viewLog.html?buildId=58558&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:06:10


I'll file an issue.


> implementing it as a generic function, so you can hardly argue with the SAMBRO business logic as a securing factor

Your example of abuse was Alerts, which was where the busienss logic comes in.

I have NEVER disagreed with changing the functionality...just said that your *example* was way overblown.
Anyway, really this thread has gone on *way* too long, especially as we're not really disagreeing on the substance of the change ;)


The length check below is bogus.  I'm not going to rewrite the checks below to work properly because (a) this was already reviewed by others and merged into bleed (so they can fix it), and (b) the check here ensures that it won't mangle any valid names, so it doesn't matter that it is overly restrictive in how it mangles them.


@alalek got this error when adding python export


That's also fine. I think with the current design of the nodes no longer depending on the schema changer directly, we can start with having one node run all schema changes.


 If it's been down for 12 days, then it can wait until tomorrow when I have the tools.  I'll have some time now, which is why I was just checking in today.  Looks like I can get my hands dirty doing something immediately. Where immediately equals tomorrow.

 It does look like the jackal is not being processed. You may try deleting the no jackal file in the meantime, and see if that works.  I tried running the local server earlier, but all of my Ruby stuff is way out of date, and Jekyll only threw errors. I didn't get much more than that.

 If you don't get anywhere with it, don't worry. I can get it out tomorrow local China time.


is ceph-fuse a hard dependency to run a vstart cluster?


This commit is being reviewed as a part of review issue #87.


no idea, but the chrome dpkg install was failing without it :)


Also, `destinations` will in many cases contain endpoints to peers on LAN, these will very likely be the first to respond, but we're least interested in those. The idea in the RFC was that `destinations` would contain peers sorted such that non local peers would appear first and thus would be tried first.


>Did you have any error messages?

no :(

> I will check tomorrow morning, but probably not solved before next week.

thanks @karlcow :heart:


PHPCS: Opening brace should be on a new line