[UI] Cluster Template Issues - Windows Support

Moved over Windows Support specific issues from rancher/rancher#21410
Version: master-head (7/16/19)
What kind of request is this (question/bug/enhancement/feature request):
Bug
List of issues, steps, results for each:

When Set k8s to v1.14 and CNI to Flannel and Enable Windows support, when leverage template windows support is actually disabled for some reason

Create a template like shown, otherwise have defaults (all other overrides disabled. Notice we did not enable override for Windows Support but set it to Enabled.

Try to leverage the template.

RESULT: For some reason the UI shows it as disabled when trying to create a cluster with the template.

You can also have your template set with the override Enabled and set Windows Support to Enabled and for some reason when you leverage the template it is still disabled by default even though template defines it to be enabled.

Flannel Backend disabled when create new clone from default

Note: You can create a clone from the last one created in the last issue.

(If you don't have one already) Create a cluster template like this otherwise with defaults:

Note that Flannel Backend is not grayed-out (not disabled).
2. Create a clone of this (in my case new revision based on default). Notice for some reason Windows Support override is not enabled again, Enable it. Look at Flannel Backend section.
RESULT: Flannel backend is grayed out and I can't change the settings.

Things can break if you try to use the template. I get lots of errors in rancher logs like this:
E0710 22:59:03.033356       7 reflector.go:134] github.com/rancher/norman/controller/generic_controller.go:175: Failed to list *v3.Cluster: json: cannot unmarshal number into Go value of type string
E0710 22:59:03.035129       7 reflector.go:134] github.com/rancher/norman/controller/generic_controller.go:175: Failed to list *v3.Cluster: json: cannot unmarshal number into Go value of type string

Cannot make some cluster template fields overridable with disabled elements:

Create a cluster template. Under the Cluster Options section, set each area to overridable as pictured below.

Now, in order to allow "Windows Support" to be overridable let's pick Flannel Network Provider and Kubernetes version 1.14. Good, now we can set this, but notice -- now we can't access "Project Network Isolation" so we can also make this overridable. It's still allows override, but it's confusing in the UI.

RESULT: I can create the cluster template and it will actually let me override both but it was confusing to set up to me. I had to manually pick flannel first and k8s v1.1.4 just to be allowed to see override switch for "Windows Support"; I would think I should not have this restriction for disabled items in the UI. I.e. Windows Support.

Create template with no quesitons but correct selections for windows support, still shows Windows Support as disabled when try to use template

Create a cluster template, You will fill it out so that Flannel is chosen and k8s 1.14 and newer is picked but DO NOT supply any questions (no overrides).
Leverage the template. For some reason Windows Support is still Disabled even though we did not pick this when we created the template.

RESULT:

We should have seen "Windows Support" "Enabled" since I picked this when made template.

Created another issue for fixing the override issue https://github.com/rancher/rancher/issues/21910. Now we don't allow users to override the Windows settings in master.

Version: master-head (v2.3) (8/12/19)
This changes the way we handle CT for Windows Support. As Logan mentioned, we don't allow users to set an override for the Windows settings.
Here is an example, I'm creating a CT here and do not have overrides for "Windows Support" and "Flannel Backend"

[x] When Set k8s to v1.14 and CNI to Flannel and Enable Windows support, when leverage template windows support is actually disabled for some reason
(See issue description for any additional details)

This is no longer happening. When I leverage the template now, "Windows Support" is enabled now.

[ ] Flannel Backend disabled when create new clone from default

This still isn't working correctly. Flannel Backend is still disabled when I clone my CTR.
Updated steps to reproduce this issue:

Create a new CT, make it like this:

Now clone this revision we just made. Flannel Backend is grayed out but should not be:

Use case: I am cloning my revision and want to make a revision that sets "Host Gateway (L2bridge)"

[x] Cannot make some cluster template fields overridable with disabled elements:

This is effectively fixed because we removed "Windows Support" override. It's still a little confusing because if I switch CNI to Flannel the override switch disappears for "Project Network Isolation" but this is an improvement. Perhaps this is something we can revisit after v2.3 release.

[x] Create template with no quesitons but correct selections for windows support, still shows Windows Support as disabled when try to use template

This is no longer the case. Windows Support is indeed "enabled" now when I leverage my template:

@loganhz could you investigate the 2nd issue? "Flannel Backend disabled when create new clone from default"
Or if you prefer I can make a new issue to track this. This is the only one that still isn't working quite right otherwise the other 3 issues are fine now.

Version: master-head (v2.3) (8/13/19)
This concludes testing. All issues are now resolved. reference https://github.com/rancher/rancher/issues/21557#issuecomment-520596657 (my last comment) for the other 3 tests done.
With the last fix this resolves:
Flannel Backend disabled when create new clone from default
The flannel backend setting is no longer grayed-out (disabled) when we clone our CTR.


object::Env is very high cost

it must be used in ref-form, and its implementation is to be changed from recursive func to linear stack

js and python are also.


fix MissingForegroundServiceTypeException

fix https://github.com/X-SLAYER/flutter_overlay_window/issues/111

This PR fix the android.app. MissingForegroundServiceTypeException when running example project by this way

LGTM!


feat: carbon-table should support valueDom attribute of cells

Fixes #3783

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[ ] 🐛 Bug fix
[x] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

closing in favor of future work #3784


toLeopard: "NaN" is a valid sprite name, and a syntax error!

Terrifying! We caught this while glancing over globals.builtins for #138. Sprites should definitely never be generated with this name.

Infinity too, dang it!


[python-package] fix mypy typing discrepancies in basic.py

Contributes to https://github.com/microsoft/LightGBM/issues/3867.
mypy currently raises the following errors in basic.py file:
python-package\lightgbm\basic.py:1127: error: Incompatible types in assignment (expression has type "_Pointer[c_int64]", variable has type "_Pointer[c_int32]")  [assignment]
python-package\lightgbm\basic.py:1132: error: Incompatible types in assignment (expression has type "_Pointer[c_double]", variable has type "_Pointer[c_float]")  [assignment]
python-package\lightgbm\basic.py:1444: error: Incompatible types in assignment (expression has type "c_void_p", variable has type "None")  [assignment]
python-package\lightgbm\basic.py:1448: error: Argument 1 to "byref" has incompatible type "None"; expected "_CData"  [arg-type]
python-package\lightgbm\basic.py:1499: error: Incompatible types in assignment (expression has type "c_void_p", variable has type "None")  [assignment]
python-package\lightgbm\basic.py:1510: error: Argument 1 to "byref" has incompatible type "None"; expected "_CData"  [arg-type]
python-package\lightgbm\basic.py:1595: error: Item "None" of "Optional[Dict[str, Any]]" has no attribute "get"  [union-attr]
python-package\lightgbm\basic.py:1689: error: Incompatible types in assignment (expression has type "c_void_p", variable has type "None")  [assignment]
python-package\lightgbm\basic.py:1694: error: Argument 1 to "byref" has incompatible type "None"; expected "_CData"  [arg-type]
python-package\lightgbm\basic.py:1827: error: Incompatible types in assignment (expression has type "c_void_p", variable has type "None")  [assignment]
python-package\lightgbm\basic.py:1842: error: Argument 1 to "byref" has incompatible type "None"; expected "_CData"  [arg-type]
python-package\lightgbm\basic.py:1857: error: Incompatible types in assignment (expression has type "Array[_Pointer[c_float]]", variable has type "Array[_Pointer[c_double]]")  [assignment]
python-package\lightgbm\basic.py:1883: error: Incompatible types in assignment (expression has type "c_void_p", variable has type "None")  [assignment]
python-package\lightgbm\basic.py:1887: error: Argument 1 to "c_int" has incompatible type "Optional[Any]"; expected "int"  [arg-type]
python-package\lightgbm\basic.py:1893: error: Argument 1 to "byref" has incompatible type "None"; expected "_CData"  [arg-type]
python-package\lightgbm\basic.py:1905: error: Incompatible types in assignment (expression has type "c_void_p", variable has type "None")  [assignment]
python-package\lightgbm\basic.py:1924: error: Argument 1 to "byref" has incompatible type "None"; expected "_CData"  [arg-type]
python-package\lightgbm\basic.py:1936: error: Incompatible types in assignment (expression has type "c_void_p", variable has type "None")  [assignment]
python-package\lightgbm\basic.py:1955: error: Argument 1 to "byref" has incompatible type "None"; expected "_CData"  [arg-type]
python-package\lightgbm\basic.py:2030: error: Incompatible types in assignment (expression has type "c_void_p", variable has type "None")  [assignment]
python-package\lightgbm\basic.py:2037: error: Argument 1 to "byref" has incompatible type "None"; expected "_CData"  [arg-type]
python-package\lightgbm\basic.py:2322: error: Value of type variable "SupportsRichComparisonT" of "sorted" cannot be "object"  [type-var]
python-package\lightgbm\basic.py:2565: error: Incompatible return value type (got "Union[List[Any], ndarray[Any, Any], Any, Any, None]", expected "Optional[ndarray[Any,
Any]]")  [return-value]
python-package\lightgbm\basic.py:4256: error: Need type annotation for "values" (hint: "values: List[<type>] = ...")  [var-annotated]
python-package\lightgbm\basic.py:4326: error: No overload variant of "__setitem__" of "list" matches argument types "int", "ndarray[Any, dtype[floating[_64Bit]]]"  [call-overload]
python-package\lightgbm\basic.py:4326: note: Possible overload variants:
python-package\lightgbm\basic.py:4326: note:     def __setitem__(self, SupportsIndex, None, /) -> None
python-package\lightgbm\basic.py:4326: note:     def __setitem__(self, slice, Iterable[None], /) -> None
python-package\lightgbm\basic.py:4330: error: "None" has no attribute "ctypes"  [attr-defined]
python-package\lightgbm\basic.py:4336: error: Argument 1 to "len" has incompatible type "None"; expected "Sized"  [arg-type]
python-package\lightgbm\basic.py:4341: error: "None" has no attribute "size"  [attr-defined]
python-package\lightgbm\basic.py:4342: error: "None" has no attribute "reshape"  [attr-defined]
python-package\lightgbm\basic.py:4343: error: Incompatible return value type (got "Optional[Any]", expected "ndarray[Any, Any]")  [return-value]

These errors occur due to missing type annotations in the code and lack of None type validations.
This PR proposes fixing these errors by adding the missing annotations and None type checks.
Notes for Reviewers
This was tested by running mypy as documented in https://github.com/microsoft/LightGBM/issues/3867.
mypy \
    --exclude='python-package/compile/|python-package/build' \
    --ignore-missing-imports \
    python-package/

