Xenserver/XCP-NG Inter-cluster VM migration with ROOT on managed storage

ISSUE TYPE

Bug Report
Improvement Request
Enhancement Request

COMPONENT NAME

API, UI

CLOUDSTACK VERSION

4.16.0

CONFIGURATION

ACS 4.16.0
1 Zone
Cluster A: Two Xenserver 7.0 hosts
Cluster B: Two XCP-NG 8.2 hosts
Each Cluster has each own NFS primary storage
There is a Zone-wide Solidfire Storage (managed storage)
OS / ENVIRONMENT

VM-A1, on Cluster A,  has a ROOT disk  on managed storage
SUMMARY

The objective is to migrate VM-A1 from Cluster A to Cluster B
VM-A1 migration fails and throws the following message:
Migration failed
Migration of the VM: i-2-11-VM (ID: c87de296-e0e4-4052-81a2-53798cfadd6c) from host xen1 (ID: 7e374139-b522-44da-9ea5-2a03b369f787) to destination host xcp1 (ID: 99d6b992-d6b9-49f2-9cec-3e0394251664) doesn't involve migrating the volumes
STEPS TO REPRODUCE

1. Create VM on Xenserver 7.0 cluster with ROOT disk on managed storage
2. Migrate VM from Xenserver 7.0 Cluster to XCP-NG Cluster

EXPECTED RESULTS

VM should be migrated from xenserver cluster to xcp-ng cluster

ACTUAL RESULTS

VM-A1 migration fails and throws the following message:

Migration failed
Migration of the VM: i-2-11-VM (ID: c87de296-e0e4-4052-81a2-53798cfadd6c) from host xen1 (ID: 7e374139-b522-44da-9ea5-2a03b369f787) to destination host xcp1 (ID: 99d6b992-d6b9-49f2-9cec-3e0394251664) doesn't involve migrating the volumes

Hi @shwstppr, you have worked on inter-cluster migration, is it only supported on Vmware?

@nvazquez I've not tested it myself but I don't remember code conditions that prevent it on other hypervisors. Also, recently in issue https://github.com/apache/cloudstack/issues/5894 it was reported to be working with NFS stores on 4.16 so this could be an issue only with managed storages

Hi @tsinik-dw are you attempting a live VM migration? In that case, do you get the hosts from the different cluster listed on the UI or are using the API? If not using the UI, can you check if the target host is listed on the output of the findSuitableHostsForMigration API?

Can you also please upload the management server logs for the migration attempt? @tsinik-dw

I suspect the issue is that the underlying API invoked is migrateVirtualMachineWithVolume instead of migrateVirtualMachine - can you please try using a client such as cloudmonkey the following API attempt?
migrate virtualmachine virtualmachineid=<VM_UUID> hostid=<HOST_UUID>

Hi, @nvazquez, I am setting up a test environment and reporting the results in a while along with the log.

Hi @nvazquez,
I tried a live VM migration from Xen to XCP. The destination host (XCP, Host 9 in log) is presented as expected  in the UI along with the AutoSelect option. I checked the XCP host as destination and tried the migration.
In the log I can see that  findHostsForMigration command is able to identify the XCP host as a possible destination but the migration fails. Management  log attached live_intercluster_vm_migr_Xen70_to_Xcp82.txt
Your guess is right!... Indeed, the API invoked is migrateVirtualMachineWithVolume instead of migrateVirtualMachine.
But, when the same migration tried using cmk the VM migration fails with the following output:
(localcloud) 🐱 > migrate virtualmachine virtualmachineid=c87de296-e0e4-4052-81a2-53798cfadd6c hostid=73cd3fa4-01ce-4b55-970f-14efd0143aa1
cmd = org.apache.cloudstack.api.command.admin.vm.MigrateVMCmd
jobstatus = 2
jobprocstatus = 0
jobresultcode = 530
accountid = 48ae3b86-8032-11ec-8660-4626a7ce6ce6
userid = 48bd974a-8032-11ec-8660-4626a7ce6ce6
created = 2022-02-16T10:07:28+0200
completed = 2022-02-16T10:07:30+0200
jobid = be5805f5-2330-4033-a314-bde52e666da2
jobresulttype = object
jobresult = map[errorcode:534 errortext:Resource [Host:9] is unreachable: Host 9: Unable to prepare for migration due to Exception: com.cloud.utils.exception.CloudRuntimeException
Message: There are 0 VMs named i-2-11-VM
Stack: com.cloud.utils.exception.CloudRuntimeException: There are 0 VMs named i-2-11-VM
        at com.cloud.hypervisor.xenserver.resource.CitrixResourceBase.prepareISO(CitrixResourceBase.java:4157)
        at com.cloud.hypervisor.xenserver.resource.wrapper.xenbase.CitrixPrepareForMigrationCommandWrapper.execute(CitrixPrepareForMigrationCommandWrapper.java:59)
        at com.cloud.hypervisor.xenserver.resource.wrapper.xenbase.CitrixPrepareForMigrationCommandWrapper.execute(CitrixPrepareForMigrationCommandWrapper.java:36)
        at com.cloud.hypervisor.xenserver.resource.wrapper.xenbase.CitrixRequestWrapper.execute(CitrixRequestWrapper.java:122)
        at com.cloud.hypervisor.xenserver.resource.CitrixResourceBase.executeRequest(CitrixResourceBase.java:1757)
        at com.cloud.agent.manager.DirectAgentAttache$Task.runInContext(DirectAgentAttache.java:315)
        at org.apache.cloudstack.managed.context.ManagedContextRunnable$1.run(ManagedContextRunnable.java:48)
        at org.apache.cloudstack.managed.context.impl.DefaultManagedContext$1.call(DefaultManagedContext.java:55)
        at org.apache.cloudstack.managed.context.impl.DefaultManagedContext.callWithContext(DefaultManagedContext.java:102)
        at org.apache.cloudstack.managed.context.impl.DefaultManagedContext.runWithContext(DefaultManagedContext.java:52)
        at org.apache.cloudstack.managed.context.ManagedContextRunnable.run(ManagedContextRunnable.java:45)
        at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
        at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
        at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
        at java.base/java.lang.Thread.run(Thread.java:829)

The corresponding management log during this cmk operation is attached too cmk_vm_migr_mgmt_log.txt
PS.1 I also tried to specify on cmk both hostid and storageid but got the (expected): "Only one of hostId and storageId can be specified". Log available if desired.
P.2 If I stop the VM on Xen cluster, then I can start it on the XCP cluster as normal. This is easy for my test environment, but (almost) impossible in production :-)

@tsinik-dw
Hi, is global setting migrate.vm.across.clusters set to true ?

Hi @weizhouapache,
Yes, the migrate.vm.across.clusters is set to true.
Just checked (for fun) the migration with this global setting to false and got the exact same error message  (:trollface:), if that is of any help.

Thanks @tsinik-dw - can you please check the output of xe vm-list on the host 9 (xcp1.noc-dev.localdomain)? It seems like the VM is not found somehow

Hi @nvazquez,
the thing is that host 9 is the destination host (the XCP). So the VM is not present there because the migration didn't took place.
Initially, the VM is on host 4 (XEN) , see xe vm-list on XEN:
[root@xen1 ~]# xe vm-list
uuid ( RO)           : ad62d6a1-aed9-a5dc-4c91-9cca448e2fd3
     name-label ( RW): s-22-VM
    power-state ( RO): running

uuid ( RO)           : c391332d-ea29-b6da-fce8-367c75606a5a
     name-label ( RW): i-2-11-VM
    power-state ( RO): running

uuid ( RO)           : 1cad5d03-b217-f99e-aba8-7e50f42e77fd
     name-label ( RW): v-23-VM
    power-state ( RO): running

uuid ( RO)           : db65d88e-8441-3a45-cc61-3210bae79cbc
     name-label ( RW): r-20-VM
    power-state ( RO): running

uuid ( RO)           : 5b6a6762-8845-41fe-9fc8-ba465f668f01
     name-label ( RW): Control domain on host: xen1
    power-state ( RO): running

the same command on host 9 (XCP) is
[13:53 xcp1 ~]# xe vm-list
uuid ( RO)           : 94091d89-92b8-4de6-8297-c9221a3d9d70
     name-label ( RW): Control domain on host: xcp1.noc-dev.localdomain
    power-state ( RO): running

I also attach the host details from DB, in CSV format hosts.txt

Thanks @tsinik-dw, I've been investigating and it seems like the migrateVirtualMachine API does not support this type of migration and indeed the migrateVirtualMachineWithVolume API should be used.
Can you please try passing the migrateto parameter to the migrateVirtualMachineWithVolume API and send the management server logs?
migrate virtualmachinewithvolume virtualmachineid=<VM_UUID> hostid=<HOST_UUID> migrateto[0].volume=<VOLUME_UUID> migrateto[0].pool=<ZONEWIDE_POOL_UUID>

Reference: https://cloudstack.apache.org/api/apidocs-4.16/apis/migrateVirtualMachineWithVolume.html