@IdoKendo are you still interested in pursuing this PR?
If so, please see me comment above about using string literals for hints like ctypes._Pointer, and please update this branch to latest master.
If not (or if you'd prefer to work on something else), just let me know and I'll continue addressing these particular mypy errors.

@IdoKendo are you still interested in pursuing this PR?
If so, please see me comment above about using string literals for hints like ctypes._Pointer, and please update this branch to latest master.
If not (or if you'd prefer to work on something else), just let me know and I'll continue addressing these particular mypy errors.

@jameslamb Unfortunately, not sure I have the capacity at this time, I'll close this PR so you can address it, and hopefully, soon I can get back to helping more.

ok no problem! Thanks so much for all your help so far, you've really helped re-invigorate this type annotation project that we'd been neglecting for a bit. Would love to have you come back and contribute any time.


How to avoid multiple Slide in RecyclerView

Is there any way I avoid swiping 2 cards at same time in RecyclerView? I have requirement to allow user to swipe only one item at any time..

@venugopalraog : did you find a solution? I'm also getting this task for my own project.
Please, post it here if you know how to do. Thanks.

Hi Venu,  try this example -  http://android-pratap.blogspot.in/2015/07/swipe-recyclerview-using.html ,
its clearly explained your requirement

Very nice @tlvprasad, thx ;)


Use Video Thumbnail as Per-Episode Cover

Since according to Apple's specification setting a different cover per episode is also supported.

The itunes:image tag is also supported at the  (episode) level. For best results, Apple recommends embedding the same artwork within the metadata for that episode’s media file prior to uploading to your host server; using Garageband or another content-creation tool to edit your media file if needed.

From my experience, many major podcast clients also support this. I think it would be nice to use a thumbnail of the video as episode cover. This StackOverflow post lists an easy way to get the thumbnails of a known video ID.
https://img.youtube.com/vi/<insert-youtube-video-id-here>/0.jpg
https://img.youtube.com/vi/<insert-youtube-video-id-here>/1.jpg
https://img.youtube.com/vi/<insert-youtube-video-id-here>/2.jpg
https://img.youtube.com/vi/<insert-youtube-video-id-here>/3.jpg

I'd be happy to implement this myself, just not right now. This issue mainly serves as a reminder to myself and a survey if you'd be willing to merge this behaviour?

This would be good. +1


Remove Rebass from the project

Removes Rebass.
Also restructures how the home page scales, hiding the accent image on xs screens and fixing the ever-present horizontal scrollbar.
Also tweaks how buttons look on the Show card (sorry, couldn't help myself)
Album of changes: https://imgur.com/a/2vd1w
Closes #155
Closes #130 (Duplicate PR)

LGTM. 🚀
Only qualm (and technically a non-requirement) I have is on CreateShowForm I would move the date pickers underneath the other form fields on smaller screens – right now they collapse horizontally. I can do this when I get to #156.


bug/DACT-1150-added comma to home address premises

Separated home address premise/house number by comma

Is this a bug?
When viewing the prototype (https://add-and-remove-dev-ready-f0f9fa3155cc.herokuapp.com/address-lookup/correspondence-address)
The address is displayed as :-
"15 New Road, Cardiff, Wales CF5 1BR"
The bug fix now displays as :-
"6, Duncalf Street, Stoke-on-trent, England ST6 3LJ"
Which doesn't seem correct to split house number and street when listing an address?
Saying that, if requirement set by content then I will approve.

Re-read notes from product owner.
Comma is expected so I will approve.


Update default installation namespace

Currently, when installing the dpu-network-operator via the web console, the operator will be deploying in the "openshift-operators" namespace by default. Set this default to "openshift-dpu-network-operator" instead.

@bn222 @wizhaoredhat PTAL

LGTM but you will need a OCPBUG.

/lgtm
/approve

/jira refresh


Fix split-package conflicts with org.hl7.fhir.*.model packages in FHIR Core projects

When the Fhir resource models were moved over to org.hl7.fhir.core, the fhirversion.properties files in hapi-fhir remained in the *.model Java packages (in src/main/resources directory). This created a "split package" situation that is a problem when loading these bundles in the OSGi framework., With OSGi, each Java package can only be resolved to a single bundle/jar and no OSGi tricks (like Fragment-Host) can resolve the issue.
This PR corrects this split-package situation by renaming the folder containing the fhirversion.properties file within the src/main/resources folder. This seems a safe thing to do since it looks like that is only referenced by the IFhirVersion implementation classes.

Heya @bdenton , I will review this in the morning! Had a quick gander and it looks good, just want to vet it against some downstream projects.
Cheers

Once these tests pass, I'll go ahead and get it merged. Looks good to me


missing subdir_glob implementation?

I've been trying out using buck2 on some C++, and i've noticed that while theres references to subdir_glob, it isn't implmented. I'm not sure if that is expressible in starlark or needs to be a primitive in buck2 itself though

It can be expressed in starlark. Here's an implementation I found: https://github.com/LoopPerfect/rxterm/blob/master/subdir_glob.bzl. It looks basically the same as our internal one, maybe that's actually available already too.

We seem to recommend this as the standard pattern with things like android_resource. Should we move subdir_glob into the prelude and perhaps even make it available alongside glob by default?


Unexpected slave response if large request received

If a master node sends a request for more than the number of available registers, this library seems to send a very unexpected response. It just returns a very large, never ending stream of data. Using the code below, with 4 declared input registers, if 5 or more registers are requested by the master the error occurs.
#include <ModbusRTUSlave.h>

#define de PB15
#define re PB14

HardwareSerial rs485_serial(PC7, PC6);
ModbusRTUSlave modbus(rs485_serial, de);

const uint16_t numInputRegisters = 4;
uint16_t inputRegisters [numInputRegisters] = {0};

void setup() {
  pinMode(re, OUTPUT);
  pinMode(de, OUTPUT);
  rs485_serial.begin(19200, SERIAL_8E1);

  modbus.begin(1, 19200, SERIAL_8E1);
  modbus.configureInputRegisters(inputRegisters, numInputRegisters);

  inputRegisters[0] = (uint16_t) 100;
  inputRegisters[1] = (uint16_t) 34;
}

void loop() {
  modbus.poll();
}

Hmm, sounds like it could be an off by one error somewhere. I'll look into it.

I am noticing a couple of things in your code.

The serial object you are using is unfamiliar to me. What processor are you using?
Also it appears you are configuring holding registers, not input registers.

What processor are you using?
An STM32 F4xx MCU.

It appears you are configuring holding registers, not input registers.
Ah, whoops. The error appears when using either input or holding registers. That was just me fiddling to reproduce the error. I'll edit the code for consistency.

I am not seeing the issue when running the following code on an Arduino Uno.
#include <SoftwareSerial.h>
#include <ModbusRTUSlave.h>

const uint8_t rxPin = 10;
const uint8_t txPin = 11;
const uint8_t dePin = 13;

SoftwareSerial mySerial(rxPin, txPin);
ModbusRTUSlave modbus(mySerial, dePin);

const uint16_t numInputRegisters = 4;
uint16_t inputRegisters[numInputRegisters];

void setup() {
  modbus.configureInputRegisters(inputRegisters, numInputRegisters);
  modbus.begin(1, 38400, SERIAL_8N1);

  inputRegisters[0] = 0x002A;
  inputRegisters[1] = 0x0000;
  inputRegisters[2] = 0x0808;
  inputRegisters[3] = 0xFFFF;
}

void loop() {
  modbus.poll();
}

When requesting 4 input registers, this is the response I get: 01 04 08 00 2A 00 00 08 08 FF FF 2D DD.
When requesting 5 input registers, this is the response I get: 01 84 02 C2 C1, which is the appropriate exception response.

So if I use your code the response I get is
Asking for 4 registers works as expected but 5 registers dumps a never ending stream of rubbish after printing the exception code. The first modbus tool I used, mbpoll, stopped listening after receiving the exception code but the bus was clearly full.
This is a screengrab from IoNinja. Blue is my request and green is the MCU's response.

I think I see what is going on here.
When I use RS-485, I normally connect the DE and RE pins of the driver together, so when it is transmitting it is not receiving.
I see in the initial code you posted, you have an RE pin setup.
It looks like whenever a response is transmitted, the library also processes the transmitted message as though it were a request from the master device.
When you request 4 input registers, the library recognizes the function code, but it always expects 8 bytes with that function code, so it ignores the message.
However, when you request 5 input registers, the library sends and exception response: 01 84 02 C2 C1 (02 - ILLEGAL DATA ADDRESS). Then it sees the 01 in the receiving buffer and assumes the message there is addressed to it, and that 84 is the function code, so it responds with with the exception response code: 01 84 01 82 C0 (01 - ILLEGAL FUNCTION).
And it continues to respond to its own exception responses with exception responses.

I've added this bit of code to the end of the _writeResponse function to quickly clear out the receiving buffer after transmitting.
while(_serial->available()) {
  _serial->read();
}

This should prevent the issue you've been having.

Thanks Chris! This does look like it has fixed the issue. It would be great if you could release this to Arduino libraries.

Would this issue be better served by adding an RE pin option to the library? I've noticed that the ArduinoRS485 library seems to have that functionality and it seems like it could be quite useful here too.


dokany-np: Add version <IP_ADDRESS>0

Thought this might be useful (I need it for https://github.com/bailey27/cppcryptfs).
Do you know if I can open a PR to include a manifest for cppcryptfs into the scoop-extras bucket and make it depend on dokany-np?

Thank you! I'm actually not sure, but it might be rejected. Give it a shot anyway, I guess. :smile:


Ignore non existing previous backup.cfg files

There are many reasons this could happen, for example:

the user switched backup stores or deleted files manually
another process deleted that backup after we pulled the volume.cfg
Therefore we should expect the absence of a prior backup and proceed
with a full backup instead.

longhorn/longhorn#1266
Signed-off-by: Joshua Moody<EMAIL_ADDRESS>

Integration test completed successfully


Remove deprecated std::hash::hash

std::hash::hash is deprecated since 1.3.0, and removed at 1.4.0.
Implement own hash in accordance with the doc and use it.

Thanks!


Intercooler Error : TypeError: c is not a function

Any idea why I get this error in console?
jquery 2.2.4 could be the culprit? At docs always jquery 1.x is mentioned.
Thanks in advance.

can you attach the error?

I mean the stack trace for the error?

Any update?  I'd like to see the stack trace for the element, to understand the issue.  It looks like an internal issue or you are using minified code?

ping

Closing out for lack of response


Performance loss

ChirpBanner seems to degrade performance greatly.
Most noticeable if you zoom in and start rotating the camera around; With ChirpBanner, it'll lag severly. Without ChirpBanner, it'll be smooth as butter.
Looking deeper into the code, I'm thinking maybe this has something to do with all the code related to the relative positioning of the Chirps. Maybe the game just doesn't cope well with constantly scrolling text?
What I'm going to try testing is taking out the scrolling functions, so Chirps are just tossed into the banner and go away after 30-60 seconds.

This was due to the way scrolling was implemented.
Same functions, but done inside the IThreadingExtension's OnUpdate() function, and we're good to go!


Change jurisdiction via POST

Operations that change survey should be POSTs not GETs
Problem was introduced by changes from #1133
Fixes #1158

WWRFD


Continuous Delivery for mina-block-explorer

Goal is to continually deploy code changes to staging and production environments

There are security implications on allowing forks to automatically trigger builds from untrusted 3rd parties. We must discuss and decide on a direction at a later time.

GitGuardian has some security best practices (around GitHub Actions) that may provide some helpful insights.

Related Indexer issue: https://github.com/Granola-Team/mina-indexer/issues/212

@jhult Let me know what you need to automate this, please, @jhult. I would like to up-prioritize.

I just sent a request.

@robinbb, I believe there is something for you to review here.

We're still deploying a preview instance: https://buildkite.com/granola/mina-block-explorer/builds/1390#018fc08c-11d4-4725-8a15-12a612abd49f


email tracking

@GalenWright Would you like to be in charge of the hackseq gmail account?

Keep track of upcoming emails
Answer or direct emails to the corresponding person
track google forms, etc.
It would be fantastic if you could help us with that :)
If yes, we need to let everyone know.
Thanks!

Hey @adrisua, sorry for only replying now - we were away in Whistler for the weekend. Good idea, I'm happy to take on that role


changed .sh name to npm-g_nosudo from npm-g_no-sudo

thanks so much for creating this .sh script! your & sindre's work has been really helpful!
the reason for this pullrequest: the sh file name and the command to run the file were named differently (no-sudo vs nosudo). since the repo was named nosudo i removed the dash in the .sh script. this pr would close #7.
if there's anything i can change or fix, please let me know. again, thank you for your work!

Thanks for doing that :+1:


Docker Swarm Support

I don't see Docker Swarm support in the roadmap or mentioned in the docs anywhere, or in this repo. I see some Stack Exchange questions from 2017 indicating there were whispers of Itsio implementing swarm support in the future, but currently see nothing about it in the roadmap.
What's the status here, when can we expect Swarm support, if ever?

Chanwit (Docker Captain on twitter) may have some progress to share https://twitter.com/chanwit/status/1004155314109759488

@chanwit has anyone from Istio followed up on your tweet?  Did you have something WIP for a Docker Swarm adapter?  I contributed to most of the platform adapters in pilot so I would be willing to help or pick up where you left off

@GregHanson Thank you for pining me.
What I have done so far has been based on Consul with Swarm service instances. Ideally, this feature should be using Swarm as the Service Registry. Pilot would be scanning the list of services from a Swarm manager, obtain a list of service instances and convert them to the Istio model.
I've been studying your Pilot codes mainly trying to fit the Swarm Service model to the Consul / Istio model. But I don't have much progress. It would be nice to be working with you guys to make this happen.
Here's my summary:

My Istio proxy injection for Swarm is partially working so far. I intercept container events on every Swarm node and start a proxy container inside the same network namespace for each service instance. But a lot of testing is needed.
Ingress from 0.8.0 is working so far, but I haven't tried 1.0.x with Swarm yet.
The Swarm Service model does not fit well to the Istio model as the Service is an IPVS virtual IP. I'm still not sure how to properly convert it to be an Istio service.

@chanwit We can address the IPVS VIP by adding some logic to the init script to create the IPVS rule instead of an iptables entry.
Once the one rule forwards into the sidecar, Envoy takes care of directing it to the desired endpoints. Beyond this entry, we do not need to continue programming iptables/IPVS like kube-proxy.
That said, we would need to test to ensure that all features are properly working since there may be functionality that is expecting TPROXY to be setup.

I guess support for Docker Swarm is not a high priority. Does the Istio team see Docker Swarm as a distraction? I find Swarm easier to setup and would love to play with Istio+Docker Swarm.

+1 for the support for Istio+Docker Swarm. Hope we will get it!

I think whole Istio approach barely applicable to Swarm due to it's nature — there is no such thing as sidecar container in Swarm therefore you can't run Envoy with each replica of service.


Superset homepage not working/up-to-date

Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if any
[x] I have reproduced the issue with at least the latest released version of superset
[x] I have checked the issue tracker for the same issue and I haven't found one similar

Expected results
https://superset.incubator.apache.org/ and https://superset.apache.org/ should correspond to the most recent production version of Superset (0.25.5 as of writing)
Actual results
The homepage is not up to date, and some links don't work, e.g. https://superset.incubator.apache.org/security.html

It'd be great to automate the publication process or formalize it as part of the release process.

Agreed about keeping docs around for multiple versions. Out-of-the-box apache.org doesn't offer that, we'd have to probably use readthedocs on top of that which is somewhat easy.
Note that for API docs it's super important to have versioned docs but for Superset I feel like it's less critical. Definitely nice to have though.

I fully agree, right now valuable core developer time is best spent implementing awesome new functional features :)


Fix query parameter encoding

This fixes handling of URIs with special characters. Without this patch, segment URIs such as e.g. http://example.com/0.webm?param=%26 are translated to http://example.com/0.webm?param=&.

:+1:


bug: Ionic View App: Images not displayed

Type: bug
Platform: ios 8 webview
Ionic View App: Images that are in a subdirectory of www/img are not included/viewable within the app.

Whats the path being used for the images?

I'm having the same issue, my path is ng-src="/img/open.png"

Moving this to the View issues repo.

Same issue here.
My path is ng-src="/../img/cover.png", but I already tried with ng-src="../img/cover.png" and src="/../img/cover.png" (regular html).
Using iOS 8.3.


Models not being saved

I have been trying to run an A3C chainer script on the environment and although the learning is taking place and is evident in the rendering but the model is not being saved.
When I replace the environment with Cartpole it works fine and the model gets saved.
Could you please update?

I ran the a3c script from chainerRL - https://github.com/chainer/chainerrl
The problem is that when any other different environment is chosen the checkpointing of models works perfectly fine.
In my log, I keep getting the message that the model is being saved but actually nothing is being saved or written.
We have been using this code- https://github.com/Ujwal2910/RL-in-Football
If you could help us figure why only with this environment such an issue is happening it would help us a lot in the research.

Could you share how the log looks like when you run Cartpole vs GFootball?
Also please share "ls -l" output of the outdir for both cases.

for the Cartpole models the log returns-

and for the Football  env it returns-

For the result of ls -I the result is as follows-
(base) ujwal@ujwal-Precision-7920-Tower:~/PycharmProjects/Gfootball/football_models$ ls -l
total 4
(base) ujwal@ujwal-Precision-7920-Tower:~/PycharmProjects/Gfootball/football_models$

and for cartpole it gives-
(base) ujwal@ujwal-Precision-7920-Tower:~/PycharmProjects/Gfootball/cartpole_models$ ls -l
total 620
drwxr-xr-x 2 ujwal ujwal   4096 Jul  1 16:18 200_finish
drwxr-xr-x 2 ujwal ujwal   4096 Jul  1 16:18 4024_except
drwxr-xr-x 2 ujwal ujwal   4096 Jul  1 16:18 best
-rw-r--r-- 1 ujwal ujwal 310639 Jul  1 16:18 model.npz
-rw-r--r-- 1 ujwal ujwal 306909 Jul  1 16:18 optimizer.npz
-rw-r--r-- 1 ujwal ujwal    182 Jul  1 16:18 scores.txt
(base) ujwal@ujwal-Precision-7920-Tower:~/PycharmProjects/Gfootball/cartpole_models$

Thanks for pointing out this issue. After debugging it is indeed a problem in our environment that it changes the current working directory. We are working on a fix.
For now I'd just recommend using absolute paths for your logging. It should work that way.

We just pushed a new release, the fix is included.


Add missing tests_require requirement (unittest2)

unittest2 is listed in test-requirements.txt, but is missing from tests_require in setup.py. It may be worth reading the contests of test_requirements.txt to populate tests_require, but I'd leave that design decision to you :) cc @dstufft

Coverage remained the same at 80.39% when pulling 6bface4f72864f45f6f9bc3b07a7a9d050295798 on koobs:patch-1 into 51db4256ef1ddd091a803139d4977f4c0bc8a377 on samgiles:master.

Hi, thanks for this.  I'll take a look at this when I have my laptop.  I'm
the current maintainer of slumber BTW ;).
Sam
On Tue, 27 Jan 2015 02:58 Coveralls<EMAIL_ADDRESS>wrote:

[image: Coverage Status] https://coveralls.io/builds/1810801
Coverage remained the same at 80.39% when pulling 6bface4
https://github.com/samgiles/slumber/commit/6bface4f72864f45f6f9bc3b07a7a9d050295798
on koobs:patch-1 into 51db425
https://github.com/samgiles/slumber/commit/51db4256ef1ddd091a803139d4977f4c0bc8a377
on samgiles:master.
—
Reply to this email directly or view it on GitHub
https://github.com/samgiles/slumber/pull/104#issuecomment-71582089.

My pleasure Sam :)


ART: Waze App

Link: (www.waze.com/live-map/)
Archived Link: (https://web.archive.org/web/20241107225316/https://www.waze.com/live-map/)

It looks like this is adding multiple links. Please add only one link per PR, thanks!


Update README.md

Looks like the readme wasn't updated when HeaderStorer.HeaderPrefix became HeaderStorer.HeaderFieldName.

Thanks for the PR, merging!


fix(helpdesk-3099) ensure that missing 2.346.x (jdk8 and jdk17-preview) images are published

Ref. https://github.com/jenkins-infra/helpdesk/issues/3099
This PR introduces the following changes:

Bump jenkins plugin CLI to 2.12.8, because it's the version used on the current 2.346.2 and 2.346.3 published images from the master branch:

docker git:(fix/missing-2.346.x-images) ✗ docker run --rm --entrypoint=jenkins-plugin-cli jenkins/jenkins:2.346.1-lts-jdk11 --version
2.12.6
➜  docker git:(fix/missing-2.346.x-images) ✗ docker run --rm --entrypoint=jenkins-plugin-cli jenkins/jenkins:2.346.2-lts-jdk11 --version
2.12.8
➜  docker git:(fix/missing-2.346.x-images) ✗ docker run --rm --entrypoint=jenkins-plugin-cli jenkins/jenkins:2.346.3-lts-jdk11 --version
2.12.8

=> but only for JDK17 and JDK11. For JDK8, we are using 2.17.2, which is the latest version supported.

Linux images:

Ensure that only the 2.346.x are checked by the publication script
$ bash ./.ci/publish.sh -n
Docker repository in Use:
* JENKINS_REPO: jenkins/jenkins
Dry run, will not publish images
Tag is already published: 2.346.1
WARNING: could not get a valid manifest at the URL https://index.docker.io/v2/jenkins/jenkins/manifests/2.346.2-alpine-jdk17-preview.
  (For debugging purposes) manifest={"errors":[{"code":"MANIFEST_UNKNOWN","message":"manifest unknown","detail":"unknown tag=2.346.2-alpine-jdk17-preview"}]}
2.346.2 not published yet
Version 2.346.2 higher than 1.0: publishing 2.346.2 latest_weekly:false latest_lts:false
WARNING: could not get a valid manifest at the URL https://index.docker.io/v2/jenkins/jenkins/manifests/2.346.3-alpine-jdk17-preview.
  (For debugging purposes) manifest={"errors":[{"code":"MANIFEST_UNKNOWN","message":"manifest unknown","detail":"unknown tag=2.346.3-alpine-jdk17-preview"}]}
2.346.3 not published yet
Version 2.346.3 higher than 1.0: publishing 2.346.3 latest_weekly:false latest_lts:true

Ensure that no jdk11 version is built (to avoid overriding images already published)

Cherry-picks from master:

Bumped JDK17 (to use the same as other 2.346.x published images)
Fixed build tooling (test harness, test tools, linter fixes on Dockerfile)
Bumped the OS images (debian/alpine)
https://github.com/jenkinsci/docker/pull/1435 fix on the jenkins startrup script for Windows images

Bumped JDK8 to latest available to benefit from last changes

There are still test error with the JDK17, when passing custom JVM_OPTS or JENKINS_..._OPTS option.

I think the test errors you're seeing were fixed on the main branch in #1409 and/or #1411.

Thanks @basil ! Given the temporary character of this branch (that should be used a few times for publication), I've adapted the behavior:

It should only build and test JDK8 images on public CI
It should only publish:

JDK8 images  by building and publishing as usual
JDK17 "preview" images by copying from their already existing JDK17 images: same shasum to avoid surprises (instead of rebuilding it there), hence fulfilling the "least surprise" principle.

WDYT folks?


Random usage indentation

I've got a simple sub-command that's generating usage with random indentation, with some options having more indentation than others
Usage: emt application create [-hV] [-ev=<emtVersion>] [-g=<groupId>] [-n=<name>] [-pd=<parentDir>] [-pn=<projectName>] [-rsv=<muleRuntimeSubVersion>]
                              [-rv=<muleRuntimeVersion>] <artifactId>
      <artifactId>          Project artifactId
      -ev, --emtVersion=<emtVersion>
                            Enhance Mule Tools version
                              Default: 1.2.7-beta4-SNAPSHOT
  -g, --groupId=<groupId>   Maven group id
                              Default: com.company
  -h, --help                Show this help message and exit.
  -n, --name=<name>         Name of the project directory
      -pd, --parentdir=<parentDir>
                            directory where the project directory will be created
      -pn, --projectName=<projectName>
                            Project name
      -rsv, --runtimeSubVersion=<muleRuntimeSubVersion>
                            Mule runtime sub-version
                              Default: 20201013
      -rv, --runtimeVersion=<muleRuntimeVersion>
                            Mule runtime version
                              Default: 4.3.0
  -V, --version             Print version information and exit.

My code:
@Command(name = "create", aliases = "cr", mixinStandardHelpOptions = true)
public class ApplicationCreateCmd implements Callable<Integer> {
    @Parameters(arity = "1",description = "Project artifactId")
    private String artifactId;
    @Option(names = {"--parentdir", "-pd"}, description = "directory where the project directory will be created")
    private File parentDir;
    @Option(names = {"--name", "-n"}, description = "Name of the project directory")
    private String name;
    @Option(names = {"--groupId", "-g"}, description = "Maven group id", defaultValue = "com.company", showDefaultValue = ALWAYS)
    private String groupId;
    @Option(names = {"--projectName", "-pn"}, description = "Project name")
    private String projectName;
    @Option(names = {"--runtimeVersion", "-rv"}, description = "Mule runtime version", defaultValue = "4.3.0", showDefaultValue = ALWAYS)
    private String muleRuntimeVersion;
    @Option(names = {"--runtimeSubVersion", "-rsv"}, description = "Mule runtime sub-version", defaultValue = "20201013", showDefaultValue = ALWAYS)
    private String muleRuntimeSubVersion;
    @Option(names = {"--emtVersion", "-ev"}, description = "Enhance Mule Tools version", showDefaultValue = ALWAYS)
    private String emtVersion;

Full project is here: https://gitlab.com/aeontronix/oss/enhanced-mule/enhanced-mule-tools

Hi @ymenager, the usage help shows one-character POSIX short options in the first column, and any other options in the second column.

Hi @ymenager, the usage help shows one-character POSIX short options in the first column, and any other options in the second column.

That doesn't explain the mis-alignment. for example both those lines have short options and long options, and they're on different indents:
-n, --name=         Name of the project directory
-pd, --parentdir=
directory where the project directory will be created

That doesn't explain the mis-alignment. for example both those lines have short options and long options, and they're on different indents:
-n, --name=         Name of the project directory
-pd, --parentdir=
directory where the project directory will be created

on, it considers single character options only as short options, i get it

on, it considers single character options only as short options, i get it

Is there a way to disable or customize that behavior ?

Is there a way to disable or customize that behavior ?

Hi @ymenager, yes it is possible to customize this.
Background
By default, the usage help message has a Layout, which has a TextTable with the following columns:

marker for required options (2 chars)
short options (2 chars)
, separator between short and long options (1 char)
long options (by default 24 but customizable with CommandLine::setUsageHelpLongOptionsMaxWidth)
description (remaining, by default 51 for the standard 80-char width)

Customized Example
This ticket has example code to customize that layout.
With the customized layout, there are just three columns:

marker for required options
all options, comma-separated if necessary
description

I believe that adding this code to your application should give the desired result.
Can you give that a try?

Hi @ymenager, yes it is possible to customize this.
Background
By default, the usage help message has a Layout, which has a TextTable with the following columns:

marker for required options (2 chars)
short options (2 chars)
, separator between short and long options (1 char)
long options (by default 24 but customizable with CommandLine::setUsageHelpLongOptionsMaxWidth)
description (remaining, by default 51 for the standard 80-char width)

Customized Example
This ticket has example code to customize that layout.
With the customized layout, there are just three columns:

marker for required options
all options, comma-separated if necessary
description

I believe that adding this code to your application should give the desired result.
Can you give that a try?

Hi @ymenager, did the sample code in #774 meet your needs?

Hi @ymenager, did the sample code in #774 meet your needs?

@ymenager I assume that the layout customization code solved the issue, so closing this ticket.
Feel free to comment here or open another ticket if there is any work remaining. :-)
Enjoy picocli!

@ymenager I assume that the layout customization code solved the issue, so closing this ticket.
Feel free to comment here or open another ticket if there is any work remaining. :-)
Enjoy picocli!