Hi @nvazquez,
I've got good news!... I tried the command you proposed and the VM named MIGR-VM-SF-2 was successfully migrated from XEN to XCP with the following output on cmk:
(localcloud) 🐱 > migrate virtualmachinewithvolume virtualmachineid=28fb0eed-4277-4f2e-a179-38ed3051c723 hostid=73cd3fa4-01ce-4b55-970f-14efd0143aa1 migrateto[0].volume=291563e5-2ea0-4aaa-b14e-b747d48231cd migrateto[0].pool=2a30020a-76f9-4f47-89f1-a447739b934d
virtualmachine = map[account:admin affinitygroup:[] cpunumber:2 cpuspeed:1200 cpuused:3.68% created:2022-02-16T20:53:28+0200 details:map[hypervisortoolsversion:xenserver61 platform:vga:std;videoram:8;apic:true;viridian:false;device_id:0001;timeoffset:-1;pae:true;acpi:1;hpet:true;secureboot:false;nx:true rootdisksize:6] diskioread:0 diskiowrite:0 diskkbsread:2.2594307e+07 diskkbswrite:1.1737023e+07 diskofferingid:ed9e9144-c46a-4b33-a16d-6a16a32cbafa diskofferingname:VOL-on-SF-200 displayname:MIGR-VM-SF-2 displayvm:true domain:ROOT domainid:ce4fbcaf-8031-11ec-8660-4626a7ce6ce6 guestosid:1dcfb598-8032-11ec-8660-4626a7ce6ce6 haenable:false hasannotations:false hostid:73cd3fa4-01ce-4b55-970f-14efd0143aa1 hostname:xcp1.noc-dev.localdomain hypervisor:XenServer id:28fb0eed-4277-4f2e-a179-38ed3051c723 instancename:i-2-37-VM isdynamicallyscalable:false isodisplaytext:CentOS-7-x86_64-Minimal-2009 isoid:129a82d9-86cd-4323-a5af-f2f0767bb70e isoname:CentOS-7-x86_64-Minimal-2009 lastupdated:2022-02-16T22:20:34+0200 memory:2048 memoryintfreekbs:0 memorykbs:2.09714e+06 memorytargetkbs:2.097152e+06 name:MIGR-VM-SF-2 networkkbsread:0 networkkbswrite:0 nic:[map[broadcasturi:vlan://138 deviceid:0 extradhcpoption:[] gateway:<IP_ADDRESS> id:08b1c5b0-3846-4aa8-a25f-2841254d7aed ipaddress:<IP_ADDRESS> isdefault:true isolationuri:vlan://138 macaddress:02:00:1e:11:00:0d netmask:<IP_ADDRESS> networkid:78a105a3-0e08-4077-89f5-fa929be5d833 networkname:xenserver-net-1 secondaryip:[] traffictype:Guest type:Isolated]] osdisplayname:CentOS 7 ostypeid:1dcfb598-8032-11ec-8660-4626a7ce6ce6 passwordenabled:false pooltype:Iscsi receivedbytes:0 rootdeviceid:0 rootdevicetype:ROOT securitygroup:[] sentbytes:0 serviceofferingid:e3a933aa-269e-4de7-9876-4b79dab6b885 serviceofferingname:SF-200 state:Running tags:[] templatedisplaytext:CentOS-7-x86_64-Minimal-2009 templateid:129a82d9-86cd-4323-a5af-f2f0767bb70e templatename:CentOS-7-x86_64-Minimal-2009 userid:48bd974a-8032-11ec-8660-4626a7ce6ce6 username:admin zoneid:7980e0b4-ebbd-425a-97d3-c09af5890cee zonename:ZONE1]
+
+
(localcloud) 🐱 >

I attach the related management log nv_interclust_vm_migr_sf_success.txt
It is also important that I was able to migrate a VM with ROOT disk on managed storage (Solidfire) and a DATA disk on non-managed (NFS) storage by adding the appropriate entries in the command  for this extra disk!
The cmk output was :
(localcloud) 🐱 > migrate virtualmachinewithvolume virtualmachineid=28fb0eed-4277-4f2e-a179-38ed3051c723 hostid=73cd3fa4-01ce-4b55-970f-14efd0143aa1 migrateto[0].volume=291563e5-2ea0-4aaa-b14e-b747d48231cd  migrateto[0].pool=2a30020a-76f9-4f47-89f1-a447739b934d migrateto[1].volume=e64b3984-65e2-4786-8189-d654deda52ec migrateto[1].pool=bda4aa2b-d38b-341b-a968-98762ac528cf
virtualmachine = map[account:admin affinitygroup:[] cpunumber:2 cpuspeed:1200 cpuused:9.4% created:2022-02-16T20:53:28+0200 details:map[Message.ReservedCapacityFreed.Flag:false hypervisortoolsversion:xenserver61 platform:device-model:qemu-upstream-compat;vga:std;videoram:8;apic:true;viridian:false;device_id:0001;timeoffset:-1;pae:true;acpi:1;hpet:true;secureboot:false;nx:true rootdisksize:6] diskioread:0 diskiowrite:0 diskkbsread:3.0130144e+07 diskkbswrite:1.651675e+07 diskofferingid:ed9e9144-c46a-4b33-a16d-6a16a32cbafa diskofferingname:VOL-on-SF-200 displayname:MIGR-VM-SF-2 displayvm:true domain:ROOT domainid:ce4fbcaf-8031-11ec-8660-4626a7ce6ce6 guestosid:1dcfb598-8032-11ec-8660-4626a7ce6ce6 haenable:false hasannotations:false hostid:73cd3fa4-01ce-4b55-970f-14efd0143aa1 hostname:xcp1.noc-dev.localdomain hypervisor:XenServer id:28fb0eed-4277-4f2e-a179-38ed3051c723 instancename:i-2-37-VM isdynamicallyscalable:false lastupdated:2022-02-17T10:21:24+0200 memory:2048 memoryintfreekbs:1346 memorykbs:2.09714e+06 memorytargetkbs:668210 name:MIGR-VM-SF-2 networkkbsread:51031 networkkbswrite:51031 nic:[map[broadcasturi:vlan://138 deviceid:0 extradhcpoption:[] gateway:<IP_ADDRESS> id:08b1c5b0-3846-4aa8-a25f-2841254d7aed ipaddress:<IP_ADDRESS> isdefault:true isolationuri:vlan://138 macaddress:02:00:1e:11:00:0d netmask:<IP_ADDRESS> networkid:78a105a3-0e08-4077-89f5-fa929be5d833 networkname:xenserver-net-1 secondaryip:[] traffictype:Guest type:Isolated]] osdisplayname:CentOS 7 ostypeid:1dcfb598-8032-11ec-8660-4626a7ce6ce6 passwordenabled:false pooltype:Iscsi receivedbytes:0 rootdeviceid:0 rootdevicetype:ROOT securitygroup:[] sentbytes:0 serviceofferingid:e3a933aa-269e-4de7-9876-4b79dab6b885 serviceofferingname:SF-200 state:Running tags:[] templatedisplaytext:CentOS-7-x86_64-Minimal-2009 templateid:129a82d9-86cd-4323-a5af-f2f0767bb70e templatename:CentOS-7-x86_64-Minimal-2009 userid:48bd974a-8032-11ec-8660-4626a7ce6ce6 username:admin zoneid:7980e0b4-ebbd-425a-97d3-c09af5890cee zonename:ZONE1]

Here is the management log for this second trial nv_interclust_vm_migr_sf_nfs_success.txt

Great news @tsinik-dw! It is the UI missing the migrateto parameter in this case, can you please post a screenshot of the migrate dialog you get on the UI? From your logs shared earlier, I the findSuitableHostsForMigration information retrieved is:
2022-02-16 09:56:37,628 DEBUG [c.c.s.ManagementServerImpl] (qtp1603198149-21:ctx-69ab5152 ctx-32f4cdb3) (logid:c71a4e95) Hosts having capacity and suitable for migration: [Host {"id": "9", "name": "xcp1.noc-dev.localdomain", "uuid": "73cd3fa4-01ce-4b55-970f-14efd0143aa1", "type"="Routing"}]

@nvazquez @tsinik-dw on the migrateto parameter, support was added in https://github.com/apache/cloudstack/pull/5145 and should be available in 4.17.0

Excellent, thanks @shwstppr - closing this issue

@nvazquez,
This is a screenshot of the migrate dialog on the UI (ACS 4.16.0)

...looking forward to see the https://github.com/apache/cloudstack/pull/5145 UI 😄 which is great!

@tsinik-dw this PR is already merged and should be available on the nightly packages, if you can test it please let us know your results

Excellent thanks @tsinik-dw!


AS-600709

Add support for Kerberos authentication to PXF-Hive plugin.
authenticate() in HiveUtilities provides an interface to handle authentication. It also prevents repeated authentication attempts (MetadataFetcher is not the only place that requires authentication; it is also required by HiveDataFragmenter).

Closing this in favour of https://github.com/arenadata/incubator-hawq/pull/17


Pre-trained models

Hi there,
How can I get the pre-trained models for MSRA and ICVL?
Also, in the paper, it is mentioned that 10 epochs was used (with data aug), but the train code has 50 epochs set as default. Which one is correct?
Thanks

Thanks for your question.
You can find the pre-trained models and the corresponding results at release.
For the experiment details, the default hyper-parameters in this codebase should be correct. But it can be different with the arXiv version since we have made some changes after uploading. You can find the new version at IEEE TIM.

Thank you very much for the quick response.
Did you use the code from awesome-hand-pose-estimation/evaluation to generate figures 8 and 9?
('mean error (mm) for each joint' and 'the proportion of frame that all joints error are under the given threshold')
If so, which metric type did you use? max-frame or mean-frame?

Yes, I use tools from awesome-hand-pose-estimation to draw the comparison figures. In particular, figures 8 and 9 in the arXiv version are generated by max-frame.

Thank you very much.


Host output of radiance_cli somewhere online

Right now it's about ~50MB for the HTML file, */{0,51.png}, and *.gif - so it's kind of big to just stick in git? Also it's potentially nontrivial amount of monthly bandwidth (> 1 GB)

Also at this point we should port/write a README

This is mostly done; travis-ci.org builds each push and then writes build artifacts back to the gh-pages branch. We just need to get radiance_cli to run on Travis.

For now I'm manually updating the gh-pages branch periodically. I think this is good enough for now, given that it's hard to actually run radiance_cli on travis.


Can't uninstall Unlocked package

Summary
I can install the package but can't uninstall it. And the error message does not make sense, because I can open the org "brasilcap-dev" from sfdx with sfdx force:org:open.

When I go to Salesforce Setup and try to uninstall the package I get just this message. Which also says nothing about what is the problem.

Steps To Reproduce:
I can't reproduce it unless I share my screen with your team. It is happening on my Customer Dev org.
Expected result
The cli should tell why I can't uninstall the package.
Actual result
The cli does not say why I can't uninstall the package.
Additional information
**SFDX CLI Version: 7.74.1

Hi guys, I still need help with this. I can't understand why I can't remove this package if there aren't useful error messages.


command+option+esc (⌘ ⌥ ⎋) does not open Force-Quit when Karabiner Elements is active

I've seen one reference of this in comments a few months ago, but no other mentions. It seems that the "Force-Quit" shortcut is disabled when K-E is active.
I'm using version 0.91.2, remapping my Caps-Lock key to a Hyper key, on a 2016MBP 15" with touch bar running Sierra 10.12.5

This shortcut is working for me with Karabiner Elements 0.91.3, macOS 10.12.5, 2013 MBP 15". Caps-Lock is set as hyper key if used with any other key and as caps-lock if alone.

Still exists on my computer
Karabiner version 12.1.0
macOS version 10.13.6

Sorry my fault. I need to check touchbar in device menu.

@tekezo could we re-open that issue, as it is not fixed yet?


document typescript usage

I was using the Replicate client in a TypeScript project with an aggressive linter today, and it wasn't immediately obvious to me how to specify the types.
This PR adds a bit of documentation to the README that shows how to import the types ESM-style, and how to use them in a real-world example.

Thanks for the fixes, @erbridge 🩷


The training code hangs with multiple train_data_paths

Describe the bug
I am testing the train.py code with multiple train_data_paths, specially 240 .bin and .idx files. Each .bin corresponds to maximum 1M docs. The code hangs right after printing the BlendedDataset index creation as in the screenshot below. The code hangs without any error. (> 1.5 days once)
To Reproduce
Have many data paths as input.
Screenshots

RANK 5 elapsed time for building blendable dataset indices: 69.91 (sec)
RANK 5 elapsed time for building blendable dataset indices: 0.01 (sec)
RANK 5 elapsed time for building blendable dataset indices: 0.02 (sec)
RANK 6 elapsed time for building blendable dataset indices: 69.95 (sec)
RANK 6 elapsed time for building blendable dataset indices: 0.01 (sec)
RANK 6 elapsed time for building blendable dataset indices: 0.02 (sec)
RANK 7 elapsed time for building blendable dataset indices: 69.99 (sec)
RANK 7 elapsed time for building blendable dataset indices: 0.01 (sec)
RANK 7 elapsed time for building blendable dataset indices: 0.02 (sec)
RANK 4 elapsed time for building blendable dataset indices: 70.05 (sec)
RANK 4 elapsed time for building blendable dataset indices: 0.01 (sec)
RANK 4 elapsed time for building blendable dataset indices: 0.02 (sec)
RANK 1 elapsed time for building blendable dataset indices: 70.09 (sec)
RANK 2 elapsed time for building blendable dataset indices: 70.10 (sec)
RANK 1 elapsed time for building blendable dataset indices: 0.01 (sec)
RANK 2 elapsed time for building blendable dataset indices: 0.01 (sec)
RANK 1 elapsed time for building blendable dataset indices: 0.02 (sec)
RANK 2 elapsed time for building blendable dataset indices: 0.02 (sec)
RANK 3 elapsed time for building blendable dataset indices: 70.21 (sec)
RANK 3 elapsed time for building blendable dataset indices: 0.01 (sec)
RANK 3 elapsed time for building blendable dataset indices: 0.02 (sec)

Environment (please complete the following information):

GPUs: 8 GPUs
Configs: XL.yml and local_*.yml

Have you tried merging the files into one? 240 files is far beyond anything we've ever tested.

@StellaAthena that is our next step, we can mitigate this issue with #files < 128. yet to find the reason behind that.

@StellaAthena Is there a script to merge multiple .bin and .idx files?

@StellaAthena Is there a script to merge multiple .bin and .idx files?

Not currently, no. You can concatenate the text files and reprocess, but I would have to look into the data loading code (which I haven’t looked at in close to a year) to see about merging existing files

Closing as we don’t intend on addressing this any time soon.


Review the Makefile documentation

There are some Makefile targets missing documentation, let's review the current description and update when necessary.

Looking into this!


create vite template react gives typescript errors on jsx files

Describe the bug

vite template react written in JS a bunch of vs code errors.  if i add an empty jsonconfig.json they go away.  any ideas why? 'React' refers to a UMD global, but the current file is a module. Consider adding an import instead.ts(2686) example error and why is this a TS error? yarn create vite newApp --template react is how i created the project and i expected it to work out of the box in vscode
strangly these errors go away when i add an empty jsconfig.json file to the root of my project but this shouldn't be required or should be added as part of the create template script
Reproduction
https://stackblitz.com/edit/vitejs-vite-sjzzfv?file=vite.config.js&terminal=dev
Steps to reproduce
No response
System Info
yarn create vite testapp --template react
open in vs code = see above screen shot

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Did you install the dependencies? I don't see any errors locally.

I got the same error. I try vscode and neovim and is the same. The project works but it's really annoying

Just guessing, is there any tsconfig.json up the directory from the project? TypeScript can recursively pick a tsconfig.json outside of your project too.

I do have another react project that does have a TS config file (both are on my desktop) but it's totally unrelated to this project that would be really terrible design if that's truly the case


use absolute refs when ref name is provided

Use commit.AbsoluteReference() to show the full reference when .spec.ref.name is provided. For eg: refs/heads/main@sha1:<SHA>.

will tag github.com/fluxcdpkg/git and github.com/fluxcd/pkg/git/gogit and update this PR to use that tag after an approval


Sessions are linked together

Hello,
I'm trying to build web app with multiple app modules similar to this Genie tutorial:
https://learn.genieframework.com/docs/guides/adding-reactive-pages#multiple-app-modules
However, sessions seem to be linked together if multiple user open the same route. To confirm, I tried the above Genie example without any modification.
https://github.com/narawat/Ai_playground/blob/master/Screenshot from 2024-03-09 17-12-47.png
The result is the same. I test with Firefox and Edge in private windows.
https://github.com/narawat/Ai_playground/blob/master/Peek 2024-03-09 15-28.gif
Package versions:
Julia v1.10.2
GenieFramework v2.1.0
Is this normal behavior?

Thanks essenciary. Happy to report that your suggest works!. I also make the docs PR.

I think we shouldn't close this, but rather fix it. I'll have a look where the issue comes from.

Just added a potential fix for the issue.
@narawat could you try your original version with the hh-multipage-fix branch?
You can switch to that branch by entering
pkg> add Stipple#hh-multipage-fix

in the package manager.

The example works as it should now. Thanks.
Btw, Is it possible to use async MQTT client (MQTTClient.jl) inside a module? I'm would like to connect a reactive control dashboard to MQTT-based services to monitor/manage/control IoT devices. I think Genie have great potential in this area too.

Thanks a lot for checking!
I'll add tests for this issue and will merge soon.
Concerning MQTT I remember that we had an issue in the past. I propose that you build a MWE and open a new issue with that. Let's see whether we can help you.

Here's a Genie issue containing a MWE. Didn't read through it yet, though.
https://github.com/GenieFramework/Genie.jl/issues/702


It is impossible to use classes containing numbers

Notably Bootstrap has alot of classes like mx-4, px-3, gy-2.

Hi @JohnDowson, thanks for the suggestion and PR!
My primary concern with this (and the reason why numbers weren't permitted originally) is that in Rust proper, 0000069 and 69 are considered to be the same number. But if we allow your proposal, then Maud will introduce a distinction between the two. I'm unsure what implications this might have.
In particular:

Does Rust itself guarantee that number literal tokens are passed through unchanged to proc macros? (So that the token 0000069 is not normalized to 69 or anything else.)
What about other tools, like IDEs or code formatters?
Finally, from a user's point of view, would they be confused by this?

Also, consider that there is already an alternative syntax (."mx-4"). See #267.

For me this is more of a documentation issue than a matter of usability. However since I hit this problem myself and found this issue to resolve it, the documentation was updated and explains how to the "alternative syntax" in order to use numbers in class names.
From my point of view there is no need to spend (probably) a lot of time to evaluate the implications mentioned above.


support rtl

thanks for this fine component,
but when I tried horizontal scrolling while in rtl mode, it simply didn't work
by setting <html dir="rtl"> it shows white empty space.
please fix it so I can use it in my app
much appreciated

Sorry, we don't have time to fix this. If you'd like to fix it, please do so & submit a pull request.
Thanks.

Any updates on that?

#410


DLP-1474: Predefined profiles must be imported for use

Predefined DLP profiles cannot be created. They must be imported to use them. Modify the example code to show this.

thanks for the PR! unfortunately, the file you've edited is a generated artifact of the documentation process. to see how to make changes to the documentation, you're best to check out https://github.com/cloudflare/terraform-provider-cloudflare/blob/master/contributing/registry-documentation.md
additionally, instead of including the import blocks (which are terraform 1.5+ only), can we add a note to the import example? or a note in the docs themselves?

Updated to modify resource.tf

thanks!


Bigtable Admin: Promote models to top level classes

This aims be a mechanical structural change. It promotes the inner classes of TableAdminRequests & TableAdminResponses to be toplevel classes in com.google.cloud.bigtable.admin.v2.models and moves all of the static convertX methods in TableAdminResponses to be static fromProto methods in the relevant model.

@kevinsi4508 can you take a look?

Sure, taking a look.

Thank you for reviewing.  All feedback should either be addressed. Please note that I added TODOs for most of the comments because these are pre-existing issues that I plan on addressing in followup PRs and this PR is pretty noisy as is

LGTM

All feedback addressed. Thanks for reviewing.
@garrettjonesgoogle would mind taking a look?


Increase CRI test timeout

Summary
CRI timeout of 20 seconds seem too tight for executing on a self-hosted runner in a multipass VM.
Excluded scale-from-0 test due to its sporadic failure. Raised #655

Obsolete


jotai-history is not in the npm registry

jotai-history is now on npmjs: https://www.npmjs.com/package/jotai-history

Thanks for resolving this! I appreciate the quick turnaround. Everything’s working fine 👍


Installation OpenShift with Operators

Hello, I'm wondering whether there are any instructions on how to install KServe standalone (Serverless) on OpenShift with the appropriate operators (kiali, jaeger, service mesh, cert manager, serverless) from the catalog (+ the models web app)?
There is a ~2 year old instruction under docs, however that is quite outdated I guess.
Service Mesh seems to be up and running:
$ oc get pods -n istio-system
NAME                                   READY   STATUS    RESTARTS      AGE
grafana-847757d67c-rwzmc               2/2     Running   0             5d19h
istio-egressgateway-7b47b5b78c-ln28c   1/1     Running   0             5d19h
istio-ingressgateway-bc446949b-2tc88   1/1     Running   0             5d19h
istiod-basic-59d7548c55-6g6vz          1/1     Running   0             5d19h
jaeger-6bfb9c97bc-qklx9                2/2     Running   0             2d2h
kiali-7b4c9db8ff-jdwwq                 1/1     Running   2 (11h ago)   5d7h
prometheus-664b4b8975-gmcxc            2/2     Running   4 (45h ago)   5d19h

Kserve + webapp is also installed:
[root@p1217-bastion ~]# oc get pods -n kserve
NAME                                         READY   STATUS    RESTARTS           AGE
kserve-controller-manager-5d577f846b-5xgcm   2/2     Running   1005 (6m24s ago)   5d7h
kserve-models-web-app-7f574c68-p6pkk         1/1     Running   40 (141m ago)      5d7h

I would expect to reach the webapp under istio-ingressgateway-istio-system.apps.<cluster-url>/kserve-endpoints however that doesn't work 🤔
Any input is appreciated :-)

@mgiessing please checkout the latest openshift doc https://github.com/kserve/kserve/pull/2855


On windows, for wildcard dns to work the * must be removed from the NRPT rule

the NRPT will not work properly if the star is left on the intercept address. Example
correct:
Add-DnsClientNrptRule -Namespace ".blue" -NameServers "<IP_ADDRESS>"

current rule being added:
Add-DnsClientNrptRule -Namespace "*.blue" -NameServers "<IP_ADDRESS>"

fixed/closed in the desktop-edge-win branch


A.2.7 Federated Discovery Service

This issue has been extracted from the issue list on:https://ies-svn.jrc.ec.europa.eu/issues/2685
Comment
The validation can only be done knowing the Discovery Scenario chosen when registering the service in the INSPIRE Geoportal.

Proposed Resolution

First check whether the operation constraint called ôFederatedCatalogues" is present in the Discovery Service's GetCapabilities document. XPath: "/ows:OperationsMetadata/ows:Constraint[@name='FederatedCatalogues']"
Then, check whether "/ows:OperationsMetadata/ows:Constraint/ows:Value" contains the URL of the federated Discovery Service.

Also, add A.2.6 as prerequisite if issues on A.2.6 (issue #39) can be resolved (step 1 can then be removed)

I'd missed this one because it changed status before I looked... see #39

Added label feedback:for-discussion as this depends on issue #39, which is also flagged for discussion.

Federated option will be removed, as discussed during the MIWP-5 meeting on 2016-04-28


Out of date instructions

the instructions don't accurately describe recent changes to the M365 Defender portal, please update the images and text

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 55901976-4431-c470-88cb-af7430bb7339
Version Independent ID: 0e80c507-335d-bd5e-7315-b8fe30aec0ff
Content: Grant access to managed security service provider (MSSP)
Content Source: microsoft-365/security/defender-endpoint/grant-mssp-access.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@mjcaparas Please help us in resolving this issue. Thanks

Will update


move hideKeyboard to removeKeyboard

hide Keyboard is deprecated since Bot api 2.3
I tested it with my bot

From Telegram API docs:

hide_keyboard will still work for a while for backward compatibility.

Maybe we can support both for now? Similar to this.
Also, it would make sense to add some kind of internal deprecated function to output a warning/error to the log, that an obsolete function is being used.

I thought about it but is not the same case. For database if you don't have retrocompatibility you cannot replay the logs so you lose data. Instead this change has no impact on the data and there's no meaning to support it anymore in the library. Everybody need to change the method in the Commands if they use this.

I was thinking to give users time to change, not just suddenly stop supporting it.
Having said that, if they update to 0.36 they will need to change many other things anyway, so you're quite right.
I'll merge then, ok?

ok!


Factor out array extraction from derived table instance_nature_content

The derived table instance_nature_content appears to trigger a bug in PostgreSQL under some circumstances.  In the past we have worked around this bug by doing array extraction in one query and additional joins in another query, rather than doing them both in a single query.

This solution is not correct - closing.


Remove Uninitialized state from connection

Summary of Bug
We should align with the ICSes.

@adizere Is there a particular reason the Rust implementation uses the Unitialized state?
If not, I suggest we remove it and likewise in the SDK (see above comment) to match the specification, then close this issue.

Originally posted by @cwgoes in https://github.com/cosmos/ics/issues/452#issuecomment-683810821

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

We should wait for the protobuf change in SDK before we proceed with this.

For reference, the SDK issue tracking this work is https://github.com/cosmos/cosmos-sdk/issues/7175.

Channel messages need a significant rework and cleanup (#315 ) and the change for the channel state will be done there.


Insta 1.41.0 finds spurious duplicate snapshots from tests in package root

What happened?
When insta is used in tests located in the package root (specified via the [[test]] Cargo.toml key), the CLI will find each generated snapshot twice, even though only one file is generated per snapshot.
Reproduction steps
The following is a minimal workspace setup to reproduce the issue:
$ ls
Cargo.toml test.rs

Cargo.toml:
[package]
name = "insta-testing"
version = "0.1.0"
edition = "2021"

[dev-dependencies]
insta = "=1.41.0"

[[test]]
name = "test"
path = "test.rs"

test.rs:
#[test]
fn some_test() {
    insta::assert_snapshot!("name", "content");
}

cargo insta test yields the following output and resulting workspace state:
$ cargo insta test
    Finished `test` profile [unoptimized + debuginfo] target(s) in 1.40s
     Running test.rs (target/debug/deps/test-68ebf84566e4320f)

running 1 test
stored new snapshot /home/julia/code/personal/insta-testing/snapshots/test__name.snap.new
test some_test ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.04s

info: 2 snapshots to review
use `cargo insta review` to review snapshots
$ tree -I target
.
├── Cargo.lock
├── Cargo.toml
├── snapshots
│   └── test__name.snap.new
└── test.rs

2 directories, 4 files

Running cargo insta review at this point will present the same snapshot twice. cargo insta pending-snapshots will list the same snap.new file twice. Running cargo insta accept will complain (once) about the snap.new file having been removed by another process, presumably upon failing to find it again after accepting and removing it the first time.
Insta Version
1.41.0
rustc Version
1.82.0
What did you expect?
I would have expected the snapshot to be found by the CLI only once; only reported as one snapshot from insta test and insta pending-snapshots, only presented once by insta review, only removed once by insta accept, etc.
Prior versions of insta display the expected behavior; setting the insta dependency version to =1.40.0 and installing<EMAIL_ADDRESS>yields the expected behavior.
As far as I've been able to tell, this behavior only occurs when the test is in the package root directory. For instance, it does not occur (i.e. the expected behavior occurs) if test.rs is in the default tests directory, or if it's in a different subdirectory specified via the [[test]] key. However, changing the snapshot directory via with_settings! (or manually via Settings::bind) does not affect the behavior; it seems to depend only on the location of the test itself.

Thanks for the issue. We're more "advanced" about looking up the snapshots for each target now, including the paths that each target specifies — previously we would miss something outside the root (e.g. https://github.com/mitsuhiko/insta/issues/625)
Unfortunately that doesn't work so well here, because the file could belong to the root or to the tests target, and so insta is counting it as both.
I think probably the solution is to dedup when we're presenting the number of snapshots etc, though I'll think about whether there's some way of defining the file as belonging to only the tests target.
If you need a workaround in the short-term, passing --tests should work (lmk if it doesn't, would mean my understanding is wrong)

Hm, doesn't seem to work. cargo insta test --tests yields the same behavior with the same setup.

OK thanks a lot @wuggen , I can reproduce, and my --tests suggestion wasn't correct. I will look into it shortly.


Feature/pictogram pos db

Status
PRODUCTION / DEVELOPMENT
Description
A few sentences describing the overall goals of the pull request's commits.
Related PRs
List related PRs against other branches:

branch
PR

other_pr_production
link

other_pr_master
link

Todos

[ ] Tests
[ ] Documentation

Steps to Test or Reproduce
Outline the steps to test or reproduce the PR here.
roslaunch pkg_A executable_A

The car should move.

@masa1228masa Please add a description about this PR.


Make the Application aware of the old version of the installer and remove it if found.

This issues presents some problems/questions:
The installer complains when trying to delete the folder containing Erlang due lack of privileges.

According to @ https://github.com/beltex/SMCKit/issues/2, using Authorization Services  with Swift is broken since:
a. AuthorizationExecuteWithPrivileges() is deprecated
b. Swift class FileManager has no way to request permissions.
c. An alternative is to write a helper via launchd, but doing this will need code signing. (More details here:
http://stackoverflow.com/questions/6841937/authorizationexecutewithprivileges-is-deprecated)
c. It looks like the only way to prompt the user to escalate privileges and without code signing is by using class NSAppleScript as
described here: http://stackoverflow.com/questions/6841937/authorizationexecutewithprivileges-is-deprecated using AppleScript. (I tested it and works).

On usability:

The installer is currently checking the presence of the old installer at "first launch".
a. Should it be done each time the application launches?
ab. Should it prompt only if it's an updated version?

Other approach:
a. Should this issue be	solved just by just changing	the prompt to	 "A deprecated ESL Erlang installation has been
found at /xxx/xx/xx.."  and leave the user the decision to manually uninstall the old version?

closing out - pending confirmation


Control Plane LoadBalancer

Describe the solution you'd like
actually when running in k3s ha mode with an external database one still needs to have a "Configure the Fixed Registration Address" this could be overcome with a built-in kube-vip (https://github.com/plunder-app/kube-vip), however that would mean that some needs to provide a VIP, but I think that might be the lower evil
btw. even kubeadm now introduces kube-vip: https://github.com/kubernetes/kubeadm/blob/master/docs/ha-considerations.md#kube-vip
and it's also possible to use it directly with it:
https://kube-vip.io/control-plane/#first-node
Describe alternatives you've considered
keep k3s as it is, however that would mean that it would still be a hassle in a ha scenario (it still is since etcd is not built into it).

Another simple thing to do would be to just use a DNS alias maintained by ExternalDNS or something.

It would be great if the Kubelet would be made multi-master aware, so that a load balancer wouldn't be needed for HA setups.

@nustiueudinastea the apiserver is already multi-master. The issue is that clients need to be given a fixed address in order to do the initial bootstrapping. This can be a loadbalancer, dns alias, or just the address of one of the servers.

@brandond sadly I did not answer directly but unfortunatly a dns alias is most often not possible especially not in the IoT world.
So I think it would be good to include something that is as simple as saying --master-loadbalancer-ip= if not specified k3s does not configure a ha ip or use the code at all.

@schmitch you can manually setup kube-vip with k3s server cluster. if it satisfies your environment, please feedback. Currently, this is not high priority request for the use cases.

Why not accept a list of IPs representing all the available masters?

Why not accept a list of IPs representing all the available masters?

this also needs some extra code to filter an IP to interact with the agent. if the master gets down or add more master nodes, we also need to update the IPS in your ways.

Indeed, this is more complicated and thats what I meant when I suggested multi-master support in the kubelet. While it would add some complexity in the implementation, it would also make k3s operationally less complex in HA environments and much easier to setup. An LB is often not available in self-hosted or IoT environments.

Indeed, this is more complicated and thats what I meant when I suggested multi-master support in the kubelet. While it would add some complexity in the implementation, it would also make k3s operationally less complex in HA environments and much easier to setup. An LB is often not available in self-hosted or IoT environments.

for ipvs support node, we can use ipvs to create a vs in local node, add host alias to this vs name. it also get rid of this control plan issue.

Note that k3s only needs a stable registration endpoint for the initial startup of the agent. After that it uses a client load balancer to route connections between available servers.

@brandond thanks for your reminder. so k3s don't need to add more components to handle the ha logic.

yeah this can be closed of now, I do not think that many IoT devices need a highly available registration ip. It might need to be documented tough.

actually when thinking about it, it might still make sense to provide that. so that services like clilium can work without kube-proxy: https://docs.cilium.io/en/v1.9/gettingstarted/kubeproxy-free/ (this needs high available api server)


chore: the errors in the document have been corrected

What ❔

fixed typo

Why ❔

the errors in the document have been corrected

Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[x] Tests for the changes have been added / updated.
[x] Documentation comments have been added / updated.

Also, it is a duplicate of: https://github.com/matter-labs/era-contracts/pull/105/files


Fix handling for HTTPFileSystem URIs

In order to load EMIT data over HTTP, it's best to use a token to authenticate using fsspec as documented in this comment.
This below is working code, but it requires this attached fix.
s3_url = "s3://lp-prod-protected/EMITL2ARFL.001/EMIT_L2A_RFL_001_20230123T004529_2302216_003/EMIT_L2A_RFL_001_20230123T004529_2302216_003.nc"
s3_url = s3_url.replace("s3://", "https://data.lpdaac.earthdatacloud.nasa.gov/")

fs = HTTPFileSystem(headers={
    "Authorization": f"bearer {token}"
})
# ds = xr.open_dataset(fs.open(s3_url))
ds = emit_xarray(fs.open(s3_url))
ds

Thanks @alexgleith
Glad you were able to get HTTPS access working. We are planning add some instructions for HTTPS access to a future notebook. For other users in the meantime, the easiest way to access and work with an EMIT granule via HTTPS would be something similar to what you posted above:
# Use earthaccess library to login and retrieve a token
earthaccess.login()

# Provide granule URL
url = 'https://data.lpdaac.earthdatacloud.nasa.gov/lp-prod-protected/EMITL2ARFL.001/EMIT_L2A_RFL_001_20220903T163129_2224611_012/EMIT_L2A_RFL_001_20220903T163129_2224611_012.nc'

# Get an https fsspec session
fs = earthaccess.get_fsspec_https_session()

# open granule using xarray or emit_xarray function from emit_tools.py
with fs.open(url) as file:

    # Using xarray (only reads root group)
    dataset = xarray.open_dataset(file)

    # Or using the emit_xarray function
    dataset = emit_xarray(file, ortho=False)


board: ti: cc1352p7_lp: Add complementary instructions about OpenOCD

Currently, OpenOCD doesn't support the cc1352p7.
There are two open PR adding its support to Zephyr's OpenOCD[1] and upstream OpenOCD[2].
Until this gets merged, we need to use a downstream version of OpenOCD. This updates the documentation to explain where to find it and how to use it.
[1] https://github.com/zephyrproject-rtos/openocd/pull/65
[2] https://review.openocd.org/q/owner:abailon%2540baylibre.com

Thanks @anobli , I tested locally with your version of OpenOCD and it is working

I should probably add the same instructions to beagleplay and beagleconnect freedom


[1.8.1]: ^C on pull broken

^C on a pull is unrecoverable, and does not even show the message "layer being pulled by another client"

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

I think this is a dupe of #6928

this is a regression
On Fri, Aug 14, 2015 at 1:19 PM, Andy Haden<EMAIL_ADDRESS>wrote:

I think this is a dupe of #6928
https://github.com/docker/docker/issues/6928
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/issues/15589#issuecomment-131228985.

Is this a "good first bug"?

@hekaldama probably not. The distribution code is rather tricky.

@calavera okay. Thanks for the reply. Will keep looking for stuff to work on.


Are extern functions methods or functions

In Appendix F in the "can be a run-time parameter..." table, there's a column for functions and methods but there isn't a column for extern functions. Should they be classified under functions or methods, or can the methods column be renamed to externs to clear that up?

Can you create a pull request for this fix?


UUID as ID key for entities

🚀 Changes proposed by this PR

The extension does not work for schema files that use uuid.UUID for ID fields. This fix checks for the flag AllowClientUUIDs and takes necessary action.
🔗 Related bug reports/feature requests

fixes #69

🧰 Type of change

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

📝 Notes to reviewer

🤝 Requirements

[x] ✍ I have read and agree to this projects Code of Conduct.
[x] ✍ I have read and agree to this projects Contribution Guidelines.
[x] ✍ I have read and agree to the Developer Certificate of Origin.
[x] 🔎 I have performed a self-review of my own changes.
[x] 🎨 My changes follow the style guidelines of this project.

[x] 💬 My changes as properly commented, primarily for hard-to-understand areas.
[x] 📝 I have made corresponding changes to the documentation.
[x] 🧪 I have included tests (if necessary) for this change.

Given there is a concrete definition for what the a UUID actually is defined by RFC 4122, it seems like ogen or this extension should know how to marshal [16]byte regardless of which library is being used to actually generate UUIDs. For reference:
// A UUID is a 128 bit (16 byte) Universal Unique IDentifier as defined in RFC
// 4122.
type UUID [16]byte

@jordanfowler while there may be an RFC for UUIDs, to my knowledge, Ent does not impose those restrictions, so we should not. I'm confident some Ent users use "uuid" fields in which are not using an RFC-compliant solution.

Sorry, I'm super late on getting to the implementation of this, but take a look at https://github.com/lrstanley/entrest/pull/81 and let me know your thoughts. Plan on merging sometime tomorrow.
Wanted to make sure the approach used overlaps as much as possible with how Ent marshals/unmarshals UUID (or custom) ID fields. It's not exactly the same, but it's very close (since Ent doesn't expose reusable logic for it).
I've tested with a few different UUID implementations (and other ID types, like string), and seems to work well. Feel free to test yourself and let me know if you have any concerns with my approach.


Missing validation in FlexOrder => missing OptionReference when FlexOffer has more than one option

In the specification of a flex order request, there are two possible scenarios:

the flex offer mentioned in the flex order has one option only => this is fine
2.the flex offer mentioned in the flex order has more than one option => is this case the optionReference must be
filled in in the flex order

There is no explicit validation for case 2: what if there are more than one offer options in the flex offer, and there is no optionReference filled in; it would be helpful to make this an explicit validation with its own error message.

Change section on FlexOrder:
Wording of "(only if the FlexOrder contained multiple OfferOptions) " is incorrect: it should be FlexOffer.
Proposal: move above sentence to the table of attributes below the XML summary.
Add to the specification that there must be a validation on FlexOrder that an OptionReference is specified is there are >1 options in the FlexOffer.

Also adjust section on FlexOffer: " (only if there are multiple OfferOptions)" is incorrect. In the XSD FlexOfferOptionType.OptionReference is actually required. And this makes sense.

Fixed in "doc_update" branch.

Fixed in PR !87


Change the logo

Change the picture for the BEATS Facebook page.

Duplicate issue https://github.com/goodbotai/karma/issues/54


default logdirectory

if no log directory is set in the configuration section the log would be written to a log directory which does not exist!
-> create the "logs" directory!

logs directory is part of git repository so there is no need to create it at if no directory is set in config file. if the specified log directory does not exit code will raise an configuration error


fix: Fix conflicting nodeclaim.garbagecollcation controller name

Fixes #N/A
Description
Fix conflicting nodeclaim.garbagecollection controller name to now be instance.garbagecollection
How was this change tested?
make presubmit
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.025%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/controllers/nodeclaim/garbagecollection/controller.go
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
5512

Relevant Lines:
6639

💛  - Coveralls


New layout

The layout update for all parts of the app.

Reviewed and tested!

Reviewed and tested!


Allow Cool or Any as the $what for colored

So that I can use colored(.hour, "red") given DateTime.now instead of having to colored(.hour.Str, "red") given DateTime.now

well, this particular example is actually bad since what I actually really wanted was .fmt("%02d") in order to have it aligned properly. But I think the point still stands ...


move "CoinBlocker Domains List" from "Blu Go Pack" to "Blu Pack"

Energized Protection - block ⚡

Let's make an annoyance free, better open internet, altogether!

Issue Submit Form

  Provide the following infos properly, which will help us to resolve your issue quickly.

Issue(s):
Type x in between [ ] and make sure there isn't any space between brackets. Example; for Your Selected Issue(s), type like this - [x]
You can select more than one category of issues if you need to!

[x] Whitelist
[ ] Blacklist
[ ] App Broken
[x] Website Broken
[x] Request
[x] Other Issue

Pack(s):
Write the name of the pack(s) you are using.

Pack(s) Name: https://github.com/EnergizedProtection/block#packs-2

Extension(s):
Also name the extension(s) if you are using any. If you don't then leave it blank.

Extension(s) Name: https://github.com/gorhill/uBlock/releases

Domain(s):
If you are submitting this issue for whitelist/blacklist issue, send us the domain(s) for whitelisting/blacklisting here. Kindly use the Code Tag to prevent tracking.

Domain(s):

Your Config:
Just to ensure there is no issues or conflicts with other app/software/magisk module/extension/source list.
Make sure you are running Energized Protection Service only.

Client:  ublock

Version: spark, i want to use blu go

Details:
Write us a lil bit more about your issue or query. You can attach any screen shot or log of the issue or advert, this will help to highlight it.

Your Issue Detail:

CoinBlocker list he has an aggressive policy, and some domains are blocked/broken by my exstension if i use Blu Go list

Thank you for making Energized Protection great with your kind help!

A project by Ador with ❤

I don't think we can do it for now. But, you can move to Blu Go, and report us for the domain whitelisting.
And, we also have a strong list of whitelisted domains here - https://github.com/EnergizedProtection/unblock which may help to keep the legit sites whitelisted.

I don't think we can do it for now

can be replaced at nocoin https://github.com/hoshsadiq/adblock-nocoin-list

But, you can move to Blu Go, and report us for the domain

it's uncomfortable for me


[feature request]: Instagram Videos Downloader

Is there an existing issue for this?

[X] I have searched the existing issues.

Describe the feature.
Instagram Videos Downloader is an upcoming feature to be added to this repository
Problem/Motivation.
We want to have media like photos and videos for offline access many a times.
Possible Solution/Pitch.
python's "instalooter" or "instaRadier" modules can be useful
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

These are few totally random contents from instagram


log as LaTeX messages

@rultor release, tag is 0.0.4

@rultor release, tag is 0.0.4

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.4

@yegor256 Done! FYI, the full log is here (took me 1min)


Any plans to support Go plugins with delve ?

I know that delve does not currently support plugins.
Are there any plans to support plugins or any technical challenges that make this difficult to implement?

Duplicate of #865

Also you've opened #1099 which got closed as duplicate as well.
Clearly this is important for you, so if you want, why don't you try and contribute the issue yourself to delve? Spamming the developers like this will not help move the issue any further.

Logically support for PIEs would have to be implemented first, I don't foresee any substantial technical challenges but you never know.
Closing as duplicate.


Issue with scrollview when scrollview is at bottom of content.

using this type of layout:-
      <View style={{flex: 1}}>
        <MasterNav/> <--- that's a scrollview with login boxes
        <KeyboardSpacer/>
      </View>

When the keyboard is dismissed, if the scrollview is at the very bottom, the scrollview resizes, but doesn't scroll back - the bottom of it's content is where the top of the keyboard is, as soon as you drag it a little bit, it corrects itself.
I presume scrollview has a special case for being at the bottom maybe, if so can it be triggered to do a refresh?
After dismissing:

+1
Scroll all the way to the bottom and then dismiss the keyboard. The listview does not push the content back to the bottom.
(https://cloud.githubusercontent.com/assets/11079457/20124169/51868018-a5d9-11e6-86e3-0646683d3890.png)


[Schema Registry] Content Type Tweaks

Removing the handwritten ContentType class and moving the values into SchemaFormat, since ContentType is not being generated right now. Using Azure.Core.ContentType instead, since that is what the generated client is expecting.
Also changing the default content type from custom to the value passed in.

/azp run net - schemaregistry - tests

/azp run net - schemaregistry - tests

/azp run net - schemaregistry - tests

I'm actually going to do something different - will open a different PR


Return type based on content negotiation

Considering MVC & Web API are now merged and can be served from the same controller, I was wondering if there is a way to return an appropriate type for the requested content type?
For example, consider a POST request that was successful:
From a form/MVC we may return RedirectToAction("Index")
From a REST call with JSON, return new ObjectResult(newObject);
Is there a return type that either the route or object could be returned as appropriate?

@damienpontifex MVC 6 provides content negotiation. What in particular are you looking for?

Say a simple post method you would have in your controller like this to respond to a form submission on a page:
[HttpPost]
public IActionResult Post(MyModel model)
{
    if (ModelState.IsValid)
    {
        // Save model as appropriate
        return RedirectToAction("Index");
    }

    return View(model);
}

Vs I have been writing this for an API endpoint for a JSON result
[HttpPost]
public IActionResult Post(MyModel model)
{
    if (ModelState.IsValid)
    {
        // Save model as appropriate
        return CreatedAtRoute("Details", new { id = model.Id }, model);
    }

    return HttpBadRequest(ModelState);
}

Can I merge these methods (only have one POST method) and still get the same functionality as their only difference is the return objects? i.e. first returns HTML, second returns JSON.

@damienpontifex so you are looking to do action selection based on the request's Accept header?
I think you're looking for the [Consumes] attribute: https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc.Core/ConsumesAttribute.cs#L19
And here's an example: https://github.com/aspnet/Mvc/blob/eef6c3883a7e27b8387b0925f0b6a88df0a484c5/test/WebSites/ActionConstraintsWebSite/Controllers/ConsumesAttribute_NoFallBackActionController.cs#L9-L22

@Eilon actually the exact opposite really. I want to merge the methods, but not have to do a check to determine whether to return JSON or HTML.

Ah I see. I don't believe there's any built-in feature of MVC for this. You could write an ActionExecutedFilter to sniff the header and convert the result. Do you want to give that a try?

Haven't done one in the past, but very happy to give it a try. Do you have a link on the documentation/source for implementing one?

I think you could create a ViewOrJsonActionResult and return that. The action result would decide based on the accept header whether to render the view or render JSON.


Octoprint container creating a new entry when running, showing as "not managed by Dockge"

⚠ Please verify that this bug has NOT been reported before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
I moved my two octoprint stacks into the stacks folder and run them via the web interface, however upon start they create a new entry in the sidebar which is faded out and shows "This stack is not managed by Dockge." The original entry stays on "inactive"
👟 Reproduction steps

Move existing octoprint stack to the stacks folder (maybe also works with new stack)
Run via web interface

👀 Expected behavior
Only one entry on sidebar which shows the stack as "running" and lets me stop, update and restart.
😓 Actual Behavior
"Inactive" entry stays, another entry shows up which states the container is not managed by Dockge.
Dockge Version
1.1.1
💻 Operating System and Arch
Proxmox VE 8.1.3 x86_64
🌐 Browser
Safari (MacOS)
🐋 Docker Version
No response
🟩 NodeJS Version
20.10.24+dfsg1
📝 Relevant log output
"This stack is not managed by Dockge."

I did move some of my stacks before, I haven't seen this. The faded out item usually means that the compose file is not stacks directory
You can check if the stack is running under the "stacks" directory: docker compose ls

I found the problem. Stack-folder name was "octoMk3", had to change it to "octomk3" .. took me a while to remember that only small letters are allowed.


[Question/Feedback]: Stop deploying email action if the parameter is empty

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Description
Hi!
If I omit ALZMonitorActionGroupEmail within policyAssignmentParametersNotificationAssets in alzArm.param.json (because I don't want notifications by email), then an email notification to<EMAIL_ADDRESS>is still configured in the action group.
Is it possible to stop this from happening?
Thanks!

hey @Brunoga-MS.
I only want to use ALZWebhookServiceUri.
Thanks!

Perfect. I will investigate.

Perfect. I will investigate.

Thanks!

Hello @jaywithersbrewin ,
as a quick update, we are working on a solution to allow empty email address parameter so it will not deploy any email action with bogus addresses when not needed.
As soon as the code is ready, the merging of the corresponding PR will close this issue and you will get a notification.
Thanks,
Bruno.

Hello @jaywithersbrewin , as a quick update, we are working on a solution to allow empty email address parameter so it will not deploy any email action with bogus addresses when not needed.
As soon as the code is ready, the merging of the corresponding PR will close this issue and you will get a notification.
Thanks, Bruno.

Hey, @Brunoga-MS. thanks for the update!


Root path in child component router

When creating additional routing inside of one of components I am not able to set routing for "/" as this causes an error (see sub1 in http://plnkr.co/edit/PrVbeBt32LQEpQurBIYx?p=preview).
What I want to achieve is to load into "sub" viewport content of sub1 when main component is loaded. I've also tried redirect inside component approach but that didn't work either (see http://plnkr.co/edit/9lgBMg4xPb4UB6Jz4DQX?p=preview)

I am getting the same issue...since there is no example on this, I assumed it should be coded like @lukaszbros first example.
What are we missing in our code to make this work properly?

One thing...it looks like it is calling activate function twice and on the second time, it looses the routeConfig parameters, so it fails when it tries to run serializeInstruction and assign it the path variable.
Not sure if that helps with the issue any...

+1

+1

Seeing the same thing. At this point it looks like /dist/router.es5.js isn't what we should be using since it uses ng-viewport while /src/router-directive.es5.js uses ng-outlet (and the samples do too). The /src file has been updated very recently but it looks like it's been nearly a month since /dist has been updated.  Not sure if I'm just missing something there....a bit confusing.

+1
Semantically / should be considered default state at any level in the component hierarchy.

+1

+1

Please stop +1'ing issues on GitHub. Rather leave some constructive feedback that actually helps. Thanks everyone :)


feat(layout): auth layouts

Please read and mark the following check list before creating a pull request:

[x] I read and followed the CONTRIBUTING.md guide.

Short description of what this resolves:

Implements Sign In layouts (3x)
Implements Sign Up layouts (3x)
Implements Forgot Password layout
Adopts TopNavigation in AppBar

I'm not sure, but Forms seems to be stateless.
In some cases it's good to create method for the same actions: (value: boolean) => 'success' | 'danger';
Avoid useless methods such as: private isValue = (): boolean => this.state.value;


gha: cleanup stale AWS-CNI iptables rules in EKS tunnel workflow

Extend the eks-tunnel script to additionally flush the stale AWS-CNI iptables rules left by the aws-node DaemonSet, which break host to pod connectivity. Using the cleanup commands from the documentation 1.
For https://github.com/cilium/cilium-cli/pull/2035.

Link to successful run: https://github.com/cilium/cilium-cli/actions/runs/6529848997/job/17728137356?pr=2045
Dropping the temporary commit.

This should be done by this post-start hook: https://github.com/cilium/cilium/blob/main/install/kubernetes/cilium/files/agent/poststart-eni.bash
Should we update this script to also remove AWS-CONNMARK-CHAIN rules?

This should be done by this post-start cilium Daemonset hook: https://github.com/cilium/cilium/blob/main/install/kubernetes/cilium/files/agent/poststart-eni.bash
Should we update this script to also remove AWS-CONNMARK-CHAIN rules?
@giorio94 ^^

I don't know the details of the poststart-eni script, but makes definitely sense to me (it is very easy to miss the documentation and hit this issue). I edited this workflow because from a quick look at https://github.com/cilium/cilium/issues/25804 it seemed that the script was not always executed.
Even if we extend the poststart-eni.bash script, I guess we'll still need this change for the moment, until a new release is cut.

it seemed that the script was not always executed.

Just double-checked, and it is not run when running on EKS in tunnel mode, because eni.mode is not set.
/cc @nebril

AKS failure is unrelated, marking ready to merge

This seems like it's fixing CI, but isn't the same problem true in production systems?

This seems like it's fixing CI, but isn't the same problem true in production systems?

Unfortunately yes. More info in https://github.com/cilium/cilium/issues/25804


[Bug] - Undo of Transfer Shipment

When we do undo of a transfer of a item with variante code in a diferent unit of measure of the Base unit.
The system gives this error:
Invoiced Quantity must have the same value as Quantity in Item Journal Line Journal Template Name='',Journal Batch Name='',Line No.='0'.
AL call stack:
"Item Jnl.-Check Line"(CodeUnit 21).RunCheck line 86 - Base Application by Microsoft
"Item Jnl.-Post Line"(CodeUnit 22).Code line 15 - Base Application by Microsoft
"Item Jnl.-Post Line"(CodeUnit 22).PostSplitJnlLine line 12 - Base Application by Microsoft
"Item Jnl.-Post Line"(CodeUnit 22).RunWithCheck line 16 - Base Application by Microsoft
"Item Jnl.-Post Line"(CodeUnit 22).OnRun(Trigger) line 3 - Base Application by Microsoft
"Undo Posting Management"(CodeUnit 5817).PostItemJnlLine line 16 - Base Application by Microsoft
"Undo Posting Management"(CodeUnit 5817).PostItemJnlLineAppliedToList line 44 - Base Application by Microsoft
"Undo Transfer Shipment"(CodeUnit 9030).PostItemJnlLine line 62 - Base Application by Microsoft
"Undo Transfer Shipment"(CodeUnit 9030).Code line 56 - Base Application by Microsoft
"Undo Transfer Shipment"(CodeUnit 9030).OnRun(Trigger) line 15 - Base Application by Microsoft
"Posted Transfer Shpt. Subform"(Page 5744).UndoShipmentPosting line 6 - Base Application by Microsoft
"Posted Transfer Shpt. Subform"(Page 5744)."UndoShipment - OnAction"(Trigger) line 2 - Base Application by Microsoft
"Transfer Order Subform"(Page 5741)."Quantity Shipped - OnDrillDown"(Trigger) line 9 - Base Application by Microsoft

Great catch! And thank you for reporting the issue. Unfortunately, this repository isn't the right place to report your bug.
In this repository you can create issues concerning the extensibility of Business Central, such as missing events. You can also create issues you would like to fix yourself in the code present in this repository, which are our 1st party apps.
 Do you want me to transfer this issue to the BaseApp pilot repository?
Note, that the transferred issue would have to get approved in triage and you would have to create a pull request yourself. GitHub is not the right place to report issues you would like Microsoft to address. For that, please create a support request.
PS: You need to be granted access to the pilot repo, if you would like to collaborate on the BaseApp with us. To get access, please fill out the following form: https://forms.office.com/r/E9wyE5a58f.
PPS: I recognize that the process on where you can report which issues is a bit messy these days. We're in between processes and hope to be able to streamline and document the right processes in the very near future.


Bower install -> ECONFLICT Unable to find suitable version for polymer

Hello!
I am trying to add Polymer/paper-elements to my project using bower, but after downloading everything it fails saying:

ECONFLICT Unable to find suitable version for polymer

My bower.json looks like:
{
"name": "2StepAuth-web",
"description": "2StepAuth-web",
"version": "0.0.0",
"private": true,
"dependencies": {
"polymer": "Polymer/polymer#^1.0.0",
"jquery": "",
"bootstrap": "",
"socket.io-client": "*"
}
}
Now if I run:
 bower install --save Polymer/paper-elements
or
 bower install --save Polymer/core-icons
like described in https://www.polymer-project.org/0.5/docs/start/usingelements.html
I have this:

What I tried so far:

Update bower (1.4.1)
Clear bower cache (bower cache clean)
Delete everything from /bower_components and reinstall

Please let me know if I missed something or if there is anyway I can help!
And sorry in advance if I posted this issue on the wrong section.
Thank you!
Frank

In a clean directory, try:
bower init  << just press enter for each question
bower install --save Polymer/polymer
bower install --save PolymerElements/iron-elements
bower install --save PolymerElements/paper-elements
and you'll be ready to go.

Hi Mark!
Thank you! You just pointed out the problem!
Acutally I was trying:
bower install --save Polymer/paper-elements
where the correct is 'PolymerElements'
bower install --save PolymerElements/paper-elements
Everything works fine now.
Cheers!
Frank

Hi Frank,
The Google team appear to have moved all of the Elements to the PolymerElements git hub
The way to check is to look in the GitHub repository :
https://github.com/PolymerElements
Cheers,
Mark.

Hi Mark,
Oh, I see, this is also helpful!
Thank you again!
Cheers,
Frank


⚠ Samsung has degraded performance

In cab3e3c, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1908 ms

Resolved: Samsung performance has improved in eac6bfd.


Add Lora text to prompt info

Issue:
The text that is saved to the layer name has the lora removed
Fix:
Walk the API request and re-add the lora to the positive prompt
Notes:
I don't know if there is a way to get the original prompt strings for the request but that would be ideal.

need to remove the .safetensors extension

I don't know if there is a way to get the original prompt strings for the request but that would be ideal.

You can access the full prompt (including the lora) directly from the stored configuration. I.e:
self.cfg("txt2img_prompt", str)
It's never removed there. Maybe using that is easier that searching from the lora nodes in history? You'd have to figure out which mode (txt2img, img2img, inpaint...) was used though.

ok thanks I will look at that and see if it works.


Can the requirement of package.json and/or database.json be avoided?

I'm submitting a...

[ ] Bug report
[ ] Feature request
[x] Question

The app I'm working on is packaged into a single executable file using zeit/pkg . However, when including db-migrate in the app and using it via its API, I'm running into several issues. Some of them have simple workarounds (like including node_modules/db-migrate/lib/commands/** to the scripts array of pkg's configuration (or else it complains about some missing files at runtime))
My main problem is the requirement of the package.json file. As it is I understand it's required by db-migrate to check which plugins are installed. I don't include the package.json of my project with the app I'm distributing since it's a single exe file. Same goes with database.json actually. What I'm doing as a work around is to create those ones (a fake package.json and a real database.json) at runtime. This is not ideal, but it works.
Could db-migrate's API accept a configuration object for the database (instead of looking through the database.json file) ? Same goes for package.json for plugins.
Thanks in advance :)

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

You're asking for programmatic use. For the database.json, no changes are needed. This is defacto already possible in programmatic use. The docs need some love on that part (PRs welcome are welcome <3), but some is documented: https://db-migrate.readthedocs.io/en/latest/API/programable/#programable-api
But for your part: https://github.com/db-migrate/node-db-migrate/blob/b03c3f02e955a232681b7b99f23913ddb39ee459/index.js#L49. See this part, you can pass an options parameter and if you pass in here a config object, this will be used directly, for this see here: https://github.com/db-migrate/node-db-migrate/blob/b03c3f02e955a232681b7b99f23913ddb39ee459/api.js#L51
For the package.json, this is also already possible. you can pass the options param noPlugins see here https://github.com/db-migrate/node-db-migrate/blob/b03c3f02e955a232681b7b99f23913ddb39ee459/index.js#L56
And combine this with the options param plugins (you now need to gather the plugins completely by yourself) to pass in plugins by your logic: https://github.com/db-migrate/node-db-migrate/blob/b03c3f02e955a232681b7b99f23913ddb39ee459/index.js#L61

Closing since everything you asked for is already there. Feel free to reach out again shouldn't this solve your request.

Thanks for your help ! Indeed the docs could need some love, because I've read them and was pretty sure I didn't see anything about that :)


Does OVN have an LTS version

openvswitch has a clear LTS version.
openvswitch releases

https://www.ovn.org/en/releases
https://www.ovn.org/en/releases/release_22.03.0/


No output without -o

REPRO
get https://finnventions.com
Outputs nothing
WHAT SHOULD IT DO
Output the HTML text of the page to stdout.

This seems to only be a problem on the review branch and might already be fixed.


Move iptables check out of runtime init() to separate function

Due to the iptables package being init()ed at start of the docker
runtime, this means the iptables --wait ... command listing all rules
is run, no matter if the command is simply "docker -h".  It makes
more sense to both locate the iptables command lookup and check for the
wait flag support at the time iptables is actually used, as it
may not be used at all if certain network support is off/configured
differently.
Additional details:
The reason this is of interest is that on a significantly large box (many hundreds of CPU threads) and thousands of rules (e.g. few thousand docker containers up), docker -h can take significantly longer than on a standard SMP x86 box with only a few docker containers:
*** POWER8 (640 threads) w/thousands of containers

# time docker -h

real 1.648s

*** x86 VM w/10-15 containers

# time docker -h

real 0m0.027s

While a kernel issue revealed by this problem related to per-CPU counters will be fixed upstream, that may take some time to exist in distros that end users run.
Docker-DCO-1.1-Signed-off-by: Phil Estes<EMAIL_ADDRESS>(github: estesp)

can you give the time, with the patch too :) would be interesting to see the difference

@jfrazelle I will definitely try and get that.. I'm not the guy with the huge POWER8 box, but I can get him a binary, I think :)  I left out all his perf. data that showed where iptables --wait was hanging up on the per_cpu counter loop, so this really is the time-sink in this particular case; not to mention docker -h doesn't do much more than init a bunch of packages and then dump the usage string..

oh ok no worries then :)
On Tue, Jan 20, 2015 at 4:18 PM, Phil Estes<EMAIL_ADDRESS>wrote:

@jfrazelle https://github.com/jfrazelle I will definitely try and get
that.. I'm not the guy with the huge POWER8 box, but I can get him a
binary, I think :) I left out all his perf. data that showed where iptables
--wait was hanging up on the per_cpu counter loop, so this really is the
time-sink in this particular case; not to mention docker -h doesn't do
much more than init a bunch of packages and then dump the usage string..
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/10231#issuecomment-70761804.

LGTM

LGTM


Epaslog

Type: Feature Request
Epslg
VS Code version: Code 1.77.0 (7f329fe6c66b0f86ae1574c2911b681ad5a45d63, 2023-03-29T10:02:16.981Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: Yes

Closing as not enough information is provided


Creating a fileshare status is error, but looking at the database doesn't have such a fileshare

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug
/kind feature

What happened:
I created a fileshare and the staus is error. but looking at the database doesn't hvae such a fileshare.

What you expected to happen:
When the status of the fileshare created is error or the creation failed, the API needs to return the specific reason
How to reproduce it (as minimally and precisely as possible):
Create as many fileshare as possible, and it is easy to have error fileshare
Anything else we need to know?:
Environment:

Hotpot(release/branch) version:development branch
OS (e.g. from /etc/os-release): Ubuntu 16.04.2 LTS
Kernel (e.g. uname -a): Linux ubuntu 4.4.0-62-generic
Install tools:manually
Others: dashboard: development branch

as per @PravinRanjan10 , the db shows the error entry also currently.
(actually, this should be removed in case of error!;)
@jimccfun , please check this. Then close the issue!


Add a way to flow Razor editor parsing errors to the IDE

Follow up to https://github.com/aspnet/Razor/pull/2192#discussion_r175602613
This is useful for cases like when we want to report an error if we can't read a view import.

You'll see this error at build time now because of precompilation.

We're closing this as the compilation will be failed anyway.


test: improves e2e tests performance by reading the block metadata instead of the entire block data

Closes #3667
I also used this PR to make a change to E2ESimple to address a flaky behavior. The sequence count of the txclient has been increased to 10 to ensure there are sufficient transactions at the end of the test.

I also ran E2ESimple and can confirm it works fine with this new change.


[feature]: require Channel Acceptor to be registered before accepting channels

Channel Acceptor is a great tool to control what type of channels will be accepted. However, if no Channel Acceptor is registered, doesn't any channel get accepted? Seems to me like we should have a config flag on lnd to require at least one Channel Acceptor to be registered to the RPC before any channel will be accepted. Otherwise, you may have a situation on startup where a channel is accepted before the Channel Acceptor has time to register itself. Alternatively, if you have a bug in your Channel Acceptor and it crashes, you would want no channels to be accepted until that is fixed.

Just a note that for zero-conf channels they will be auto-rejected unless an acceptor is present


Add Flintrock commands for remotely executing shell commands on cluster nodes

Users launching clusters often need to do their own additional setup to install packages they need or the like.
We should offer one, maybe two new Flintrock commands to make it easy for users to run commands remotely on all the nodes of their cluster. I'm imagining they will work like stripped-down, Flintrock-specific versions of pssh.
Here's a proposal for what those commands might look like:
./flintrock run-command my-cluster 'yum install -y package'
./flintrock run-script my-cluster /path/to/local/script.sh

For starters I want to keep these commands as simple as possible. Just run the shell command or script on each of the cluster nodes in parallel and report whether they all passed or not. In the event of a failure, I might not even want to show which specific node or nodes failed, at least for an initial cut of this feature.
Anything more, like getting the output back from each of the nodes, will be pushed off to later, or redirected to utility tools like pssh which are better suited to the task.

I'm closing this issue since we have run-command (#26) implemented. I'm going to punt on implementing run-script since equivalent functionality can be composed by the user using copy-files (#7) and run-command.


Dev

range iter method beginning value initialization bug fixed.

Codecov Report

Merging #8 (fa791ce) into master (c0c8a25) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master       #8   +/-   ##
=======================================
  Coverage   99.20%   99.20%
=======================================
  Files           3        3
  Lines         252      252
=======================================
  Hits          250      250
  Misses          2        2

Flag
Coverage Δ

unittests
99.20% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

crontools/__about__.py
100.00% <100.00%> (ø)

crontools/crontab.py
99.16% <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 c0c8a25...fa791ce. Read the comment docs.


Properties file + GameClock settings

This fixes #12 and fixes #57

These issues have now been fixed


Performance issue deserializing elements with a large number of relationships

This is due to the check for duplicate relationships in the JacksonDeSerializer.addRelationship(...) method.
The current implementation compares the properties for all existing relationships which is a quadratic algorithm.

A separate hashmap can be maintained to check for duplicate relationships.

Resolved with PR #14


The Division for Corsair Straife

I install the Logitech patch, when I get into the Division the option to enable LED support is still grayed out.

Make sure you have Visual C++ Redistributable Packages for Visual Studio 2015 installed. If not, then run both Aurora and Division as admin.

Tried both of those and even reinstalled the game and it will not work still.

I'll test it tomorrow.

I remember people had issues with other Logitech games, and said that installing LGS and then patching fixed it for them.

Installed LGS and installed patch again and ran aurora and division as admin I get the option to enable the keyboard support, but when I enable it and get in game my keyboard and mouse do not light up at all.

Good. We are both on the same page. I just launched the division and experienced the same behavior (entire kb black), as well as a pile of errors.
I will be fixing that up now. Thank you for your report.


add brkraw

The ‘BrkRaw’ is a python module designed to provide a comprehensive tool to access raw data acquired from Bruker Biospin preclinical MRI scanner. This module is also compatible with the zip compressed data to enable use of the archived data directly.
The module is comprised of four components, including graphical user interface (GUI), command-line tools, high-level and low-level python APIs.
For the GUI, we focused on improving convenience for checking metadata and previewing the reconstructed image.
For the command-line tool, we focused on providing tools for converting, organizing, archiving, and managing data. The command-line tool also provides easy-to-use function to convert large set of raw data into organized structure according to [BIDS](https://bids.neuroimaging.io/).
For the high-level python API, we focused on enhancing the accessibility of reconstructed image data with preserved image orientation and metadata for the image analysis. It compatible users' convenient objects type ([nibabel](https://nipy.org/nibabel/) or [SimpleITK](https://simpleitk.readthedocs.io/en/master/gettingStarted.html#python-binary-files)) without the conversion step.
For the low-level python API, we focused on providing a consistent method to access raw Bruker data including parameter and binary files with the python compatible datatype while keeping the sake of simplicity.

@Vbitz - could you try building this container using your new system?

Dear @timwahoo - brkraw is added now :) You need to run the update function in the application menu to see the entry in the application menu.
Let us know if it works well for you!

That's amazing, Thank you!

Hi @stebo85,
I tried to use brkraw on neurodesktop and was able to use it through singularity exec. Is there a way to add the library's CLI as well?
Thank you!

Dear @timwahoo,
You have have two options to access the CLI:

Click the update button in the Application menu and start the application from the Data Organisation tab -> brkraw -> brkraw 0.3.11. This will open the terminal with the CLI:

Alternatively, you can use the module system (there was a little bug there before, but this is fixed now):

Let us know if this works well for you :)

Hi stebo85,
Thank you for the update! Yes I was able to get it to work, but I ran into a problem with the brkraw when testing it with my datasets. I'm not sure why this error occurred.
I am a developer for this software, so I'd be happy to get this software working on neurodesktop. Is there anyway for me to debug the singularity container?

What do you mean with debug? Would it help to be able to install packages?
If yes you can make the singularity container writable: singularity shell --writable

I think the issue is the reading/opening the Bruker files through the singularity container. However, I'm not sure if that is the issue without confirming it because it works fine when installed on a local host machine.
I do not think any other packages need to be installed.

Ahh, when running through singularity did you make sure you can read the file in the first place? Did you bind mount the directory where the data is located?

Dear @timwahoo - are there any updates on this problem? Can you give us test dataset so we can also try to figure out why it's not working on our end?

Hi @stebo85,
Yes, I have tried it with different datasets and it only fails on datasets obtained from Bruker ParaVision 360. The problem is definitely in the brkraw side of things.
Here is a link to my datasets https://drive.google.com/drive/folders/1_3xzqR8CJD4VC1DIhy7lT_zyFYe2Pfyy?usp=sharing

Dear @timwahoo
Ok, that's great to know! So should we update to v0.3.11 and the problem is solved?

Should be fixed with #625. A new container should be built soon.
brkraw changed their pip package name in a recent version. I've tested locally and it returns v0.3.11

thank you!

Dear @timwahoo,
This is fixed now:

can you test on your side?

It works great, thank you for taking the time and effort!

great to hear and thank you for confirming!


document using "provider" as a type

Fixes #711 , apparently this was already possible

I don't think this is displaying on the docs
https://aquasecurity.github.io/tfsec/v1.26.3/guides/configuration/custom-checks/#what-does-a-check-file-look-like
I don't see this under requiredTypes.


DO NOT MERGE: Change a md file to test path filters

DO NOT MERGE
:scroll: Description

:bulb: Motivation and Context

:green_heart: How did you test it?
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] No new PII added or SDK only sends newly added PII if sendDefaultPii is enabled
[ ] I updated the docs if needed
[ ] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps

Works as expected, as it's only running Danger


Added Testimonials

Related Issue

#427

Proposed Changes

Added 2 testimonials with mobile responsive
fixed the padding issue in hacktoberfest btn

Checklist

[x] ✅ My code follows the code style of this project.

[ ] 📝 My change requires a change to the documentation.

[ ] 🎀 I have updated the documentation accordingly.

[x] 🌟 ed the repo

[ ] Bug fix (non-breaking change which fixes an issue)

[ ] New feature (non-breaking change which adds functionality)

[ ] Breaking change (fix or feature that would cause existing functionality to change)

Output Screenshots

It has conflicts, fetch the latest code


Fix last remaining links to "/" that do not respect base_url

Refs #1023
datasette % git grep '"/"' -- '*.html'
datasette/templates/error.html:        home
datasette/templates/patterns.html:        home /
datasette/templates/query.html:        home /

Also these:
datasette % git grep '"/' -- '*.html' ':(exclude)*/patterns.html'
datasette/templates/allow_debug.html:<form action="/-/allow-debug" method="get">
datasette/templates/base.html:        <form action="/-/logout" method="post">
datasette/templates/error.html:        home
datasette/templates/logout.html:<form action="/-/logout" method="post">
datasette/templates/messages_debug.html:<form action="/-/messages" method="post">
datasette/templates/query.html:        home /

It looks like there are also some generated redirect responses that don't take base_url into account:
datasette % git grep '"/' -- '*.py' ':(exclude)*test_*.py' ':(exclude)datasette/app.py'
datasette/_version.py:        for i in cfg.versionfile_source.split("/"):
datasette/utils/asgi.py:        path="/",
datasette/views/base.py:            should_redirect = "/{}-{}".format(name, expected)
datasette/views/base.py:                should_redirect += "/" + urllib.parse.quote_plus(kwargs["table"])
datasette/views/base.py:                should_redirect += "/" + kwargs["pk_path"]
datasette/views/special.py:            response = Response.redirect("/")
datasette/views/special.py:            return Response.redirect("/")
datasette/views/special.py:        response = Response.redirect("/")
datasette/views/special.py:        return Response.redirect("/")

That utils/asgi.py line is the default path for setting cookies. That should likely take base_url into account too:
https://github.com/simonw/datasette/blob/4f7c0ebd85ccd8c1853d7aa0147628f7c1b749cc/datasette/utils/asgi.py#L331-L342

Blocked on #904

I fixed the queries.html one. I'm not going to fix these two:
datasette/templates/error.html:        home
datasette/templates/patterns.html:        home /

Because the error.html one does not get passed a context (which makes sense since an error has occurred) and the pattern portfolio doesn't need to link to anywhere in particular.

Changed my mind, error.html needs access to urls in order to link to its CSS file. Passing it after all (it already got passed ds.config("base_url") so ds was available previously).

I'm going to leave the cookies code setting cookies to default to the "/" top level.


version api call for pro

 Version API for Pro

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 3 (33.33%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.007%) to 84.058%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

services/src/pro/server.rs
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

operators/src/processing/temporal_raster_aggregation/temporal_aggregation_operator.rs
1
88.61%

services/src/pro/server.rs
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.007%

Covered Lines:
23542

Relevant Lines:
28007

💛  - Coveralls


minor changes, mainly for readability

It makes sense, but lockupTime is used everywhere and changing it only in the FarmingPool does not make much sense. We should change it everywhere. I do not see the necessity, the lockupTime is in days. It is clear enough.

lockupTime appears it too many contracts. I think we should not change it.
I integrated the other suggestions in https://github.com/superpowerlabs/synr-seed/pull/59


Documentation with YMAL Excample

Hey could you add and Excample for usinig it with an YAML file ?

Hi! I added an example in the README: https://github.com/happytal/substitute-variables#substitute-variables-from-a-yaml-file. Does it help you?

yes sir it does ! thank you !


add new 'Compare to' section under ESC in docs

Proposed changes

Added new Compare to directory under ESC in docs, and page for ESC vs Vault

We should change the get started here at the bottom of the page. It is talking about IaC:


Update categorical_encoding to use woodwork typing

This add-on library should updated with the new function calls, parameters, and the accessor approach.
For example, the OneHotEncodng here: https://github.com/alteryx/categorical_encoding/blob/597fa91259a94f94e5e38804cfd751d96616b946/categorical_encoding/primitives/one_hot_enc.py

Did some work on this here:

https://github.com/alteryx/categorical_encoding/pull/25
Anyone else can pick it up.


Update CHANGELOG, bump version and fix monitor_stdout

CHANGELOG updated for 0.6.1 version, using new version
of decorating on decorating/__init__.py and as well
added the monitor_stdout to the __init__.py  for easily
access.

Codecov Report

Merging #37 into dev will increase coverage by 0.33%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev      #37      +/-   ##
==========================================
+ Coverage   51.35%   51.69%   +0.33%
==========================================
  Files          10       11       +1
  Lines         331      354      +23
==========================================
+ Hits          170      183      +13
- Misses        161      171      +10

Impacted Files
Coverage Δ

decorating/__init__.py
100% <100%> (ø)
:arrow_up:

decorating/monitor.py
54.54% <0%> (ø)

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 67e6d6e...95e904f. Read the comment docs.


Update: NLP - Categories Testing Framework, closing #115's suggestion

Please include answers to these questions as part of your pull request
In the GitHub webUI, use the Write tab to modify the Markdown text that is part of the pull request. For each question simply place an X inside the square brackets, [X], that represents your answer. Make sure there are no blanks inside the brackets, otherwise MarkDown doesn't render properly. Using the Preview tab while editing this form, you can see the formatted/rendered version of the message.

Q1: Confirm that you have the right to submit the code that is being contributed. Please consider the origin of your code and confirm you have the appropriate rights to make the submission subject to the Apache 2.0 license that applies to everything in this repository of custom steps. If so, follow the instructions for the Contributor Agreement (which is based on the industry-standard Developer Certificate of Origin (DCO)).

[X] Yes, I have the right to submit the contributed code on behalf of myself, my company, or any other owner of the code. I have also attached my signed copy of the DCO to this message.
[ ] No

Q2: Confirm that your contribution does not include any personally identifiable information (PII), for example, in any examples used in your README file.

[X] My contribution does NOT include PII data
[ ] My contribution includes PII data

Q3: Confirm your contribution does not include any encryption or other export-controlled technology.

[X] My contribution does NOT contain encryption or other export-controlled technology
[ ] My contribution includes encryption or other export-controlled technology

ContributorAgreement.txt

Hi @snlwih , addressing this while it's still fresh in mind.  Thanks for the suggestion.  I've changed the error message accordingly.

SAS Internal Due Diligence Review - COMPLETED on 12JAN2024