Slash Commands

Yes, it's planned.
No, I don't know when it will be out.
But when I add a slash command, it will be logged here

https://github.com/discordjs/discord.js has been updated to v13, and it includes slash commands. IT MEANS WE CAN START on this


BitString type and codecs reimplemented

Targeting high-performance and convenience to use.
Sampling just BitString en/decoding performance -- new implementation is 100x faster. ;-\

Codecov Report

Merging #25 into master will decrease coverage by -0.91%.
The diff coverage is 83.33%.

@@           Coverage Diff            @@
##           master    #25      +/-   ##
========================================
- Coverage    84.9%    84%   -0.91%
========================================
  Files          23     25       +2
  Lines        3365   3400      +35
========================================
- Hits         2857   2856       -1
- Misses        508    544      +36

Impacted Files
Coverage Δ

pyasn1/codec/ber/encoder.py
91% <100%> (-1.03%)
:x:

pyasn1/type/univ.py
80.2% <72.56%> (-1.68%)
:x:

pyasn1/codec/ber/decoder.py
84.28% <93.75%> (-0.57%)
:x:

pyasn1/compat/integer.py
96.66% <96.66%> (ø)

pyasn1/compat/binary.py
25% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 61ceeae...6794089. Read the comment docs.


Unknown CMake command "add_style_check_target".

CMake Error at cuda/CMakeLists.txt:84 (add_style_check_target):
Unknown CMake command "add_style_check_target".
-- Configuring incomplete, errors occurred!

@VamWolf  add_style_check_target is defined in Paddle/cmake/cpplint.cmake.
Could you show us more details? How to reproduce your issue?

You compile paddle in wrong directory!
The root CMakeLists.txt is in git clone directory.
Please git clone Paddle; cd Paddle, not git clone Paddle, cd Paddle/paddle

no response for long time, so close it.


Make HTML navbar more compact

With version 0.14 of the theme, the navbar is prone to overflowing. E.g., a window with a width of half my screen looks like this:

The changes to CSS proposed here turn it into this:

This removes the fake search field and replaces it by a plain button. Functionality-wise it is a button anyway and opens a dialogue in the centre of the screen just like with version 0.13.
I also shortened the 'About' title in order to conserve a little more space. (Not in the screenshots)
See https://scicatproject.github.io/scitacean/ for how this can look in the wild.

Rebased on the changes of #93. This now only updates versions and changes the search icon.
@SimonHeybrock do you think the risk that this breaks is too high or can we merge?

I misunderstood the docs at first. It is possible to replace the search field with the old style button. This way, we don't need custom CSS. The button is slightly misaligned though:


Module parse failed: Unexpected token type Props

I installed as per the documentation, and immediately upon use this is the error I see:
./node_modules/react-swipe-to-reveal-actions/SwipeToRevealActions.tsx 5:5
Module parse failed: Unexpected token (5:5)
You may need an appropriate loader to handle this file type.
| import './styles.css';
|
> type Props = {
|     children: React.ReactNode;
|     actionButtons: {

Any suggestions or help appreciated!

same, tried it in fresh create-react-app and same result and install as per documentation.
I had some issue with this line Unable to import SwipeToRevealActions, module not found underCompiler options. When commented out webpack was still unable to compile :

Hi! Sorry about this @rmeissen , @bvalasek . Please upgrade to version 1.0.7, and this should be fixed. Thanks!


Pandoc not found

Pandoc is installed:

(as are knitr and rmarkdown packages)
But when doing "Preview HTML", I get this:

Error: pandoc version 1.12.3 or higher is required and was not found.
Execution halted
Markdown conversion failed

I've also tried restarting VS, since I installed Pandoc after getting the message for the first time. But it still can't find it after the restart.

It might have something to do with the installation path. I used the default that Pandoc installer suggested, which is C:\Users\int19\AppData\Local\Pandoc (so it was a per-user install). Do we require a global install?

@jflam, do you remember if you had to do any extra steps above and beyond what's here?
http://microsoft.github.io/RTVS-docs/rmarkdown.html

It needs to be installed globally and VS needs to restart to pick up new PATH.
On Apr 15, 2016, at 18:30, Pavel Minaev <notifications<EMAIL_ADDRESS>wrote:
@jflamhttps://github.com/jflam, do you remember if you had to do any extra steps above and beyond what's here?
http://microsoft.github.io/RTVS-docs/rmarkdown.htmlhttps://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fmicrosoft.github.io%2FRTVS-docs%2Frmarkdown.html&data=01|01|mikhaila%40microsoft.com|bd3ef92f349d4558779b08d36596b132|72f988bf86f141af91ab2d7cd011db47|1&sdata=axG5xXpCuleEOcf%2B3V4syhhi40%2FYSmUFmpNVyAc3rdQ%3D

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHubhttps://github.com/Microsoft/RTVS/issues/1582#issuecomment-210705441

Okay. This needs to be doc'd somewhere, because by default Pandoc installs per-user, and it's not readily obvious that you even have other alternatives (it's hidden under "Advanced").

Hm... I've reinstalled Pandoc globally - it's now in C:\Program Files (x86)\Pandoc - but rmarkdown still can't pick it up (after VS restart). Same exact error message.

Ah, figured it out. Since I was running in experimental hive, I had to restart the "outer" VS instance, too. It works now.

I’ll update the docs to reflect the restart in case they have VS running.
From: Mikhail Arkhipov<EMAIL_ADDRESS>Sent: Friday, April 15, 2016 6:35 PM
To: Microsoft/RTVS<EMAIL_ADDRESS>Cc: John Lam (AZURE<EMAIL_ADDRESS>Subject: Re: [Microsoft/RTVS] Pandoc not found (#1582)
It needs to be installed globally and VS needs to restart to pick up new PATH.
On Apr 15, 2016, at 18:30, Pavel Minaev <notifications@github.com<mailto:notifications@github.commailto:notifications<EMAIL_ADDRESS>wrote:
@jflamhttps://github.com/jflam, do you remember if you had to do any extra steps above and beyond what's here?
http://microsoft.github.io/RTVS-docs/rmarkdown.html<https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fmicrosoft.github.io%2FRTVS-docs%2Frmarkdown.html&data=01|01|mikhaila%40microsoft.com|bd3ef92f349d4558779b08d36596b132|72f988bf86f141af91ab2d7cd011db47|1&sdata=axG5xXpCuleEOcf%2B3V4syhhi40%2FYSmUFmpNVyAc3rdQ%3Dhttp://microsoft.github.io/RTVS-docs/rmarkdown.html<https:/na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fmicrosoft.github.io%2FRTVS-docs%2Frmarkdown.html&data=01|01|mikhaila%40microsoft.com|bd3ef92f349d4558779b08d36596b132|72f988bf86f141af91ab2d7cd011db47|1&sdata=axG5xXpCuleEOcf%2B3V4syhhi40%2FYSmUFmpNVyAc3rdQ%3D>

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHubhttps://github.com/Microsoft/RTVS/issues/1582#issuecomment-210705441
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHubhttps://github.com/Microsoft/RTVS/issues/1582#issuecomment-210705830

Yes, the message is cryptic, it is coming from rmarkdown package without explanation what to do :-( . This is why RS install pandoc locally inside RS installation folder. Pandoc is GPL though and we won't be able to directly preinstall it in Dev15 Willows file-only setup. However, we can consider creating 'third party components' setup as MSI like Cortova tools used to have and offer users to run it at RTVS first launch. Opened https://github.com/Microsoft/RTVS/issues/1589

So it turns out that per-user or global install doesn't actually matter; the only real problem here was caused by running in experimental hive, and not restarting the host VS. On the doc side, John has already updated it to indicate that restart is needed, so this is fixed.

I have installed Pandoc and it looks like RTVS still thinks that I don't have it installed. However, when I run VS2015 in Administrator mode, this seems to work just fine.
Julie

Pandoc is located by system PATH. It can be installed for a single user - but please install it for all users on the machine. You need to restart VS since changes in system environment variables do not propagate to running processes.


Implement SSE _mm_set* intrinsics

This implements SSE intrinsics:

_mm_set_ss
_mm_set1_ps
_mm_set_ps1
_mm_set_ps
_mm_setr_ps
_mm_setzero_ps

Awesome thanks! Looks like the tests may be failing for i686 though?

Hm it looks like this may be segfaulting on the new i586 target? I'm not sure where though... If you run locally with RUST_TEST_THREADS=1 (or push that to CI) it should say what test

Nice!


[BUG] Get-PnPCustomAction Does Not Find Any Custom Actions

Reporting an Issue or Missing Feature
I am reporting a bug
Expected behavior
I am expecting to get a list of custom actions when running Get-PnPCustomAction against a site that definitely has user custom action.
If running `Get-PnPCustomAction``` command that is a part of SharePointPnPPowerShellOnline 3.26.2010.0 module

I am getting the following results:

Actual behavior
If I run the same code from "Expected behavior" section with PnP.PowerShell 0.3.18 loaded

I am getting an error:
Get-PnPCustomAction: No CustomAction found with the Identity "00000000-0000-0000-0000-000000000000" within the scope 'Web

Specifying Web or Identity parameters would be no help (although an error returned is different):

Steps to reproduce
Connect-PnPOnline -Url https://tenantname.sharepoint.com/sites/sitename $PnPSite = Get-PnPSite -Connection $PnPConnectionSite -includes UserCustomActions PnPConnection = Get-PnPConnection Get-PnPCustomAction -ThrowExceptionIfCustomActionNotFound -Connection $PnPConnectionSite
What is the version of the Cmdlet module you are running?
Non-working environment

Name    : Get-PnPCustomAction
Source  : pnp.powershell
Version : 0.3.18

Working Environment:

Name    : Get-PnPCustomAction
Source  : SharePointPnPPowerShellOnline
Version : 3.26.2010.0

Which operating system/environment are you running PnP PowerShell on?

[X ] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

Hi! Thank you for reporting this. I can confirm this issue. We fixed it and the next nightly build (0.3.24-nightly) will contain this fix.

Hi! Thank you for reporting this. I can confirm this issue. We fixed it and the next nightly build (0.3.24-nightly) will contain this fix.


How can I push chart to an insecure OCI registry with helm v3

helm 3.10.0 and after version
Upload a chart to a registry.
If the chart has an associated provenance file,
it will also be uploaded.
Usage:
helm push [chart] [remote] [flags]
Flags:
-h, --help help for push
Global Flags:
--burst-limit int client-side default throttling limit (default 100)
--debug enable verbose output
--kube-apiserver string the address and the port for the Kubernetes API server
--kube-as-group stringArray group to impersonate for the operation, this flag can be repeated to specify multiple groups.
--kube-as-user string username to impersonate for the operation
--kube-ca-file string the certificate authority file for the Kubernetes API server connection
--kube-context string name of the kubeconfig context to use
--kube-insecure-skip-tls-verify if true, the Kubernetes API server's certificate will not be checked for validity. This will make your HTTPS connections insecure
--kube-tls-server-name string server name to use for Kubernetes API server certificate validation. If it is not provided, the hostname used to contact the server is used
--kube-token string bearer token used for authentication
--kubeconfig string path to the kubeconfig file
-n, --namespace string namespace scope for this request
--registry-config string path to the registry config file (default "/root/.config/helm/registry/config.json")
--repository-cache string path to the file containing cached repository indexes (default "/root/.cache/helm/repository")
--repository-config string path to the file containing repository names and URLs (default "/root/.config/helm/repositories.yaml")
but not find -insecure-skip-tls-verify

helm push --debug demo-0.1.0.tgz oci://<IP_ADDRESS>:5000/mytest
WARN[0000] reference for unknown type: application/vnd.cncf.helm.config.v1+json
WARN[0000] reference for unknown type: application/vnd.cncf.helm.chart.content.v1.tar+gzip
DEBU[0000] checking and pushing to                       url="https://<IP_ADDRESS>:5000/v2/mytest/demo/blobs/sha256:9ebda4cd8288db14d228fc0a911b1eeb7e74d55b01fc3534bfe4d7d46c1ffabb"
DEBU[0000] do request                                    request.header.accept="application/vnd.cncf.helm.chart.content.v1.tar+gzip, /" request.header.user-agent=Helm/3.10.3 request.method=HEAD url="https://<IP_ADDRESS>:5000/v2/mytest/demo/blobs/sha256:9ebda4cd8288db14d228fc0a911b1eeb7e74d55b01fc3534bfe4d7d46c1ffabb"
DEBU[0000] checking and pushing to                       url="https://<IP_ADDRESS>:5000/v2/mytest/demo/blobs/sha256:ff1a5b6b030beb6d80ed19ee6573edf7de40803ef73c8502cffb6f2a10ba39d7"
DEBU[0000] do request                                    request.header.accept="application/vnd.cncf.helm.config.v1+json, /" request.header.user-agent=Helm/3.10.3 request.method=HEAD url="https://<IP_ADDRESS>:5000/v2/mytest/demo/blobs/sha256:ff1a5b6b030beb6d80ed19ee6573edf7de40803ef73c8502cffb6f2a10ba39d7"
Error: failed to do request: Head "https://<IP_ADDRESS>:5000/v2/mytest/demo/blobs/sha256:9ebda4cd8288db14d228fc0a911b1eeb7e74d55b01fc3534bfe4d7d46c1ffabb": http: server gave HTTP response to HTTPS client
helm.go:84: [debug] failed to do request: Head "https://<IP_ADDRESS>:5000/v2/mytest/demo/blobs/sha256:9ebda4cd8288db14d228fc0a911b1eeb7e74d55b01fc3534bfe4d7d46c1ffabb": http: server gave HTTP response to HTTPS client

@dreamerkr support is not available for non https based OCI registries. There is a concerted effort to strengthen the capabilities surrounding OCI which is being tracked in #11352

@dreamerkr support is not available for non https based OCI registries. There is a concerted effort to strengthen the capabilities surrounding OCI which is being tracked in #11352

tks,now，How to solve this problem? the best solution use https? but I only want try it

@dreamerkr support is not available for non https based OCI registries. There is a concerted effort to strengthen the capabilities surrounding OCI which is being tracked in #11352

now，we support push helm chart to Docker Registry？

@dreamerkr can you elaborate on your question?

@dreamerkr can you elaborate on your question?

I want push helm chart file to Docker Registry，I can run Docker Registry，Question whether pushing the helm chart file to Docker Registry？

This and #11683 are duplicates. Closing this one as the other has more activity.

@joejulian , reading the description, i don't think the issue are duplicated. This thread was about non https registry, while the other is about self-signed registry. Can you/we reopen it ?

@cridam you are correct, plain HTTP support was not added in the original PR. @souleb was going to look into adding that capability once the first PR was merged.
Reopening issue.


ISPN-11690 Remove File-store relative-to attribute

https://issues.redhat.com/browse/ISPN-11690

There appears to be a few related test failures.

Integrated into master, thanks @ryanemerson !


Agent could be blocked by blobstore access issues

The retry logic for blobstore operations introduced with #258 can block the agent to execute other tasks. This situation can occur when a blobstore operation is triggered from a synchronous task like sync_dns_with_signed_url and there are blobstore access issues. In this case agents will try for 15s to access the blobstore and won't be able to start other tasks or respond to task status requests.  This can lead to failed deployments or unresponsive agents shown by the bosh vms command. We need to rethink how we can implement the retry logic without blocking the agent by this. Some ideas:

reduce the wait limit
retry only from async tasks
retry only on specific errors like network issues

Still relevant.

@mvach Do you have any context on this?

Hi @rkoster,
so this issue is still existing and could potentially occur. Team internally we will run a POC to get some insights how to finally fix the issue. Max could offer timelines ;-)

We cannot reproduce this issue anymore and therefore should close it.

@mvach thanks for getting back on this.


Add VendorConsent.getAllowedVendorIds()

The interface VendorConsent does not currently define the method getAllowedVendorIds. I feel this is missing in comparison to the JavaScript implentation.
I am adding this method and its implementation in ByteBufferBackedVendorConsent, along with updating the relevant JUnit tests.
This addresses #42.

Thanks @ssick  - I will review next couple of days as currently I am swamped.

Sure, no worries @lanusau!

Looks good. Thanks. I will cut new version shortly


Capture Audio PlaybackState from WebView

First of all, thank you very much for the work you're doing with this awesome package.
I wanted to ask if, given the existing functionality of the package it is possible to know if there is a video/audio currently playing in the WebView.

First of all, thank you very much for the work you're doing with this awesome package.
I wanted to ask if, given the existing functionality of the package it is possible to know if there is a video/audio currently playing in the WebView.

setting this prop mediaPlaybackRequiresUserAction to true helped and it works smoothly now for me.

mediaPlaybackRequiresUserAction={false} worked for me

First of all, thank you very much for the work you're doing with this awesome package.
I wanted to ask if, given the existing functionality of the package it is possible to know if there is a video/audio currently playing in the WebView.
mediaPlaybackRequiresUserAction={false} worked for me

Hello, I am having the same problem.
I want to detect audio/video playing from the webview, but cannot find a props that enable me to know that.


Implemented UK_ENGLISH and city: remote option for Indeed and Monster

Implemented UK_ENGLISH Implemented Search by Remote for Indeed US and Indeed Canada
Description
Implemented UK_ENGLISH as an option for 'locale' in the same way USA_ENGLISH and CANADA_ENGLISH are implemented, except with 'co.uk'. Additionally, URL query string cannot take a parameter for state or province for the co.uk site, so indeed.py and monster.py _get_search_url functions were modified accordingly.
Also implemented 'remote' option for 'city', with any placeholder text in 'province' to be ignored. This allows search by remote country-wide instead of searching for any type of labeled remoteness from companies with an indicated location within a city radius. Indicating city:"remote" for city searches by country instead of searching for remote-labeled roles within the city radius.
To implement 'remote' for 'city' _get_search_url in both indeed.py and monster.py were modified in the same way 'province' was discarded in the URL query string from _get_search_url functions for the UK_ENGLISH setting.
city:remote works for Monster and Indeed - US and Indeed - Canada but not Indeed - UK
Context of change
Please add options that are relevant and mark any boxes that apply.

[ ] Software (software that runs on the PC)
[ ] Library (library that runs on the PC)
[ ] Tool (tool that assists coding development)
[ ] Other

Type of change
Please mark any boxes that apply.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested?
Ran UK_ENGLISH  in my_settings.yaml with cities in the UK such as London and Cambridge for Indeed and Monster.
Ran city: "remote" on UK_ENGLISH, USA_ENGLISH and CANADA_ENGLISH for Indeed and found that "remote" is not an option for Indeed.co.uk. city:"remote" has been tested to be used for UK, USA, Canada for Monster.
Checklist:
Please mark any boxes that have been completed.

[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

Codecov Report

Merging #106 into master will decrease coverage by 1.11%.
The diff coverage is 12.32%.

@@            Coverage Diff             @@
##           master     #106      +/-   ##
==========================================
- Coverage   35.68%   34.56%   -1.12%
==========================================
  Files          22       22
  Lines        1390     1461      +71
==========================================
+ Hits          496      505       +9
- Misses        894      956      +62

Impacted Files
Coverage Δ

jobfunnel/backend/scrapers/registry.py
100.00% <ø> (ø)

jobfunnel/resources/defaults.py
100.00% <ø> (ø)

jobfunnel/backend/scrapers/indeed.py
25.49% <4.76%> (-3.09%)
:arrow_down:

jobfunnel/backend/scrapers/monster.py
23.12% <6.66%> (-3.27%)
:arrow_down:

jobfunnel/backend/scrapers/glassdoor.py
27.92% <11.76%> (-2.01%)
:arrow_down:

jobfunnel/backend/scrapers/base.py
38.56% <75.00%> (+0.97%)
:arrow_up:

jobfunnel/resources/enums.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7d0297e...f8d6228. Read the comment docs.


Problem witch installing

I read previous entries and did not find a solution.
pi@raspberrypi:~/Adafruit_Python_DHT $ sudo python setup.py install
Extracting in /tmp/tmpyBCpQu
Traceback (most recent call last):
File "setup.py", line 4, in
use_setuptools()
File "/home/pi/Adafruit_Python_DHT/ez_setup.py", line 145, in use_setuptools
return _do_download(version, download_base, to_dir, download_delay)
File "/home/pi/Adafruit_Python_DHT/ez_setup.py", line 120, in _do_download
_build_egg(egg, archive, to_dir)
File "/home/pi/Adafruit_Python_DHT/ez_setup.py", line 62, in _build_egg
with archive_context(archive_filename):
File "/usr/lib/python2.7/contextlib.py", line 17, in enter
return self.gen.next()
File "/home/pi/Adafruit_Python_DHT/ez_setup.py", line 100, in archive_context
with ContextualZipFile(filename) as archive:
File "/home/pi/Adafruit_Python_DHT/ez_setup.py", line 88, in new
return zipfile.ZipFile(*args, **kwargs)
File "/usr/lib/python2.7/zipfile.py", line 770, in init
self._RealGetContents()
File "/usr/lib/python2.7/zipfile.py", line 813, in _RealGetContents
raise BadZipfile, "File is not a zip file"
zipfile.BadZipfile: File is not a zip file
I use rpi 1 B version

This is the exact same issue that has been posted in other issues. (#91 and #92 ) If the work arounds to download the zip file didn't work you can just install setuptools which I mentioned in one of those two issues. Here is my suggestion again which you should do in a terminal window.
python -m pip install --upgrade pip setuptools wheel

If you get an error from that please let me know but if you don't get an error then you can try to do your install again.

Thx!

@geofbaum I have a different problem now but not a library. Can you contact me by email<EMAIL_ADDRESS>

Yes sure no problem, I will send you an email in a minute or two. Sorry for
the delayed response.
On Tue, May 22, 2018 at 9:15 AM, Piotr Jasiek<EMAIL_ADDRESS>wrote:

@geofbaum https://github.com/geofbaum I have a different problem now
but not a library. Can you contact me by email<EMAIL_ADDRESS>—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/adafruit/Adafruit_Python_DHT/issues/96#issuecomment-390985444,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJPDfPICvvMrkhxcbRapYoYFiVklEvpKks5t1A9_gaJpZM4UBSIg
.

Duplicates #91.


fix(module): code refactor to cjs

:tada: This PR is included in version 1.4.0-beta.3 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Basic Auth not working

when setting basic_auth to true and providing the appropriate secrets, list-flow-functions fails with a 401: invalid credentials.
I do get it to work if I replace handler.go lines 25-31 with a simple httpReq.SetBasicAuth(my_user, my_password)
From the debugging I've done I can see that the credentials are found (using non-existent secrets results in a different error) and can be read. Concerning the contents, I tried using both the ones that are created when setting up OpenFaaS on docker-swarm as well as creating new ones via docker secrete create, so they should be fine.
Platform:
docker-swarm on Docker version 18.09.0, build 4d60db4

@TPei I added some change at https://github.com/s8sg/faas-flow-dashboard/tree/issue-1
would you be able to give it a try ?
I'm not very sure what is causing the problem.

yeah, seems to be working @s8sg

Thank you for the confirmation. I've merged it.


Bunch of updates.

Changelog for the changes included in the pull request:

Added: t_talisman for talismans. These items should be equipped on layer_talisman. Remember to set the layer and add some cool functionality in the scripts :)
Added: t_armor_chain and t_armor_ring typedefs. You can still use t_armor for every piece of armor, but also allows you to use these new types to use specific behavior for each type of armor.
IMPORTANT If you want to use these types, you need to add the following definitions under [TYPEDEFS] in your scripts:
t_armor_chain			302
t_armor_ring			303
t_talisman				304
Added: SPELLFLAG_FREEZEONCAST, when a player casts this spell it gets frozen, opposite to SPELLFLAG_NOFREEZEONCAST
Example use: freeze while opening a gate or recall). MAGICF_FREEZEONCAST must be disabled.
IMPORTANTE You need to add the following definition under [DEFNAME spell_flags] in your scripts:
spellflag_freezeoncast<PHONE_NUMBER>
Added: Chars TAG, TAG.SkillMod%d being %d the SKILL ID. This allows you to increase the bonus for certaing skill instead of modifying the raw base value (just like stats affect skill bonus).
Example: TAG.SkillMod10 = 100 will bonus you 10.0% in Camping (SKILL 10). If you click on "show real" in the skill menu, you won't see this bonus but the real value.
Note that most of the internal calculations are used taking in account stats + tag bonus.
Added: @SpellInterrupt, fired when a char interrupts someone's casting. Default is the interrupter, SRC is the person casting.
ARGN1 = Spell being casted.
ARGN2 = The disturb chance calculated: 0-1000. (Internally, this number is compared against a random of 1000)
RETURN 1 cancels the interruption and allows the target (SRC) to cast.
Added: SKILLUPDATE %d, argument is the skill to update. Use this function with the tag described previously.
Added: [sphere.ini] ColorNotoGoodNPC, you can now set a custom color for good npcs (this should not affect tamed creatures).
Changed: ATTR_QUESTITEM to have internal functionality. These items can't be dropped, traded or looted (as described in script definition).
Changed: ATTR_CANNOTREPAIR to have internal functionality. These items can not be repaired.
Changed: Mining @Start/@SkillStart, now ACT is the Worldgem Bit we're mining from. You can modify it easily instead of using workarounds to get the UID.
Changed: @UserQuestArrowClick, using return 1 now prevents from showing default message when (left/right)clicking arrow.
Changed: Crafting skills now won't display the bounce message when successfully crafting an item, with the skill message is enough.

Which typedefs would you use for armors in the default scriptpack? Switch the chain and ring to the new ones? Though, a lot of typedefs for every armor are missing... In any case, i wouldn't touch the armor typedefs on the source, if someone needs more customization he (or we, in the scriptpack) could add a custom softcoded typedef as a tevents on the armor scripts

Aye, what I did here is just adding the basic armor types which (for me) are plate, ring, chain and leather. You can still use the t_armor for everything, there's no need to change the scriptpack at all, but if you prefer you can split them into the different types. Never thought about the tevents solution.
Anyway is just a few lines change (adding the definition and treating the types as armor), I can keep that commit out.

What I understand, Adding these type on the server side change nothing because action is always the same for T_armor or T_armor_chain. But, it's more evident like this that these kind of armor are different. I think we could implement it like that and eventually if someone want improve something the base will be there.
Heavy armor and medium armor was always the same and should be different. If it's merge, I could modify Script pack to reflect this and be more usefull for admin.
Yes it could be done via script but server can make the first and major division for important categorie.

I'd like to hear from others for this, my main perplexity is about the need of adding additional checks for the other typedefs in scripts. Though, i have to admit that typedefs for weapons are already fragmented, so it wouldn't be so much of a difference in the big picture

For me it's ok, even if i don't think the additional typedef t_armor_ring and chain are necessary.

For me it's ok, even if i don't think the additional typedef t_armor_ring and chain are necessary.

You are right, the type is not complete so it's weird to implement like this (but for me it's better than before). What t_armor we will use for the leaf armor? If we add the type, we should take more time and corretly define them all? Add T_armor_heavy?  And keep t_armor for all other?

I think the original idea for the two armor typedef was to differentiate between metallic and non-metallic armour

Why not add the mining worldgem part also to lumberjacking, fishing etc? or better to all skill with SKF_GATHER


DI consider command line

@samsalisbury - This is mostly a thought in passing; I don't want to burn daylight on it at the moment to determine myself.
I've been noticing that a few of the commands I'm building need to construct things out of Psyringed objects because I need to consider configuration options at the time. I wonder if there's a way to incorporate arguments into the DI, so that everything could "just work" - it might well improve uniformity of interface, as well.

I'm not sure exactly what you mean, could you send some example code of how this would look, or a link to something with a description of how it would change?

After our conversation, I understand the issue. This should stay open, I'll add "CLI flags" as a dependency in the graph so DI can use it.


Moving compute(2) resources to v1beta1 version

Moving compute 2 resources to v1beta1 version

[ ] google_compute_network_firewall_policy_rule
[ ] google_compute_region_network_firewall_policy_rule

Resources require configuration of tag-group resources from this https://github.com/upbound/provider-gcp/issues/162

This ticket depends of #216
216 ticket blocked due to need permissions for Tag creation.


Terraform apply does not stream output

Expected behavior
terraform lifecycle step should stream output to browser while apply is in progress
Actual behavior
After a couple minutes of work creating a cluster on AKS, it just shows terraform apply.

Information

Ship version:

{
    "git": "cdb9646",
    "buildTime": "2018-09-12T15:15:41Z"
}

Docker version: 18.06.1-ce
Command line run:
ship app --log-level=debug --runbook=release.yml

Steps to reproduce the behavior

Save release.yaml:

---¬
assets:¬
  v1:¬
    - azure_aks:¬
        tenant_id: '{{repl ConfigOption "azure_tenant_id" }}'¬
        subscription_id: '{{repl ConfigOption "azure_subscription_id" }}'¬
        service_principal_id: '{{repl ConfigOption "azure_service_principal_id" }}'¬
        service_principal_secret: '{{repl ConfigOption "azure_service_principal_secret" }}'¬
        resource_group_name: '{{repl ConfigOption "azure_resource_group_name" }}'¬
        location: '{{repl ConfigOption "azure_location" }}'¬
        cluster_name: "aksqa"¬
        kubernetes_version: "1.11.2"¬
        node_count: "1"¬
        node_type: "Standard_D1_v2"¬
¬
config:¬
  v1:¬
    - name: Azure¬
      title: Azure¬
      description: Azure configuration and credentials¬
      items:¬
      - name: azure_tenant_id¬
        title: Tenant ID¬
        type: text¬
      - name: azure_subscription_id¬
        title: Subscription ID¬
        type: text¬
      - name: azure_service_principal_id¬
        title: Service Principal ID¬
        type: text¬
      - name: azure_service_principal_secret¬
        title: Service Principal Secret¬
        type: password¬
      - name: azure_resource_group_name¬
        title: Resource Group¬
        type: text¬
      - name: azure_location¬
        title: Resource Group Location¬
        type: text¬
¬
lifecycle:¬
  v1:¬
    - config: {}¬
    - render: {}¬
    - terraform: {}¬
    - message:¬
        level: warn¬
        contents: Done

Run ship app --runbook=release.yml and fill in config with working Azure credentials.

Related to #434, but I can see in the Azure portal that the cluster was created by the terraform lifecycle step.


improve the no-credentials error message, fail fast for no-credentials in colab.

Tested this is working in colab.


Error message (5)

W.r.t. indices and keys placed within curly brackets by mistake (instead of outside), I found three different types of error messages:
'Invalid syntax: Found index inside curly brackets in line '"val1"       : "${list_param[1]}"'. Indices in square brackets have to be placed outside the curly brackets.'
'Invalid syntax: Found index or sub-element inside curly brackets in the parameter '${testlist[${intval}]}''
'Invalid nested parameter format: '${list_param[${dict_param}['A']]}'.The '[<index>]' or '[<key>]' have to be outside of '${<variable name>}'.'

This should be aligned. The JsonPreprocessor should use one detection mechanism only for such kind of issues and therefore should provide only one kind of error message.

Error messages are aligned. Retest successful. Issue can be closed.

integrated in RobotFramework AIO 0.11.0


Cancelable waitForCode() / port assignment fix

Fixes #127 #126

stop() will now release any caller that is currently blocked or will be otherwise blocked on waitForCode(). To ensure this is working as intended, I've added a number of extensive tests.

Jetty itself will now pick an unused port. Fixes #126, and the code has been simplified consequently.

I am not sure why this PR is still showing changes that have been already merged in #135.
@ejona86 @garrettjonesgoogle @anthmgoogle PTAL.

Ah, I didn't sync my fork with the upstream. Now everything looks good. PTAL.

This looks great to me (LGTM). One minor comment.

LGTM.


fix: issue with input and change event for angular, lit, stencil & html

Description
Please provide the following information:
closes #1484
Fixed issue where onInput and onChange treated the same for:

angular
lit
stencil
html

The generators should be able to handle both onInput and onChange if there isn't any special case for the desired framework.
Make sure to follow the PR preparation steps in CONTRIBUTING.md before submitting your PR:

[ ] format the codebase: from the root, run yarn fmt:prettier.
[x] update all snapshots (in core & CLI): from the root, run yarn test:update
[x] add Changeset entry: from the root, run yarn g:changeset and follow the CLI instructions. Alternatively, use the Changeset Github Bot to create the file.

Thank @samijaber  for the snaphot update 👍


DX APM URL

Hello, I am using ca apm 10.8 but I don't know how to create the public url, could you give me an explanation or a guided document on how to do it step by step to create the public url because I already know how to generate the token but I don't know about the url thank you.

Hi daniels570,
the public url is the "webserver port" of the MOM or Enterprise Manager that you are connecting to. By default it is port 8081 (or 8443 if you enabled https). My EM is running on server mytestmom01 with a default configuration, so my url is http://mytestmom01.broadcom.com:8081/.
HTH,
Guenter


Investigate the possibility of implementing a bridge between GDB and JDI

Currently Native-Image can generate debugging information that can be used by GDB:
https://www.graalvm.org/reference-manual/native-image/DebugInfo/
And some common debugging features are supported, including:

break points configured by file and line, or by method name
single stepping by line including both into and over function calls
stack backtraces (not including frames detailing inlined code)
printing of primitive values
structured (field by field) printing of Java objects
casting/printing objects at different levels of generality
access through object networks via path expressions
reference by name to methods and static field data

If we could create a translator between GDB and JDWP, then we could be able to use a java debugger to debug a native-image executable.

Apparently the debug info is not available for LLVM backend.

Apparently the debug info is not available for LLVM backend.

What do you mean or where do you take this from? What I saw in the code and what I tested, debug info is included with the llvm backend too. I could attach gdb to a programm, set breakpoints etc.

But there is no debug information about the original Java code, e.g., the file path, line numbers etc.

But there is no debug information about the original Java code, e.g., the file path, line numbers etc.

Yes, the original source code files is just put into a "sources" folder. But this is also true for non llvm-backend. But line numbers and file path, method names etc. are saved. e.g. I can run "break de/berstanio/x/x.java:50" and it works without the "sources" folder or any other source files. And why should we need the original source code included anyway? We would see the source code over e.g. Android Studio. And in GDB the original source code is just used to print the source code line afaik.

I can run "break de/berstanio/x/x.java:50" and it works without the "sources" folder or any other source files.

Are you sure this works for LLVM backend?

Yes but it only works partially as I know found out. What I missed on my testing was that I just tested to set the breakpoint on the first instruction on the method which is working on the llvm backend. But the other debug infos are gathered over the "sourceMapping" field in the CompilationResult class. And here only the standard "CompilationsResultBuilder" puts in informations, currently not the llvm backend. Therefor setting the breakpoint on the first instruction of the method works, on other places sadly currently not.

Are you able to set breakpoint based on the file and line number?

Yes, but only for the first line of a method.

Btw. I now looked deeper in the code and I'm pretty confident that implementing the complete debbuging feature to the llvm backend should be quite simple.

https://blog.jetbrains.com/idea/2022/06/intellij-idea-2022-2-eap-5/
"Experimental GraalVM Native Debugger for Java"
Looks interesting @Noisyfox !

It seems that the llvm debugging is pushed forward by https://github.com/oracle/graal/pull/4964
It is currently at the same level as my WIP

Seems graal native-image team is working on something like that themselves now: https://github.com/oracle/graal/issues/5648


Home Assistant 0.09

Improve web interface: use interface instead of custom, fix loading and saving of JSON
Draw left/right arrow in UI if further triggers are available

Pretty cool thanks for the improvements -- on thing that I found, when you upload HomeAssistant and not update triggers, the screen is only black. Do you think it would help to show a message "please set triggers in the web-ui" or so?

Yes sure that totally makes sense


Merged master into dev 1.17

@jimangel I'm not able to add the tide/merge-method-merge label from GUI.

These always scare me :-) Assigning @zacharysarah
/assign @zacharysarah

/hold
Waiting for https://github.com/kubernetes/org/pull/1345 to merge so @daminisatya can apply the needed labels.

Looks like https://github.com/kubernetes/org/pull/1345 merged, so:
/hold cancel
Someone else can review for accuracy.
/approve

/hold
Still waiting on merge label from @daminisatya, sorry @zacharysarah!

@jimangel still cannot add a label from GUI. kubernetes/org#1345 is merged!

@daminisatya manually added the label while I look into what permissions are needed to add labels.
/hold cancel
/lgtm
/approve
/cc @zacharysarah

@jimangel Thank you. Looks like the merge was successful because of the label now.


enable show_legend for legacy_style_wofs_summary_alltime

ows web conformance test is failing with below error so happing that this change fix it.
ERROR:root:Request to legend URL https://ows.dev.digitalearth.africa/legend/wofs_ls_summary_alltime/legacy_wofs_summary_alltime_frequency/legend.png
for layer 'wofs_ls_summary_alltime', style 'legacy_wofs_summary_alltime_frequency' returned a non 200 status of 500 Server

Error: Internal Server Error for url: https://ows.dev.digitalearth.africa/legend/wofs_ls_summary_alltime/legacy_wofs_summary_alltime_frequency/legend.png.

WMS URL [https://ows.dev.digitalearth.africa,](https://ows.dev.digitalearth.africa%2C/) version 1.3.0.

Also not sure do will still support legacy configuration or plan to do cleanup??

Do we still need this, @NikitaGandhi ?

I raised this PR because our tests are failing https://github.com/digitalearthafrica/datakube-apps/runs/5713007755?check_suite_focus=true


qt5: regularly scheduled patchset update

Description of changes
Mostly Wayland and crash fixes.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Bisect says 76973ae3b30a88ea415f27ff53809ab8f452e2ec qt5: regularly scheduled patchset update broke qt5.qtwayland in staging:
$ nix build --no-link -f. -L qt5.qtwayland
qtwayland> qmakePrePhase
qtwayland> qtPreHook
qtwayland> unpacking sources
qtwayland> unpacking source archive /nix/store/j83dzn2r3w3ify5zb5rcs1pj6lfrfnqr-qtwayland-df41e7e
qtwayland> source root is qtwayland-df41e7e
qtwayland> patching sources
qtwayland> applying patch /nix/store/lvgbasb9nczf3b9qdr2rp2667ljswx8k-qtwayland-app_id.patch
qtwayland> patching file src/client/qwaylandwindow.cpp
qtwayland> Hunk #1 succeeded at 164 (offset -3 lines).
qtwayland> applying patch /nix/store/m79nczgdf2s95v36lldhxp3igddi9np1-0080-Client-set-constraint-adjustments-for-popups-in-xdg.patch
qtwayland> patching file src/plugins/shellintegration/xdg-shell/qwaylandxdgshell.cpp
qtwayland> Hunk #1 FAILED at 437.
qtwayland> 1 out of 1 hunk FAILED -- saving rejects to file src/plugins/shellintegration/xdg-shell/qwaylandxdgshell.cpp.rej

Pushed a fix already.

qt5.qtwayland was probably fixed by https://github.com/NixOS/nixpkgs/pull/224800


"no such packet" errors in clientError event

Clients keeps throwing error. Do you know what it means?
Also sometimes I get 10s/100s of subscription logs for same client and topic whereas frontend only tried once as below:
2016-01-10T19:58:28.221Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.221Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.221Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.221Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.222Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.222Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.222Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.223Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful
2016-01-10T19:58:28.224Z - info: Client: andr_d1e5c1622809744e, Topic: , Method: authorizeSubscribe; Subscription successful

Regarding the 'no such packet' error: it happens when a client has a subscription (even offline, with clean: false), and it sends a PUBACK (or QoS 2 equivalents) with an unknown messageId. This can be either a bug on this broker or on your client lib.
In your case, I think you are using clean: false, and the client is disconnected by the broker when it cannot match. The subscription cycle happens because the client is disconnected when such event happens. When the client reconnects, the broker tries sending the same packet, and the client acks with an unknown messageId. This is happening over and over again in your code.
Are you using QoS 1 or 2? Have you got some steps to reproduce? I thought I fixed all of those, but I've probably missed some cases. Also, a stacktrace can help debugging this.

Do you have a blog that explains mqtt basics clearly with different options we should be using?
I know clean:false is mainly used for persistence sessions and we were using clean:false which when we changed to clean:true one extra step is getting completed. Also we are using QOS2.
Error Stack Trace
2016-01-10T22:34:01.804Z - error:  Error: no such packet
at MemoryPersistence.outgoingUpdate (/app/nodeApp/hopponnodejs/HopponSimulator/node_modules/aedes/node_modules/aedes-persistence/persistence.js:187:6)
at handlePubrec (/app/nodeApp/hopponnodejs/HopponSimulator/node_modules/aedes/lib/handlers/pubrec.js:13:29)
at handle (/app/nodeApp/hopponnodejs/HopponSimulator/node_modules/aedes/lib/handlers/index.js:43:7)
at Parser.enqueue (/app/nodeApp/hopponnodejs/HopponSimulator/node_modules/aedes/lib/client.js:208:3)
at Parser.emit (events.js:95:17)
at Parser.newPacket (/app/nodeApp/hopponnodejs/HopponSimulator/node_modules/aedes/node_modules/mqtt-packet/parser.js:30:10)
at Parser.parse (/app/nodeApp/hopponnodejs/HopponSimulator/node_modules/aedes/node_modules/mqtt-packet/parser.js:42:48)
at Socket.nextBatch (/app/nodeApp/hopponnodejs/HopponSimulator/node_modules/aedes/lib/client.js:55:23)
at Socket.emit (events.js:92:17)
at emitReadable (_stream_readable.js:427:10)
at emitReadable (_stream_readable.js:423:5)
at readableAddChunk (_stream_readable.js:166:9)
at Socket.Readable.push (_stream_readable.js:128:10)
at TCP.onread (net.js:529:21)
2016-01-10T22:34:01.813Z - error: client error andr_3e619e2221571966 no such packet
2016-01-10T22:34:01.814Z - info: Client disconnected -  andr_3e619e2221571966

Also this particular message is sent from aedes client library to publish using the aedes broker to the topic where above client (poho on android) is subscribed to

That is helpful @guptaashishiitr.
I will work on this and get it fixed. I know what is happening, but I do not know why and how to fix this. I will investigate.

@mcollina  - Appreciate the help. We are planning to launch our startup end of Jan. Not trying to push anything but we want to check if this is something that can be addressed soon. Let me know if I can help in any way. Thanks again.

Are you able to replicate the problem using MQTT.js or just mqtt-packet? I'm having trouble in replicating it in my unit tests.
Which version of Aedes are you running? Also, check what version of aedes-persistence is installed.
As a "quick fix", you can run on QoS 1.

@mcollina  I wanted to check this with you that you asked for mqtt-packet in publish method but I am sending normal packet json with topic, qos etc should I create it using mqtt-packet? (converting it to buffer?)  I am using 0.12.3 and I dont have aedes-persistence installed

@guptaashishiitr no need, I've just find a way to reproduce. The fix is along the way, but I'm currently busy, so it might take a couple of days (but I hope for tomorrow).

@mcollina  - Thank you very much. Appreciate the help.

Released as 0.12.4.

hi
Thanks I tested it but starting to get client disconnects now
1452602353596 INFO new client andr_3e619e2221571966
1452602353597 INFO message published from broker to  $SYS/EkYCGyCDl/new/clients 0
1452602353841 INFO Subscription Package:
{"topic":"user/10204470020541040/trip/5694f3e7c6fbf8144fd4c654/riderequests","qos":2}
1452602353841 INFO Client: andr_3e619e2221571966, Topic: user/10204470020541040/trip/5694f3e7c6fbf8144fd4c654/riderequests, Method: authorizeSubscribe; Entering function
1452602354419 INFO Client: andr_3e619e2221571966, Topic: user/10204470020541040/trip/5694f3e7c6fbf8144fd4c654/riderequests, Method: authorizeSubscribe; Subscription successful
1452602356190 INFO message published from broker to  $SYS/EkYCGyCDl/heartbeat 0
1452602378812 INFO Client disconnected -  andr_3e619e2221571966

Please open a different issue, with steps to reproduce.


Q: How to use in CI

Hi Ken,
Love the simplicity of the TagUI and all the features that come with it.
I was hoping to plug it in our CI flow, but it's not clear how to use the report option or is there an option to output the report as "test coverage"?
Perhaps you have some tips or pointers how to use TagUI in a Continuous integration flow?

I think I found it, by adding the test when running the ./tagui flow_filename firefox testit does all the testing and generates a proper test report.

Thanks @thim81 for your feedback and raising this question. Admittedly test automation is not TagUI's strength. While it can be used for test automation, in the developers reference section, I recommended this list of test automation tools which may be a better fit.
That said, by running TagUI with the test option you can leverage on the test assertions and testing framework of CasperJS. An XUnit XML file will be created which you can integrate with CI tools. There are more details on this in the testing section of developers reference. Below is a sample. We can leave this issue open to see any more inputs from you or other users. Lol we were typing at the same time.
TagUI flow file popup_test
http://tebel.org/index_mobile.php
test.assertTextExists('ABOUT','Check for ABOUT text');
click Open-source software
popup open-source_software
{
test.assertSelectorHasText(tx('gist-data'),'Creative Commons','Check for key words in gist-data element');
snap page
}

Output of tagui popup_test test
TA:Desktop kensoh$ tagui popup_test test
Test file: /Users/kensoh/Desktop/popup_test.js
Wed Jul 26 2017 23:57:48 GMT+0800 (+08)
# /Users/kensoh/Desktop/popup_test
# http://tebel.org/index_mobile.php - Tebel.Automation
PASS Check for ABOUT text
# click Open-source software
# popup open-source_software
PASS Check for key words in gist-data element
# snap page
# http://tebel.org/index_mobile.php - Tebel.Automation
PASS /Users/kensoh/Desktop/popup_test (2 tests)
PASS 2 tests executed in 3.775s, 2 passed, 0 failed, 0 dubious, 0 skipped.
Result log stored in /Users/kensoh/Desktop/popup_test.xml

Generated popup_test.xml
This XML file does not appear to have any style information associated with it. The document tree is shown below.
<testsuites time="3.775">
<testsuite name="/Users/kensoh/Desktop/popup_test" tests="2" failures="0" errors="0" time="3.775" timestamp="2017-07-26T15:57:52.463Z" package="Users/kensoh/Desktop/popup_test">
<testcase name="Check for ABOUT text" classname="Users/kensoh/Desktop/popup_test" time="0.793"/>
<testcase name="Check for key words in gist-data element" classname="Users/kensoh/Desktop/popup_test" time="2.982"/>
<system-out>
<![CDATA[ ]]>
</system-out>
</testsuite>
</testsuites>

@kensoh
I like the clean syntax for writing the test and it does exactly what I needed, so I'm going to try to setup TagUI as test suite in our CI workflow.
If I got it working, I'll post back to share how I did it with other users.

Adding some notes related to CI. There is recent appearance of test / browser automation projects base on headless Chrome. I made a list here - Chromeless, Chrominator, Chromy, Navalia, Lambdium, GhostJS, AutoGCD with short notes.
Also, I've made a commit on the awesome test automation list recommended in developers reference to include these up and coming tools :smile: - https://github.com/atinfo/awesome-test-automation/blob/master/javascript-test-automation.md#web-ui-test-automation

Closing this issue as there isn't follow-up needed from me for now. But do reopen if you run into anything, or reply here to share your findings and tips!