Terraform create new vm with 400 Parameter verification failed error

main.tf:
terraform {
  required_providers {
    proxmox = {
      source = "telmate/proxmox"
      version = "2.9.6"
    }
  }
}

provider "proxmox" {
  pm_api_url      = var.PROXMOX_API_ENDPOINT
  pm_password = var.PM_API_PASS
  pm_user = var.PM_API_USER
  pm_tls_insecure = true
  pm_log_enable = true
  pm_log_levels = {
    _default    = "debug"
    _capturelog = ""
  }
}

resource "proxmox_vm_qemu" "cloudinit-test" {
    name = "terraform-test-vm"
    desc = "A test for using terraform and cloudinit"

    # Node name has to be the same name as within the cluster
    # this might not include the FQDN
    target_node = var.PROXMOX_NODE
    clone = var.CLONE_TEMPLATE
    # Activate QEMU agent for this VM
    agent = 1

    os_type = "cloud-init"
    cores = 2
    sockets = 1
    vcpus = 0
    cpu = "host"
    memory = 2048
    scsihw = "lsi"

    # Setup the disk
    disk {
        size = "32G"
        type = "virtio"
        storage = "local-lvm"
        iothread = 1
    }

    # Setup the network interface and assign a vlan tag: 256
    network {
        model = "virtio"
        bridge = "vmbr0"
    }

    # Setup the ip address using cloud-init.
    # Keep in mind to use the CIDR notation for the ip.
    ipconfig0 = "ip=<IP_ADDRESS>/16,gw=<IP_ADDRESS>"
    sshkeys = <<EOF
    ${var.ssh_key}
    EOF
}

terraform.tfvars:
# PROXMOX
PROXMOX_API_ENDPOINT = "https://<IP_ADDRESS>:8006/api2/json"
PM_API_USER = "terraform-prov@pve"
PM_API_PASS = "password"
PROXMOX_NODE = "workstation2"
DEFAULT_BRIDGE = "vmbr0"
CLONE_TEMPLATE = "ubuntu-server-22-04-packer-template"
ssh_key = "ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAACAQCs/m+4qHt0Xlc6AFCxB3HM8TeIxxGuLviPdZgv2kGN2Wwb+NAP2r6OXon1Sz7AY5PS23CXptR+SVsbWyn5sZxnvTkWp7meaTKBEVF8tbhaD1W3x4w86k/gcxrB/nIJ1et693/7nwnn2E3uVZdLvjMOBiQ8yNdtpkQVaQQacwLCJn+IPd4EpS1QqqGgbGISteg02BxouldeaZBtbezeBrrpv9YEGdNraFdyGmc7Prkv5XjGiAjJuCV8eHeykvtreq3vAhZsxdaIDb2QLhYyXfc2AhCK0WwbMB9/fJjsm5mlODImnLl17k84ZwseKTxxiNGtZOEyMkhjEHlY+5ACXoG/4QV4a5oVElh2hzCAYN1iRxynJQIuGVzOx6krxKvZ3apweATJ4+WSECj7mQ3ckDLvuLvB7GrwEUzIBDPxSzElu8iWQdwqz9iJ4pdCibMLi/5cAQQpmcsu1dkF36iMtUolugiw0f8fVXUtSZ03y/8hBUt8c9lqoDa0k+amUFfij48LMZPb0NWEFE0VB5PEC4VSb9FBw0qrdHfScx9dZ0wLseCjjSVDYHe8JbtzvunCh1FTATeH1HdEgEI2taKquAvAVJZZQ0/rIdNVwr93GpICkics+SESOcrz/9VkA6pdHtN1T7dx1VhErbrwzR3tGpZZDrOEM5tSh+fz5kF33Ee3Rw== cardno:11 023 204"

variables.tf:
#Variables

variable "PROXMOX_NODE" {
  default     = "workstation2"
  description = "Node to use for VM creation in proxmox"
  type        = string
}

variable "PROXMOX_API_ENDPOINT" {
  description = "API endpoint for proxmox"
  type        = string
}

variable "PM_API_USER" {
  type        = string
}

variable "PM_API_PASS" {
  type        = string
}

variable "DEFAULT_BRIDGE" {
  description = "Bridge to use when creating VMs in proxmox"
  type        = string
}

variable "CLONE_TEMPLATE" {
  description = "Name of your template"
  type        = string
}

variable "ssh_key" {
  type = string
}

After I execute terraform apply, I can see that a new virtual machine is created on the proxmox web page, but the virtual machine is not bootup, and the ip information of this machine is not set correctly,  I see 400 in the terminal mistake.
Here is the specific log
https://fars.ee/XfFv

terraform version
Terraform v1.1.7
on darwin_arm64

provider registry.terraform.io/telmate/proxmox v2.9.6

Try to check status of task in proxmox. It may contain helpful information about source of problem.

Try to check status of task in proxmox. It may contain helpful information about source of problem.

I see in the proxmox task that the virtual machine has been successfully created, but the corresponding virtual machine cloud init is not properly configured

I try to reinstall the proxmox system and use different operating systems, and the issue of granting users maximum privileges remains, But the result is still the same, here is the latest debug log:
https://fars.ee/ZXB4

Sshkeys is wrong, or you put it inline with EOF, Or you use variabiles

Sshkeys is wrong, or you put it inline with EOF, Or you use variabiles

Thanks for your reply, looks like a problem with this ssh key.
This ssh key is generated by my yubikey. This seems to be a bug, Looks like not support yubikey's ssh key.
Also, thanks @yqlbu for your patience and help,  for now on everything worked fine!

I had the same error after adding an invalid option to 'vga' (I set it to 'default' - I wonder why this doesn't work, anyway, although not specifying it at all will set it to 'default', indeed).
Does it make sense to create an issue related to the error being too ambiguous? How would you normally go about debugging this? In my case I knew where to look because that was one of the latest parameters I had changed, but I wouldn't have had any ideas if I had taken over from someone else.
In /var/log/pveproxy I see only plain access requests and there's nothing relevant in the GUI. The error occurs after cloning the template (which finishes successfully),


Annotate answer choices by order displayed, even with randomization

All of the answer choices should be annotated based on what order they were displayed in. For example: "02-yes" for "yes" as the third answer choice. This should be true even if the answers are randomized.
Previously: the number was wrong if randomization
Now: number is correct even with randomization
Fixes #172

meacer ptal

Lgtm


revert to non-corrupted version of colors package

see https://www.bleepingcomputer.com/news/security/dev-corrupts-npm-libs-colors-and-faker-breaking-thousands-of-apps/
this currently makes all tests fail, see e.g. https://github.com/openstreetmap/id-tagging-schema/runs/4759030935?check_suite_focus=true#step:5:11

PS: I've already published this one on npm<EMAIL_ADDRESS>

Changed to chalk in https://github.com/ideditor/schema-builder/commit/d9c23fac2a3d1733e55c841c1e9e476ad78e2f7f


Generate where all of them implements Send + Sync?

So we can store and share this in an Arc and reuse and not need to connect all the time.

note: future is not `Send` as this value is used across an await
   --> packages/podman_cli/src/modules/podman/deploy/handle_pods.rs:34:57
    |
25  |   let api_pods = client.pods_api();
    |       -------- has type `&dyn podman_autogen_api::apis::pods_api::PodsApi` which is not `Send`

I solved it by using client.pods_api() every time I needed the sub APIs.

@backend  Ran into the same annoyance. I submitted a patch to the upstream openapi-generator project here: https://github.com/OpenAPITools/openapi-generator/pull/1920
But I can push out a new version before that's accepted. Making everything Send (and probably Sync too)

Actually, I tried pulling an image, creating a container. Both of the APIs failed.

@beckend
The latest version makes all those API structs Send + Sync.

Actually, I tried pulling an image, creating a container. Both of the APIs failed.

Which version of podman are you using? This crate was built around v5.
I just wrote some tests and some of them are currently failing on github because the github ubuntu runner is on v4.

5.1.0

@beckend
Linux or Mac? One thing that trips me up on a new linux system is remembering to enable the unix socket. Since podman doesn't have a server by default like docker. I added some documentation here  about that.
Are you using the guess_configuration function? One small breaking change is that it has been moved to be Config::guess now. The function now returns an error if it can't find a socket. Before it would silently let you go ahead and then error later when you tried to make a request.
Now that I realize people are using this already, I started a changelog and I'm actually tagging releases on github.

Please see: https://github.com/blazzy/podman-rest-client/issues/6


Dark mode foreground tab is indistinguishable when windows in in focus

Macos 12.7
Firefox 120
Thank you

This is not really a bug but rather a side-effect of the translucent tabbar tweak that only happens when your Firefox window is over a certain (bright) color background. If the underlying background (whether it be the wallpaper or another app) has a similar color as Firefox's default dark UI gray color, the tab will turn out to look awfully similar to the tab bar color making it hard to distinguish the two.
A workaround could be adding additional borders for tabs, but I opted not to to so because I only wanted to add translucency to the tab bar in this tweak, and refrain customizing other aspects of the default Firefox skin too much.

Actually, I just realized that the thin border I had added previously to alleviate this exact issue wasn't displaying at all at the moment. It must have broke at some point and I didn't see it. Sorry for the confusion, I'll push a fix now that brings back the border.


Error when updating token resource in 0.3.10

Describe the bug
In the new provider version 0.3.10 updating dbtcloud_service_token resource is failing.
Error message
│ Error: Provider produced inconsistent result after apply
│
│ When applying changes to
│ dbtcloud_service_token.tokens["token_name"],
│ provider "provider[\"registry.terraform.io/dbt-labs/dbtcloud\"]" produced
│ an unexpected new value: .service_token_permissions: planned set element
│ cty.ObjectVal(map[string]cty.Value{"all_projects":cty.False,
│ "permission_set":cty.StringVal("developer"),
│ "project_id":cty.NumberIntVal(3597),
│ "writable_environment_categories":cty.SetVal([]cty.Value{cty.StringVal("all")})})
│ does not correlate with any element in actual.
│
│ This is a bug in the provider, which should be reported in the provider's
│ own issue tracker.

Resource configuration
Service tokens are configured with the loop.
Affected token have around 50 project IDs assigned.
resource "dbtcloud_service_token" "tokens" {
  for_each = local.tokens

  name = each.key

  dynamic "service_token_permissions" {
    for_each = each.value
    content {
      permission_set = "developer"
      all_projects   = false
      project_id     = service_token_permissions.value
    }
  }

  dynamic "service_token_permissions" {
    for_each = each.value
    content {
      permission_set = "git_admin"
      all_projects   = false
      project_id     = service_token_permissions.value
    }
  }

  dynamic "service_token_permissions" {
    for_each = each.value
    content {
      permission_set = "job_admin"
      all_projects   = false
      project_id     = service_token_permissions.value
    }
  }
}

Expected behavior
Token updated in place.
Config (please complete the following information):

Installed dbt-labs/dbtcloud v0.3.10 (self-signed, key ID 644DCEA333782266)

Additional context
Downgrade to version 0.3.8 solves the problem and update is passing.

Thanks for the issue. We will try to reproduce it.
Could you please tell us what dbt Cloud tenant you are on? US MT, EMEA MT, ST (Single Tenant)?

We are using EMEA: https://emea.dbt.com/api

Thanks @kpucynski . Looking into this now. This appears to be a bug with a new feature that is conditionally enabled.

Do you still have an issue with the latest releases? It might have gone now as the env level permissions should be active for all accounts.

Closing for now. We can reopen if there is still an issue.

We have just tested code with new 0.3.22 provider version and get the same issue:
│ Error: Provider produced inconsistent result after apply │  │ When applying changes to │ dbtcloud_service_token.dcrt_tokens["token"], │ provider "provider[\"registry.terraform.io/dbt-labs/dbtcloud\"]" produced │ an unexpected new value: .service_token_permissions: planned set element │ cty.ObjectVal(map[string]cty.Value{"all_projects":cty.False, │ "permission_set":cty.StringVal("developer"), │ "project_id":cty.NumberIntVal(3190), │ "writable_environment_categories":cty.SetVal([]cty.Value{cty.StringVal("all")})}) │ does not correlate with any element in actual. │  │ This is a bug in the provider, which should be reported in the provider's │ own issue tracker.

0.3.8 still no issues.

@b-per can you reopen the issue?

I just reopened it but we haven't been able to reproduce it. We can try again.

I just retried your code with 10 projects on the latest provider
locals {
  tokens = {
    "issue280": [
        1234,
        2345,
        ... (10 in total)
    ]
  }
}

And I can't reproduce the issue.
The token gets created and I can remove/add projects without a problem.
Could you please share your dbt Cloud Account ID?

Good/Bad news. I created 40 projects on the fly and can now reproduce the issue when trying to assign permissions for 40 projects!

@chasewalden
The issue here is that when a lot of permissions are assigned to a token, we don't return all of them and actually need to loop through the paginated results. So, we need to add pagination in GetServiceTokenPermissions.
https://github.com/dbt-labs/terraform-provider-dbtcloud/blob/cc7fa2db09f374cccf5c2d2400de6c4e50c23373/pkg/dbt_cloud/service_token.go#L47-L65
this is the reply we get when we query a token with more than 100 permissions
...
            "account_id": 1234,
            "all_projects": false,
            "created_at": "2024-12-11 13:45:57.925949+00:00",
            "id": 532831,
            "permission_set": "job_admin",
            "project_id": 1111,
            "service_token_id": 22222,
            "state": 1,
            "updated_at": "2024-12-11 13:45:57.925963+00:00",
            "writable_environment_categories": []
        }
    ],
    "error_code": null,
    "extra": {
        "filters": {
            "account_id": 1234,
            "limit": 100,
            "offset": 0,
            "service_token_id": 22222,
            "state": "active"
        },
        "order_by": null,
        "pagination": {
            "count": 100,
            "total_count": 120
        }
    },
    "status": {
        "code": 200,
        "developer_message": "",
        "is_success": true,
        "user_message": "Success!"
    }
}

I checked the permissions for groups and it looks that endpoints are slightly different so we might not have the same problem there.
Can you or the team work on a fix here or do you want me to do it?

@b-per we have currently 52 projects on the problematic list, so one resource will create over 150 permissions.
I would like to not post any project related details here - ping me where(if) I can contact you with the details.

Hi @kpucynski . All good. We have all the details to reproduce it so we don't need more inputs for now. We just need to implement the fix.


New version: Memoize v0.4.1

Registering package: Memoize
Repository: https://github.com/JuliaCollections/Memoize.jl
Created by: @kmsquire
Version: v0.4.1
Commit: bd0b00d1c72c584b088c07ffaf89081b26746c89
Reviewed by: @kmsquire
Reference: https://github.com/JuliaCollections/Memoize.jl/commit/bd0b00d1c72c584b088c07ffaf89081b26746c89#commitcomment-37366562

Superceded by #9764 (this can be closed).


Compiler: error if defining initialize inside enum

Fixes #7238

That would fall under "don't suggest using it" 😬
For HTTP::Status it would be nice if .new could be overriden instead of having to resort to .code for adding a value verification to the initializer.

I just pushed a commit to allow redefining Enum.new. But to do it you have to put the correct type restriction, otherwise it will take the other new (for example if you don't put any type restriction it will choose the other one because it's more strict). And then you have to use previous_def to invoke the original new.

This isn't a breaking change since defining def initialize didn't work before, right?

@RX14 Exactly


There are the following problems with fine-tuning brain data

Namespace(data='/home/lcl/PCRLv2/dataset/BraTS_2018/MICCAI_BraTS_2018_Data_Training', model='pcrlv2', phase='finetune', b=4, epochs=100, lr=0.0001, output='./brats_finetune_weight', n='brats', d=3, workers=4, gpus='0,1', ratio=1.0, momentum=0.9, weight_decay='./pretrained_weight/simance_multi_crop_luna_pretask_1.0_240.pt', seed=42, amp=False)
pcrlv2_brats_finetune
Traceback (most recent call last):
File "/home/lcl/PCRLv2/main.py", line 46, in
data_loader = get_dataloader(args)
File "/home/lcl/PCRLv2/main.py", line 17, in get_dataloader
dataloader = getattr(generator, loader_name)()
AttributeError: 'DataGenerator' object has no attribute 'pcrlv2_brats_finetune'
Process finished with exit code 1

For brats fine-tuning, to achieve better results, we modified based on a good baseline.  I will send you the fine-tuning code through email. Please try fine-tuning based on that code.
Maybe you should first adjust the hyper-parameters to achieve comparable training from scratch results and then try our pretrained weight.

Hi @liaochuanlin,
Doing segmentation on BraTS is not easy considering there are so many details to deal with.
Please try matching the performance of the training from scratch baseline first.

Dear author, when the brain data were retrained, the size of the central clipping was (112, 128 ,112), while the original image size was (240,240,155). So how to visualize after training the model? is there any corresponding code for my reference?
爩
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "RL4M/PCRLv2"                                                                                @.>;
发送时间: 2023年9月27日(星期三) 下午3:15
@.>;
<EMAIL_ADDRESS>主题: Re: [RL4M/PCRLv2] There are the following problems with fine-tuning brain data (Issue #10)
Hi @liaochuanlin,
Doing segmentation on BraTS is not easy considering there are so many details to deal with.
Please try matching the performance of the training from scratch baseline first.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Dear author, just one more question. How is the class activation diagram implemented in that paper?
Can I share your code?
爩
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "RL4M/PCRLv2"                                                                                @.>;
发送时间: 2023年9月27日(星期三) 下午3:15
@.>;
<EMAIL_ADDRESS>主题: Re: [RL4M/PCRLv2] There are the following problems with fine-tuning brain data (Issue #10)
Hi @liaochuanlin,
Doing segmentation on BraTS is not easy considering there are so many details to deal with.
Please try matching the performance of the training from scratch baseline first.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


CrashLoopBackOff aks-periscope pods with logs "failed to create CRD".

There is a specific behaviour I have noticed since last 2 days, which is detailed below. This is incidentally already fixed as an effort to move to use client-go hence for ARC and Vanilla AKS scenario we recently merged the PR to move away from the kubectl direct command execution.
After deploying aks-periscope tool, pods will CrashLoopBackOff with error and the log will look like this.
Following error:
❯ kubectl logs aks-periscope-cxbjk -n aks-periscope
2021/12/26 00:54:42 Failed to create CRD: run command in container: exit status 1

Why this issue happening now & lesson to learn and move away from kubectl direct use

Not sure why this is happening since last 2 days, and I have no specific much answer to this but it could be that OS level kubectl binary and the direct call to kubectl is not working as it seems.

But, I really feel this is a good example for us to learn and move away from direct use of kubectl in this tool, since client-go gives us better usability.

I also feel that running kubectl binary was always a hack, which we have now transitioned all the collector minus OSM/SMI collectors to use client-go apis.

Fix \ Solution
This break seems like co-incidence but:

Prima facie it looks like it has to do with a CreateCRD function which was always a questionable part and it was already removed as part of this PR #136 as an effort to move away from kubectl binary.

Logging this in case anyone else see same issue.
Thanks, ❤🙏☕
Update 26 Dec 8:39 pm
Even though the culprit code is gone in newer version which I plan to release sometime soon, I added debugger and found this issue:
2021/12/26 07:37:38 Failed to create CRD: run command: /usr/local/bin/kubectl apply -f aks-periscope-diagnostic-crd.yaml in container: exit status 1.
so essentially this no longer works, and I am not sure possibly something within the cluster is improved to avoid this but I will keep an eye. The branch under my fork for this investigation is : bug/investigation-Create-CRD if anyone is keen.
Further logs: (This helps in explaining the issue possibly, now the part why it happened now is left.)
❯ kubectl logs aks-periscope-xxxxxxx -n aks-periscope
2021/12/26 21:35:36 Failed to create CRD: run command: /usr/local/bin/kubectl apply -f aks-periscope-diagnostic-crd.yaml in container: exit status 1 and outputStream is - std out: , err: error: error validating "aks-periscope-diagnostic-crd.yaml": error validating data: ValidationError(Diagnostic.spec): unknown field "networkconfig" in com.github.azure.aks-periscope.v1.Diagnostic.spec; if you choose to ignore these errors, turn validation off with --validate=false

🕵 and this should explain seems like validation error:
.spec.networkconfig: field not declared in schema

Temporary fix
Consuming tools should add this under their CRD schema:
networkconfig:
                type: string

Longer term fix
The troubled code which initiated all this is removed as part of clean up.


Download links generator & all drivers wd execlusions

Download links provider: When ran, it will ask you for file location, doesn't matter what the extension is ( till now ), then will provide you with file direct download link ( can be used by curl ).
All drivers WD exclusion: Adds all drivers ( A:\ - Z:\ ) to Windows Security scan exclusion, so it will technically not work at all but just says that it is working with the icon shown.

When adding something to /features it'd be best to make it into a command

When adding something to /features it'd be best to make it into a command

Aight imma make it a command rn,

Also it's a good practise to make separate prs for each addition since they are completely unrelated


Set disableReports=true in quarkus-maven-plugin config to speed up th…

…e itests execution

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/110/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/112/


Update quickstart-helm.md

Updated change to convert "MyHelmACR" to "myhelmacr"
As acr name is allowed only in lowercase, pfb the error
root@jumpbox:~# az acr create --resource-group MyResourceGroup --name MyHelmACR --sku Basic
argument error: Connected registry name must use only lowercase.
root@jumpbox:~# az acr create --resource-group MyResourceGroup --name myhelmacr --sku Basic

@zr-msft
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @raghushukla1 - thank you for your feedback! I'm working on doc updates in a separate PR and have incorporated your feedback into the changes. Since they're in my PR, I'm going to close this one. Thanks again!
#please-close


Preserve /build/{cleanup.sh,buildconfig} files.

/build/{cleanup.sh,buildconfig} files are useful for those who create Docker images based on this
baseimage, and need to make a final cleanup and/or use some
configurations that are inside buildconfig file.

@endersonmaia This PR has merge conflicts, please resolve those conflicts and let me know so I can merge your changes in! :)

@Theaxiom there's no related issue that I know of.

I am reverting this because it is breaking the build, see: https://travis-ci.org/phusion/baseimage-docker/builds/213323558#L6335
Here's some information regarding the error: http://stackoverflow.com/questions/26540813/got-exit-code-123-in-find-xargs-grep

@endersonmaia It seems that this might fix it:
ls -d -1 /bd_build/**/* | grep -v "cleanup.sh" | grep -v "buildconfig" | grep -v "services/" | xargs rm -f
I got that from https://github.com/phusion/baseimage-docker/commit/9fed983d8072c3b597a810f39580232879cb487a
Go ahead and try that change and then re-submit a new PR. Thanks!

I am currently testing this fix, if it works I will add it to next.

Okay that worked, I added it to next. Thank you!


Closure over array variables yields compiler error.

Consider this Mirah code:
class TestCallable

	def bar(param:byte[])
		runnable = lambda(Runnable) do
			puts param[0]
		end

		runnable.run
	end
end

When compiling this code, it is expected that the compilation succeeds. However, currently, the compilation fails with this error message:
ERROR: Cannot find class byte[]
1 errors

:(

:), Fixed.


PUD-438: Build the request metrics dashboards into the helm chart

This is a temporary commit to prove out the work I've been doing on the generic-service helm chart.
Basically it's a copy/paste of the generic-service chart and I've added in the following:

An optional traefik proxy in front of the main container (for "internal" services to get request metrics)
A golden signals dashboard using the metrics coming from Traefik
A golden signals dashboard using the metrics coming from the NGINX ingress

We don't really need two (almost identical) dashboards, they are both here for testing purposes. In the final cut we would only have one based on if the traefik proxy is needed or not (if we have an ingress, it's not).

Cheers :)
Nope, this is an addition to the generic-service helm chart (that both the kotlin and typescript template projects use), so basically all users of the templates and helm charts can get standardised dashboarding.


"wtmpdb last": Output inconsistent between individual invocations.

I observed this with the --present switch, but a similarly erratic behavior has been reported for --until at https://bugs.launchpad.net/ubuntu/+source/wtmpdb/+bug/2088387.
Here's what I get with wtmpdb last -p "2024-12-14 07:45:00"
Invocation 1:
tcrass   ssh          2003:dd:f0b:d000 Sat Dec 14 07:43 - 07:45  (00:01)
wfr      tty2         :0               Fri Dec 13 21:57 - still logged in
reboot   system boot  6.11.10-amd64    Fri Dec 13 20:02 - still running
tcrass   ssh          2003:dd:f0b:d000 Fri Dec 13 14:22 - crash
wfr      tty2         :0               Fri Dec 13 14:17 - crash
reboot   system boot  6.11.10-amd64    Fri Dec 13 14:16 - crash
wfr      tty2         :0               Wed Dec 11 09:57 - crash
reboot   system boot  6.11.10-amd64    Tue Dec 10 13:51 - crash
wfr      tty2         :0               Mon Dec  9 21:19 - crash
wfr      tty2         :0               Mon Dec  9 20:38 - crash
sddm     tty2         :0               Mon Dec  9 20:32 - crash
wfr      tty2         :0               Mon Dec  9 20:31 - crash
wfr      tty2         :0               Mon Dec  9 19:57 - crash
reboot   system boot  6.11.10-amd64    Mon Dec  9 19:55 - crash
wfr      tty2         :0               Mon Dec  9 18:09 - crash
sddm     tty2         :0               Mon Dec  9 17:46 - crash
tcrass   tty1                          Mon Dec  9 17:45 - crash
reboot   system boot  6.11.10-amd64    Mon Dec  9 17:43 - crash
wfr      tty1                          Mon Dec  9 16:33 - crash
wfr      tty1                          Mon Dec  9 16:28 - crash
reboot   system boot  6.11.10-amd64    Mon Dec  9 16:26 - crash
wfr      tty1                          Mon Dec  9 16:23 - crash

Invocation 2:
wfr      tty2         :0               Fri Dec 13 21:57 - still logged in
reboot   system boot  6.11.10-amd64    Fri Dec 13 20:02 - still running
tcrass   ssh          2003:dd:f0b:d000 Fri Dec 13 14:22 - crash
wfr      tty2         :0               Fri Dec 13 14:17 - crash
reboot   system boot  6.11.10-amd64    Fri Dec 13 14:16 - crash
wfr      tty2         :0               Wed Dec 11 09:57 - crash
reboot   system boot  6.11.10-amd64    Tue Dec 10 13:51 - crash
wfr      tty2         :0               Mon Dec  9 21:19 - crash
wfr      tty2         :0               Mon Dec  9 20:38 - crash
sddm     tty2         :0               Mon Dec  9 20:32 - crash
wfr      tty2         :0               Mon Dec  9 20:31 - crash
wfr      tty2         :0               Mon Dec  9 19:57 - crash
reboot   system boot  6.11.10-amd64    Mon Dec  9 19:55 - crash
wfr      tty2         :0               Mon Dec  9 18:09 - crash
sddm     tty2         :0               Mon Dec  9 17:46 - crash
tcrass   tty1                          Mon Dec  9 17:45 - crash
reboot   system boot  6.11.10-amd64    Mon Dec  9 17:43 - crash
wfr      tty1                          Mon Dec  9 16:33 - crash
wfr      tty1                          Mon Dec  9 16:28 - crash
reboot   system boot  6.11.10-amd64    Mon Dec  9 16:26 - crash
wfr      tty1                          Mon Dec  9 16:23 - crash

In the second invocation, my current login (tcrass through ssh) is not shown!
Apart from that, I was, of course, still logged in when issuing the command, so my status in invocation 1 should be "still logged in" rather than "session ended at 07:45", right?
Cheers -- tcrass

The unique key for wtmp entries /independent of plain wtmp or wtmpdb) is "account/tty".
Your openssh version uses always "ssh" as tty, so it is not possible to find the correct entry. The result is what you see.
You cannot use pam_wtmpdb.so for openssh, as openssh does not report the TTY to PAM. You need the wtmpdb patch for openssh:
commit 96b64056c812620014b65371a9e3ac86bfcd08d5

    Add wtmpdb support as Y2038 safe wtmp replacement

and disable pam_wtmpdb for openssh as openSUSE has done.

OK, thanks for the explanation -- I guess I'll just hope and wait for the patch to appear in Debian...
Cheers -- tcrass


Should the color of warning is light color?

I thought this is personal preference and you can change the color yourself.

I just want to confirm that it is intended. In 3.x, the color of warning is #fff,  https://getbootstrap.com/docs/3.3/components/#available-variations.

It's automated based on the contrast with the background.

Although I understand the contrast issue, this creates another issue: we lost consistency on text colors on buttons/badges.
Other related issue: the outline warning button is still white when hovered.


Add Proxy#association_list for adding lists of assocations in definitions

I like create_list and friends in recent FactoryGirl. We need association_list for definitions.
factory :user do
  observations { association_list :observation, 10 }
end

Perhaps this should also include
factory :user do
  assocation_list :observations, 10
end

I like this idea but I'd like to see some acceptance specs for it as well. Does this associate the records together? (e.g. if a user has_many posts, would association_list :post, 10 create ten posts specifically assigned to that user? That's where the real win would be. Let me know!

@sj26 I actually just added documentation for how to build has_many data. I'd love to see if we can get automatic association name recognition in here before merging.

Wow, master changed a bit!
I'm updating for these changes and writing an acceptance spec to make sure it works as intended.

Awesome, thanks!

Okay, I updated for master, but there are problems.
When the build strategy is create we should probably build associations then let the parent save take care of things. Should we infer the inverse association and populate it with the factory?
Also, the AttributesFor strategy probably needs to return nil instead of an array of nils, or prevent the attribute appearing at all a la assocition.
Not sure I understand FactoryGirl enough to do this yet, but I'll keep digging into it.

I think we should try to infer the association, yes. That's why this problem is so tricky and why we've typically recommended users create associated records like this in an after_create callback. association on attributes_for returns nil currently, but the result of association_list isn't (and shouldn't) be used, so I don't think that we'll need to compact or flatten the result.

I've been digging into this problem a little more. It seems great effort has been made to abstract the attributes and associations away from the defining factory. To set the inverse association on created records we need access to the parent factory's build class to reflect the association, obviously only for ActiveRecord factories.
Am I missing something? Is there an easy way to get access to the factory from the association and attribute classes? Or would there be disinclination to introducing a link back up the chain?

@sj26 yikes - yeah, there's currently no way to get from the association to the parent creating it, as you've seen. We're going to want to stay away from relying on ActiveRecord, so I wonder if we could explicitly pass in the reverse association to be assigned instead.

Yeah, that should degrade gracefully if it's not ActiveRecord, but I agree it's a bit icky.
Passing :inverse_of => :blah a la the association macros wouldn't be a crime, and it would be nice for explicit overrides, but it would still be nice to infer it if we can.

The complexity of this seemed to explode past what I imagined. I wonder if it'd be better to just simplify / combine the existing suggestion for creating associated data in the GETTING_STARTED doc by building an after_create callback instead. Then, we'd have access to the instance and we'd just need to define the inverse association.
Let me know if you're not interested in continuing; this is a really hairy issue and something we've struggled to solve for a while now and if it feels like you've bitten off a bit more than you can chew, just let me know!

I'm going to close this out since it turned out to be a lot more difficult than imagined. I'll try a shorter pass that adds ignores/callbacks to accomplish this sometime in 3.x


Modern JS without Core.js

Describe the bug
Using the experimental.modern = true in the Next.js v9.0.4-canary.4 or upper (https://github.com/zeit/next.js/pull/7961) next-i18next library is including the core.js shims to the bundle.
I guess that although is experimental yet, the bundle with modern JS should not include these shims.
Occurs in next-i18next version
1.1.0
Steps to reproduce
Under canary version, add on the next.config.js file this config:
experimental: {
   modern: true,
},

Expected behaviour
Should not include the shims for modern js.
Screenshots

I also see two copies of core-js in the final bundle, even though I don’t use modern flag yet. IMO polyfilling should be done via requiring an enhancer from
this package in  the app’snext.config.json rather than by increasing the size of the plugin’s build folder.

Hi @aralroca, you need to consume the dist/es target if you don't want core-js in your bundle.

@aralroca Any update?

I'm also having this issue, Multiple copies are bundled in production.
next-i18next: 4.4.1
next: 9.3.5
typescript: 3.7


[web]How javascript in iframe communicate with dart

    ui.platformViewRegistry.registerViewFactory(
      'subpage',
            (int viewId) => IFrameElement()
          ..src = 'https://<IP_ADDRESS>/pages/index.html'
          ..style.border = 'none');
    return HtmlElementView(viewType: 'subpage');

How javascript in index.html communicate with dart?

This question would be better posed as a question on Stackoverflow or one of the spots mentioned in flutter.dev/community
Since this is not a bug or feature request, closing.


[Bug]: The API request for contributors is not handled properly

What happened?
There is no proper error handling and the fetch method is used which doesn't provide the correct response every time.
I want to improve the API handling method and add error handling.
Please assign this issue to me. @priyankarpal
Thank you
Add screenshots
Add screenshots
What browsers are you seeing the problem on?
Chrome
Code of Conduct

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

What happened?
There is no proper error handling and the fetch method is used which doesn't provide the correct response every time.
I want to improve the API handling method and add error handling.
Please assign this issue to me. @priyankarpal
Thank you
Add screenshots
Add screenshots
What browsers are you seeing the problem on?
Chrome
Code of Conduct

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

yes, we need to add this feature. assigned to you


Fix user report domain name

This should fix an issue Dave and I found when testing out the user report page on staging. Because the domain is different where researchers run reports the previous code was using this report domain instead of the main portal domain.
This PR also includes a fix to the database schema which was breaking the tests. The new keywords field was missing from external_activities.  I'm not sure if this was because it was missed in the original keywords PR, or perhaps it was broken during the work on the new user report page code.

Looks like we have one legitimate failing test, I might get to this tonight.


[BUG] Audit OpenSearch clients to ensure Opaque-Id is not mis-used and not used as unique identifer

Describe the bug
Unique OpaqueId causing memory build up as it is being used in DeprecationLogger. It will help to audit clients to ensure it is used as intended to track unique clients and not unique requests or process.
Issue - https://github.com/opensearch-project/OpenSearch/issues/16702
Related component
Other
To Reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
..
Additional Details
..

https://github.com/search?q=org%3Aopensearch-project X-Opaque-Id&type=code


@alloy => Remove HardSource webpack plugin

Let's see if this improves speed (+ helps with some weird module caching issues).

Yep, looks good 👍


The Prefix Doesn't change

The prefix says saved sucessfully but the prefix doesn't change in the server

also.... i already have a website, how do i intergrate this with my site

This issue lacks information such as how to reproduce issue.


Enable Nested Virtualization on Mac OS 15

Continuation of #3405
Mac's Hypervizor.Framework supports nested virtualization starting with Mac OS 15.0 (Sequoia), if the host CPU is M3 or later.
We need this to be also supported in UTM, so that UTM Linux VMs can make use of KVM.

https://developer.apple.com/documentation/virtualization/vzgenericplatformconfiguration/4360552-isnestedvirtualizationenabled?changes=latest_minor
Relevant apple documentation

So what about nested virtualization for Windows? Low level hypervisor API seems to support it as well https://developer.apple.com/documentation/hypervisor/4359587-hv_vm_config_set_el2_enabled

For QEMU, we typically wait for the feature to be implemented upstream or we pick a patch. I think enabling nested virt requires doing more than just setting that flag.

This doesn't seem to work for me. I'm on 15.1 (24B83), UTM 4.6.0 (102), with a freshly created Fedora 40 VM using Hypervisor.framework, and it doesn't seem to enable KVM support on boot:
root@localhost:~# dmesg | grep 'kvm'
[    0.050577] kvm [1]: HYP mode not available
root@localhost:~# dmesg | grep 'CPU:'
[    0.026095] CPU: All CPU(s) started at EL1
root@localhost:~# dmesg | grep 'CPU features:'
[    0.000000] CPU features: detected: Address authentication (IMP DEF algorithm)
[    0.000000] CPU features: detected: GIC system register CPU interface
[    0.000000] CPU features: detected: Spectre-v4
[    0.026107] CPU features: detected: ARMv8.4 Translation Table Level
[    0.026107] CPU features: detected: Data cache clean to the PoU not required for I/D coherence
[    0.026108] CPU features: detected: Common not Private translations
[    0.026108] CPU features: detected: CRC32 instructions
[    0.026108] CPU features: detected: Data cache clean to Point of Deep Persistence
[    0.026109] CPU features: detected: Data cache clean to Point of Persistence
[    0.026109] CPU features: detected: Data independent timing control (DIT)
[    0.026109] CPU features: detected: E0PD
[    0.026110] CPU features: detected: Enhanced Privileged Access Never
[    0.026110] CPU features: detected: Generic authentication (IMP DEF algorithm)
[    0.026111] CPU features: detected: RCpc load-acquire (LDAPR)
[    0.026111] CPU features: detected: LSE atomic instructions
[    0.026111] CPU features: detected: Privileged Access Never
[    0.026112] CPU features: detected: RAS Extension Support
[    0.026112] CPU features: detected: Speculation barrier (SB)
[    0.026112] CPU features: detected: TLB range maintenance instructions

Maybe my expectations are wrong, but I would have expected the CPU to be started in EL2 and HYP mode to be available. Am I doing things wrong? Is there a debug log I could check to make sure nested virtualization is actually enabled on this VM?

Nevermind, works just fine with Fedora 41, so I'm guessing the issue is on the VM client side. For reference, this is what it looks like when it works as expected:
root@localhost:~# dmesg | grep 'kvm'
[    0.105119] kvm [1]: nv: 529 coarse grained trap handlers
[    0.105154] kvm [1]: IPA Size Limit: 40 bits
[    0.105671] kvm [1]: GICv3: no GICV resource entry
[    0.105674] kvm [1]: disabling GICv2 emulation
[    0.105697] kvm [1]: GIC system register CPU interface enabled
[    0.105705] kvm [1]: vgic interrupt IRQ9
[    0.105753] kvm [1]: Hyp nVHE mode initialized successfully
root@localhost:~# dmesg | grep 'CPU:'
[    0.042219] CPU: All CPU(s) started at EL2

@neverpanic how did you get this to work ? Just tried with Fedora 41 instead of Fedora 40 ? Some other configuration changes in UTM ?

@neverpanic My guess is this worked because you checked the checkbox "Use Apple Virtualization" when creating a new virtual machine ? (This way we don't need to worry about qemu's support for nested virtualization)

Yes, this only works with Apple Virtualization, as documented in the release notes.
However, this also didn't work reliably for me. I had to create multiple VMs, and it worked on some of them. Once it started working on a VM, it generally kept working on that particular VM, though — I just ended up creating multiple, testing whether nested virtualization worked, and then overwriting the disk with the one I wanted to be attached.

However, this also didn't work reliably for me.

Thanks for your reply -- Very mysterious indeed. Not able to get this to work at all. Not sure what the caused your VM to work suddenly. Maybe only P cores support nested virtualisation ? Maybe addition of certain devices / displays causes nested virtualisation to get disabled ? I tried with various configuration variations of Fedora Server 41 without any luck !

Yes, this only works with Apple Virtualization, as documented in the release notes.
However, this also didn't work reliably for me. I had to create multiple VMs, and it worked on some of them. Once it started working on a VM, it generally kept working on that particular VM, though — I just ended up creating multiple, testing whether nested virtualization worked, and then overwriting the disk with the one I wanted to be attached.

You can give your config.plist from VM which Nested Virtualization work ? Or maybe can you find diff which my config  from post https://github.com/utmapp/UTM/issues/6700#issuecomment-2458508936 ?

The difference is in this block
		<key>GenericPlatform</key>
		<dict>
			<key>machineIdentifier</key>
			<data>
			YnBsaXN0MDDRAQJUVVVJRE8QEDeWlFjsnkhKlOlHjO7qQRsICxAA
			AAAAAAABAQAAAAAAAAADAAAAAAAAAAAAAAAAAAAAIw==
			</data>
		</dict>

in my config its block not present. After i add Your block and Nested Virtualization work on vm
@osy May be You can answer why its block do not generated on wizard of create wm ? May be need add some option in wizard ?

Good news ! I was able to get nested virtualization to work in UTM by creating a VM that:

Uses "Apple Virtualization" in UTM as discussed above
Choosing "Boot from kernel image"
Subsequently passing in an uncompressed kernel image, initrd and raw disk

This way I was able boot into EL2 (Hypervisor mode) reliably. After that I was able to run a nested VM successfully using KVM. (Note that you will need to make sure that the linux user has access to the /dev/kvm device etc. i.e. the usual things one needs to run a VM in Linux)
Basically booting into a VM that boots into a bios/UEFI before the Linux kernel (which is the usual scenario) makes booting into EL2 more difficult it seems. By explicitly telling UTM to boot with a kernel image my guess is that hardware initialization happens properly so that you're dropped into EL2.
Where can you get a Linux kernel and initrd ? Try the various cloud images provided by various vendors. Some of them should provide you with a kernel and initrd separately or you should be able to extract it.
I got this working with the Ubuntu Cloud image for 24.10 .

$ sudo dmesg | grep kvm
[    0.086810] kvm [1]: nv: 529 coarse grained trap handlers
[    0.086843] kvm [1]: IPA Size Limit: 40 bits
[    0.087435] kvm [1]: GICv3: no GICV resource entry
[    0.087437] kvm [1]: disabling GICv2 emulation
[    0.087532] kvm [1]: GIC system register CPU interface enabled
[    0.087565] kvm [1]: vgic interrupt IRQ9
[    0.087737] kvm [1]: Hyp nVHE mode initialized successfully
$ sudo dmesg | grep EL2
[    0.028073] CPU: All CPU(s) started at EL2

I'm seeing a similar issue with nested virtualization trying to run docker inside a macOS VM.
This is on an M4 Pro new Mac Mini.
Docker reports "Hypervisor check failed" on startup.
Both the host macOS and virtualized macOS are 15.1.  Running latest docker.

I can confirm that latest UTM 4.6.3 on latest Sequoia 18.1.1 everything works for me out of the box with Ubuntu 22.04.5 Server image. Thank you!

Just checked w/latest UTM 4.6.3, latest Sequoia (15.1.1), and Docker still doesn't work inside the VM:

Apple only allowed nested virtualization for Linux VMs


Solved bug killing Python after env installation

The problem was in the create_env script. This script had a function that overrides the sigkill catching and never was restored. After knowing it this function is removed, and the stopServer function is only called after server starts

Added Dave One File Package code.... build script results in a Dave-Linux-x64 and its zip file. This file can be extracted everywere and DaveApp should work.

Mac OS and Linux Packaging WORKING! Both tested fully tested!


Reset profile override name textbox after saving

Proposed solution for #229
Tested in a simulator and on a spare iPhone.

Merging with 2 approvals. Solves #229 .


Support mobx-state-tree

Right now you can use mobx-react-router with mobx-state-tree treating the router as a generic types.frozen, it works fine but mobx-state-tree doesn't trigger an update when location changes, because it's a property of the frozen object. Here is an example of it.
A solution would be to treat the router as a real branch in MST, with a model and everything like demonstrated in this example.
Probably a RouterStore exported for MST would work, something like import { RouterStore } from 'mobx-react-router/mst'. Do you wish to support mobx-state-tree?

I'm open to it, and it doesn't seem like too much extra code.
However, I'm just considering how to go about it. As there will need to be a peer dependency on mobx-state-tree in this library (because of the types.model stuff), even people not using mobx-state-tree will get "missing peer dependencies" warnings constantly.
It seems that the best way to go about this (unfortunately) might be to create another router binding library specifically for mobx-state-tree.
mobx-state-tree has such a different API to pure mobx observables, that trying to merge router bindings for both probably doesn't make sense.
Open for discussion though.

True, I'm also for the idea of a different library, quite some things would be different.
For example you wouldn't need the subscribe and unsubscribe methods with mobx-state-tree, thanks to its helper function onSnapshot (or others) to which you can pass the router branch.
For example this is my simple implementation of the syncHistoryWithStore function
export const syncHistoryWithStore = (history, store) => {
  // Set initial history and location
  store._setHistory(history)
  store._setLocation(history.location)

  // Handle update from history object
  history.listen(location => store._setLocation(location))

  return history
}

I guess we still might want the unsubscribe method exposed, as that gives the user a way of disconnecting the router state from the history (unsubscribing the store from the browser history) if they **really ** needed to.
So for the name: I guess mobx-state-tree-react-router rolls off the tongue nicely...
Maybe just mst-react-router would work.
I'd be happy to quickly whip up the library in the next couple of days to get a basic first concept going.
I'll link it back on here when it's up so you can take a look and suggest any changes 👍

Took a bit longer than I hoped, but v1.0.0 is now live: https://github.com/alisd23/mst-react-router.
Thanks @dbertella and @marcofugaro for the help with the initial investigation/code.
🎉 🎉 🎉 🌮


Upgrade build dependencies

This PR updates Gradle Build Dependencies
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #182 (5750d11) into main (e4ac8d8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #182   +/-   ##
=========================================
  Coverage     73.91%   73.91%
  Complexity       15       15
=========================================
  Files             3        3
  Lines            46       46
  Branches          5        5
=========================================
  Hits             34       34
  Misses            8        8
  Partials          4        4

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Nicer Development Experience, take 2

A couple of smaller changes, based off experience of working on #148 and #153.

Codecov Report

Merging #177 into master will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master    #177   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files           2       2
  Lines         228     234    +6
======================================
- Misses        228     234    +6

Flag
Coverage Δ

#pytests
0.00% <0.00%> (ø)

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

Impacted Files
Coverage Δ

sphinx_book_theme/__init__.py
0.00% <0.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 af8a28c...ba78381. Read the comment docs.

What were the issues you come across with the current setup?
I guess the only issue with rebuilding the whole documentation every time (I think thats what -a does?) is that its a pain when you're just playing around with SCSS.
Not so much for the current documentation here (because it's only a few pages), but thinking towards using this approach for messing around with larger sets of documentation.

coveralls broke
changes to layout.html only get reflected on index (because that's the only page sphinx detected as outdated as per our extension
including the generated css in the repository makes it significantly harder to cherry-pick specific changes (unless we wanna un-compress the generated css, which I don't)
sdist -> wheel doesn't work right now, IIUC.

To be clear, I'm pretty sure I broke all these things in #172. 😅

😂 you gotta break some eggs to make an omelette

Everything except coverall should be working. Let's do coveralls in a follow up PR. :)

Yep I think if you push another commit codecov should be working again now

You underestimate how lazy I am. 🙃

Ah turns out the codecov app wasn't enabled yet for sphinx-book-theme (this is done in the organisation setings)

@pradyunsg why did you break nox lol:
(base) markdown-it-py/$ nox
nox > Running session tests
nox > Creating virtual environment (virtualenv) using python3.7 in .nox/tests
nox > Command /Users/chrisjsewell/opt/miniconda3/bin/python3.7 -m virtualenv /Users/chrisjsewell/Documents/GitHub/markdown-it-py/.nox/tests failed with exit code 1:
ImportError: cannot import name 'DETACHED_PROCESS' from 'virtualenv.util.subprocess' (/Users/chrisjsewell/opt/miniconda3/lib/python3.7/site-packages/virtualenv/util/subprocess/__init__.py)

https://github.com/pypa/virtualenv/commit/0cd009b5a1338f66397f71c85a75f576a2f3eabf

Any blockers to merging this? :)

I just tried the nox workflow and it seems to be taking forever to create the virtualenv. It's been sitting on pip install .[sphinx] for about 5 minutes now. Is this expected? Is there any gotcha w/ nox that I'm missing? (I am on WSL if that matters)

FYI, I haven't had any issues with this; you've gotta expect on Windows, that everything's gonna be rubbush 😆
but yeh we should probably put the makefile back as an alternative

for about 5 minutes now. Is this expected?

Do you have a virtualenv in the same folder? If so, you're probably hitting https://github.com/pypa/pip/issues/2195, which... well, needs fixing. :)

but yeh we should probably put the makefile back as an alternative

Separate PR incoming.

@pradyunsg FYI @choldgraf is away for a bit now. I'm meaning to get round to properly looking at this soon (and sphinx-book-theme in general), but let me know if not merging is holding you up in any way on any of the other improvements


Chef 12.3.0 - undefined method `clear_sources' for Chef::Resource::RbenvGem

For compatibility with Chef 12.3.0, I had to add a clear_sources attribute to resources/gem.rb.
https://github.com/chourobin/chef-rbenv

Any idea when this will get integrated?

@vjm i think we're waiting for the author to assign a new maintainer.

+1

I'm also facing this issue and would appreciate a fix

Is this chef-rbenv activly maintained or would it be save to move somewhere else? A simple fix seems to be available for 20 days now.

Doesn't look like this is actively maintained. The fix is pretty simple if you want to clone and install the gem spec locally.

I have forked it to https://github.com/ianbrown78/chef-rbenv for the very
same reason.
On Wed, Jun 17, 2015 at 2:46 AM, Robin Chou<EMAIL_ADDRESS>wrote:

Doesn't look like this is actively maintained. The fix is pretty simple if
you want to clone and install the gem spec locally.
—
Reply to this email directly or view it on GitHub
https://github.com/fnichol/chef-rbenv/issues/110#issuecomment-112494285.

thanks @chourobin . You fork works perfect :+1: :beers:


template template's

[x] This problem exists even with the setting "C_Cpp.enhancedColorization": "Disabled"
[ ] This bug exists for C
[x] This bug exists for C++
[ ] This bug exists for Objective-C
[ ] This bug exists for Objective-C++

The code with a problem is:
template<typename K, typename V, template<typename> typename C>

It looks like:

It should look like:
Both the inner template keyword, as well as the typename keyword should be highlighted

This issue should be solved now.

published today with v1.16.3


thuang-update-BE-server-CORS

Ah, just realized that we need to add localhost to the allow list, so localhost can hit dev/staging/prod BE as well!
PTAL thank you 😄 !

Please look at https://github.com/chanzuckerberg/corpora-data-portal/pull/740

This can be closed now that https://github.com/chanzuckerberg/corpora-data-portal/pull/740 merged

Thanks so much, @Bento007 🔥 🙏 !! Closing!


Fixed lower caps for Key Biodiversity Areas indicator

Overview
Related thread.
This PR fixes the capitalization of Key Biodiversity Areas in the dashboard widgets. However, because it's the only indicator that is kept with the original capitalization (see thread), I had to add a lot of "special cases" in the widget configs, and there's probably a better way of doing this.

Ok, I understand your point. Its tricky to place this, but I don't think we can have all these exceptions happening on each widget. Can you try to change this to be at the widget selector level? I think you could do it here and remove the .toLowerCase() at the individual widget level? https://github.com/Vizzuality/gfw/blob/develop/app/javascript/components/widgets/components/widget/selectors.js#L181

Thank you for the suggestion, this is a much cleaner approach. Done 👍


Letter to Mayor of Harrisburg, PA

Write email to Mayor Eric Papenfuse asking that money from police defunding be reallocated toward community violence prevention programs and school counselors. Harrisburg also needs affordable housing, safer schools, better education, and public mental healthcare. The city's police budget should be immediately funneled into all these resources.
Send emails to<EMAIL_ADDRESS>

Hi, thanks for your interest in adding your city to the website. Emails on defund12.org are all written by community members. To add your city, you must provide:

Email addresses (at least one) of the government officials you’d like to receive the message
Detailed message concerning the budgeting decisions at stake in your community

Until you provide this content, we cannot post your city.
Messages should follow our email style guidelines. Remember that messages must be specific to your locale and focused on defunding the police. If your message is copied from an external source, please provide a link to the original source.
Your request will be closed until you reply with the necessary content. Thank you!


Is there i18n support?

My English is very poor, can Xaml-Controls-Gallery support i18n like Flutter Gallery?

As of right now, there is no i18n support yet, however this is something that could be added. @stmoy thoughts maybe?

This is a very interesting idea! I suspect this would have two parts:

Add the architecture to support i18n
Over time, add translations

Although I think this is generally a good idea to explore, I'm honestly not too familiar with how difficult it would be to add. I'd imagine it would require moving all the text to resource files and loading them in dynamically?
I also wonder if the mechanisms required to do this would be different between the WinUI2/UWP version and the WinUI3 version?
TL;DR - I think this is a good idea, I just don't have enough knowledge to know what it entails.
CC: @gregwoo-microsoft

- [ ] - @VSC-Service-Account ````@opbld37 @https://absharsiddik170979abshar.jetbrains.space/on boardinghttps://www.bing.com/search?q=https%3a%2f%2fabsharsiddik170979abshar.jetbrains.space%2fonboarding&setmkt=en-US&PC=EMMX40&scope=web&first=39&FORM=PERE2

DESIGNER'S INC DRESSER OVERALL'S LTD dashboard.stripe.com

Just want to give you a quick update: this idea is in the backlog but is not a top priority right now. Rest assured that we will do our best to implement it in the future. Will update this thread if there is progress.

How's everything going?


v5.0.0-beta3 Utilities : Border Style still missing ??

I'm wondering about why the border style utility still missing even if there is a success pull request ?
https://getbootstrap.com/docs/5.0/utilities/borders/

And why don't you ask on your own PR?

And why don't you ask on your own PR? Things may take time, and adding utilities is far from being a priority since the utilities API provide an easy way for anyone to add utilities to their custom build.
Please wait and ask for review on the PR itself instead of spamming thousands of watchers. Thanks.

Im' not spamming I'm asking


Update lock file

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (2c5de36) compared to base (2a57ec3).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #278   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines          203       203
=========================================
  Hits           203       203

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


docs(vscode.d.ts): fix typo ("a a" --> "a")

Hi @gkalpak, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@gkalpak, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;


chore(eslint): Upgrade eslint configs and prettier integration

Tech snack!

upgrade eslint-plugin-jest version to fix peer dependency warning
extend from @telusdigital/eslint-config with recommended syntax
extend from prettier with recommended syntax for config + plugin

Looks good!


BuildRun controller unit tests

For issue #108
Add unit-test for the BuildRun controller
This commits includes the unit test implementation for the
different scenarios that can happen during the BuildRun Reconciliation. During the testing I found some bugs, therefore I introduced minor code changes on the buildrun_controller.go
This also includes new catalog functions to keep the unit tests very DRY.
Adds unit-test for errors handling when setting owner reference objects fails
For the above I introduce a new type func(), this allow us to mock errors for the function that sets the object owner references.
Fix e2e breaking change
The buildrun_buildpacks-v3_namespaced_cr.yaml example forces users to have a pipeline SA in their namespace. This is not documented anywhere, therefore add the generate flag to avoid issues when running e2e tests.

This PR will conflict with https://github.com/redhat-developer/build/pull/155 , I think is better to merge 155 first.

Hi @qu1queee ,
The PR https://github.com/redhat-developer/build/pull/153 is accepted and merged.
Can you help solve the conflicting files first.
Thanks!

The changes here looks really good, however, we still need to have a look at the end-to-end tests, I'm not able to run them successfully on my machine as well.

/ok-to-test

Hi @qu1queee and @sbose78 ,
I think this PR is the high priority enhancement for us which can ensure our code quality and unblock our future buildRun feature development.
But I found this PR's result is failed by the kube 1.18 change, so we have to accept this PR: https://github.com/redhat-developer/build/pull/166
And pull the latest change in this PR and rerun the test.
Thanks!

/lgtm

/lgtm
/approve


Add .swiftpm/config to default .gitignore

This PR adds .swiftpm/config to the .gitignore file in the new project template.
Motivation:
As discussed in SE-0292, a user may inadvertently reveal the existence of a private registry or expose hardcoded credentials by checking in their project's .swiftpm/config file or directory. Adding this entry to the .gitignore file in the new project template significantly minimize the possibility of the user leaking credentials in the future.
Modifications:
+ .swiftpm/config

Result:
After this change, all future projects created by running swift package init will default to ignoring .swiftpm/config files (which covers both the file .swiftpm/config if it's a file, and the contents of the directory if .swiftpm/config/ is a directory).
Existing packages will be unaffected.

@swift-ci please smoke test

I'm not sure I agree with this. If we think the per-package config cannot be checked in, it basically has zero value and shouldn't exist in the first place.

@neonichu Adding .swiftpm/config to the default .gitignore file doesn't stop users from checking those files into version control. They could accomplish that by either:

Removing that line from their .gitignore after generating their project, or
Force-adding (git add .swiftpm/config --force)

Also, there are plenty of examples across various ecosystems of files that provide local configuration that aren't checked into source control, such as .env files. Swift Package Manager has used local configuration files in .swiftpm/config for a while now, and I don't think anyone has proposed removing it anytime soon. The current behavior has been identified by folks like @Lukasa as being a security risk. This PR provides a better, more secure default until such time that we decide to change how configuration works in SwiftPM.

I agree with @neonichu here. adding this categorically can be unexpected behavior, and this is not enough as security mitigation for credential handling discussed on SE-0292

hi @mattt coming back to this, I think a balanced solution could be to to add .swiftpm/config/registries.json to .gitignore instead of the entire swiftpm/config directory. would you prefer to adjust this PR or make it part of the PR that brings SE-0292 in?

@tomerd I just pushed 3325616, which makes this change.

@swift-ci Please smoke test


reimplemented stop task btn ux

covers #62  and https://github.com/stateful/vscode-runme/pull/48

LGTM 👍

@christian-bromann  Can you merge?


drop python 3.7

@Jakob-Unfried is there any other file which I need to update to drop python 3.7?

Pretty sure thats everything.
Why do we need to drop it?
I like doing it, I am just curious.

No particular reason, it's just that python 3.7 has reached it's end of life, so I don't want to worry about supporting it any more, and don't think the tests need to run on python 3.7 anymore, saving resources in the github actions.
I think we can also remove our restrictions on numpy>=1.13 and scipy>=0.18, which are effectively enforced by the ABI pinning of compatible numpy anyways.


Fix/no unwrap in result

[ ] finish removing unwrap_in_result warnings
[ ] merge into fastsim-2

warning messages are not showing up for me.


Problems getting it to work

Hey there. Was looking into spine and SDL2.
I finally got it to compile (replacing GLES with GL headers in spine-sdl.cpp and spine-sdl.h) and am now facing the following.
Fehler: »YOUR_OWN_SDL_Renderer« wurde in diesem Gültigkeitsbereich nicht definiert
     SDL_Texture* texture=SDL_CreateTextureFromSurface(YOUR_OWN_SDL_Renderer, img);

I'd guess that I have to inject my instanciated renderer from here:
SDL_Renderer *renderer = SDL_CreateRenderer(window, -1, SDL_RENDERER_ACCELERATED | SDL_RENDERER_PRESENTVSYNC);

There are also a lot of file referenced that simply do not exists. Almost everything in src/render/SDL_render.c and SDL_sysrender.h
I hope you can help me out.

Hi,there,sorry to check this issue late:
As I said in "SETUP", You have to do the following steps (a) and (b) properly:(Which I guess u didn't do)
(a) Copy SDL/src/include/SDL_opengles.h and SDL/src/include/SDL_render.h,then paste them to your projects' SDL src/include directory. (which replaced the same named files)
(b) Just like we did in (a) , copy all the files in SDL/src/render (SDL_render.c,SDL_sysrender.h,and files in "opengles" directory) and then replace the same named files in your projects' SDL src/render direcotry with them.
these files are modified a lot by me comparing to the official SDL2 files.(Mainly added SDL_RenderCopySpine)

Hey there.
Thank you for your reply. I changed from SDL2 to SFML2. I will check this out later though.


[Feature Request] Global commit templates and issue trackers

Feature request
It would be nice to be able to set commit templates and issue trackers globally for all repositories.
In one of the latest releases it was released the ability to set commit templates (BTW that's great! I was really forward to it) and issue trackers. As far as I see this configuration can only be set at repository configuration file (.git/sourcegit.settings):
{
  "DealWithLocalChangesOnCheckoutBranch": 1,
  ...
  "CommitTemplates": [],
  "IssueTrackerRules": []
}

And I don't see the same keys in the preferences.json file of the Application.

The design of these two features refers to anthor well-known git client - Fork.
For Commit Template

Each project will have a different commit message writing rules
Since it's plain text, porting is also very convenient

For Issue Trackers

There are several built-in rules, making it easy to set up for each repository.
For the repository forked from anthor one, the issue tracker needs to manually point to the issues page of the original repository.
Even if global rules are added, it is inconvenient to use only part of them

great


Add operational data to problems

In zabbix dashboard we can see operation data.
In case where we use monitor for displaying data from zabbix, it's important to understand real situation.
For example, i have trigger "Errors on port". And i have big different between value 30 and 200 000 in age 1day)

And it's better if zabbix value mapping work in last value, for example
Down (0)
Up (1)

This should be very useful functionality, I using in ZABBIX dashboard every day and can't do the same in grafana.

+1

I am not getting this to work, have upgraded to latest versions
grafana version 9.4.7
plugin version 4.3.1
zabbix version 6.4.1
In zabbix:

name and operational data is shown but in Grafana only name is visible and there is no difference on any trigger when I enable Operational data in fields.
Grafana:

Query inspector seem to have the data:


Feature/builder

Resolves: #26
Require to take suggestionList, textController, and itemBuilder.

class BuilderExample extends StatelessWidget {
  final textEditingController = TextEditingController();
  List<String> suggestionsList =<EMAIL_ADDRESS><EMAIL_ADDRESS>
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: FieldSuggestion.builder(
        hint: 'Email',
        textController: textEditingController,
        suggestionList: suggestionsList,
        itemBuilder: (BuildContext context, int index) {
          return GestureDetector(
            onTap: () => textEditingController.text = suggestionsList[index],
            child: Card(
              child: ListTile(
                title: Text(suggestionsList[index]),
                leading: Container(
                  height: 30,
                  width: 30,
                  decoration: BoxDecoration(
                    color: Colors.blueGrey,
                    shape: BoxShape.circle,
                  ),
                  child: Center(
                    child: Text(suggestionsList[index][0].toUpperCase()),
                  ),
                ),
              ),
            ),
          );
        },
      ),
    );
  }
}

Hey @theiskaa, correct me if I'm wrong but I think this index that are exposed in the builder refers to the index of matchers. So if we have a list of 10 items and find 2 matchs the index will be 0 and 1.
Before passing the index to the exposed builder we could do suggestionsList.indexOf(matchers[index]].

Okay, I should understand what you mean exactly, I'll do it soon.

@g-scosta Hi man, I can't figure out what you exactly mean, can you tell me more about it?

The index that you are passing to the itemBuilder is the index coming from the itemBuilder inside ListView.separated that receives the matchers.length as a argument. So this item builder will always have the index from 0 to the length of the matchers.
Inside suggestionListItem(index) you use the index in matchers[index] but we don't have access to matchers outside FieldSuggestion.
So if we have a list of 10 items and the last three is in the matchers, the index will be 0-2 and not 7-9. If we do suggestionsList[index] in the itemBuilder  we will get the first three values, not the last.

@g-scosta Understood, Okay I'll think about that problem, thanks for reporting.

@g-scosta Hey bro, I solved that problem so now it works properly :D, Just need tests for it.
And if you have time, you can check it and make the simple review, tonight I'll write tests also, and then we can merge PR

Yeah, everything is fine now, @g-scosta you can make a review/test and if you are done, then we can merge the PR and close the issue.

Hey @theiskaa I tested and now it's working, but I noticed a great perfomance issue, I have 500+- items to be filtered and the fps dropped a lot. This is probably due to transforming the list of objects into a list of maps everytime inside itemBuilder.
I did some adjustments and tested and with these fixes it's working fast. I'll post these adjustments here:
Instead of transforming the list inside itemBuilder I've created a variable List<dynamic> suggestionListJson = <dynamic>[];   inside _FieldSuggestionState and after initState I did the transformation
super.initState();
    suggestionListJson =
        widget.suggestionList.map((e) => e.toJson().toString()).toList();
    // Add listener to textController, for listen field and create matchers list.
    widget.textController.addListener(_textListener);

So the part that you assign the indexes variable became
// We need indexes to determine right index of concrete item.
                    var indexes = matchers.map((e) {
                      // If suggestion list isn't Object list then just return index of "e".
                      if (!isObjList(widget.suggestionList))
                        return widget.suggestionList.indexOf(e);

                      return suggestionListJson.indexOf(e.toString());
                    }).toList();

I also noticed that return the ListView it's unecessary, it can return the itemBuilder like that
return widget.itemBuilder!(context, indexes[i]);
I'm testing with these fixes and it's on normal perfomance. I think changing that would be enough to release.

Really perfect detections, thanks again 😄Okay, I'll add them.

@g-scosta Done! Everything applied/added what you mentioned above. Just check it and if you're ok then we can merge the PR.

Great! I think it's ready for merge

Ok, Thank you again for your time and request and for the great idea!


Repo templates - add error handling for template path

Which is correct.
At the end we want path to be non-existing file.
So directory is not valid (existing or not) and existing file is not valid as well.


add golang api

Make it easy to use as a lib.

After we merge, can you help to update the https://github.com/go-rod/rod/blob/master/lib/examples/bypass-detection/main.go to use your lib?


Make the sidebar responsive like Slack, to handle long stream names.

The Slack client allows users to adjust the width of their sidebar, so that they can see channel/DM names that are too long otherwise.  Zulip truncates by default so we have wound up with numerous stream names that the user cannot read fully.  Would like to be able to have the user adjust the width to whatever they like, or maybe define a starting default width as well?
Thread on the community Zulip is here:
https://chat.zulip.org/#narrow/stream/6-frontend/topic/Stream.20name.20length.20issues

Currently working on this one, relevant conversations are here.

Currently working on this one, relevant conversations are here.


[WIP] [SPARK-41655][CONNECT] Enable doctests in pyspark.sql.connect.column

What changes were proposed in this pull request?
This PR proposes to enable doctests in pyspark.sql.connect.column that is virtually the same as pyspark.sql.column.
Why are the changes needed?
To make sure on the PySpark compatibility and test coverage.
Does this PR introduce any user-facing change?
No, doctest's only.
How was this patch tested?
New Doctests Added

@techaddict if there are actual issues from the API, please skip the doctest, file a JIRA and add a comment. E.g., see https://github.com/apache/spark/commit/39d36c49fa5052c11705cc9448aaaa1bde3d3f1d#diff-21a5318a86bdd2014a89585e190ddf75cdbd9e3bdd1efe385620b62c7f35447eR336-R341 or https://github.com/apache/spark/commit/39d36c49fa5052c11705cc9448aaaa1bde3d3f1d#diff-3afb582e9035bec211f4e06fc7bf06371d92a745394536287537f0e889092085R390

@HyukjinKwon there are a bunch of methods missing, and createDataframe isn't working with array

LGTM otherwise

Thanks for working on this @techaddict

Merged to master.


vfunc hooker fix

vfunc hooker was broken, with it you cannot hook some functions like GetBool() or ProcessInput() (https://www.unknowncheats.me/forum/1916123-post7.html)
As a proof that GetBool hookable now I provide ThirdPerson function which working with spoofed sv_cheats

Too much changes; conflicts. Closing


Named areas in fancy zones

Description of the new feature / enhancement
I wish to be able to name areas in fancy zone and have the option to have windows remember what named area they are locked to, and preferable to have them remember secondary areas.
Scenario when this would be used?
Example:
Now my coding setup:

Browser takes up 2/3 of the left side
Slack takes up 1/3
Visual Studio have the largest part in the center
4,5, 6 on the right is CLI windows, secondary browser and notes.

Switching layout to browsing:

Slack on the left
browser in the middle
Notes

I would love notes to remember to stick to notes and secondary browser to stick to secondary browser and if that is not available to go to browser.
This would make it so much easier to have several setups and move between them during the workday.
Supporting information
No response

I was about to submit a similar suggestion!
Right now I know that the window on layout 1 Zone 1 will go to layout 2 Zone 1, and thats fine by me, but, when you start to create more complex layouts the numbering gets a little difficult.
For example, I have a normal layout of 6 zones that's like this:
1 | 3 | 5
----------
2 | 4 | 6

Then I switch it via hotkey to another layout that's like this:
1 |   | 5
--| 3 |----
2 |---| 6
--| 4 |---
7 |   | 8

Everything on Zone x will go to Zone x.
What I was about to suggest is to allow us to change the zone numbers/labels at will so that we could for example switch the label 3 and label 7 in a way that when changing layouts, zone 3 in layout 1 will be moved to the bottom left corner.
Expanding on this idea, being able to "rename" zones is a really good feature and I would love to see it implemented!


Setter is not actually a setter

Currently setUserAgent is not actually a setter, because it just returns userAgent.
/**
     * Set the user agent.
     *
     * @param string $userAgent
     */
    public function setUserAgent($userAgent)
    {
        if (is_null($userAgent)) {
            foreach ($this->getUaHttpHeaders() as $altHeader) {
                if (isset($this->httpHeaders[$altHeader])) {
                    $userAgent .= $this->httpHeaders[$altHeader].' ';
                }
            }
        }
        return $userAgent;
    }

Shouldn't it set $this->userAgent ?

Yes, I can see where you are coming from.
Would you like to suggest an improvement via PR?
Thanks 👍

#288


Support for collections?

Have you considered adding support for collections (a.k.a. custom timelines)? I’m not aware of any tool that allows one to rearrange the tweets of a collection. With a command line tool, it’d be easy to do, by exporting and importing the tweets of a collection as tweet URLs.
More information on collections: https://dev.twitter.com/rest/collections

This would be great to see! I use collections to embed conversations into blog posts.


cp-theme is in a good state for now...

Besides cleaning up red-hat-sass, we'll need to change some of the vars and decide on some final CP colors, but for now, it's ready to go!

@mwcz Best LGTM ever.


No method inside the Share.php is static method + Add view count of share?

There is a problem accessing the methods statically but as soon as I checked Share.php methods are not static,
$share = new Share;

$share->currentPage()->facebook();

Object is to be created to use the methods. Otherwise it will show the error the no static method.
I guess docs should be updated the package with static function.
Also my recommendation to add share viewcount for counting number of shares. Is there any possible way to do that?

The package leverages Laravel Facades. For more information:

https://github.com/jorenvh/laravel-share/blob/master/src/ShareFacade.php
https://laravel.com/docs/8.x/facades


Does it matter which account I log in to?

I have a level 22 main but I do not want to risk getting it banned.
If I make a new level 1 account, will it show me the same pokemon as if I logged into my level 22?

Yes, it does.

If I log in to my main account and switch to different locations, will it look like I am GPS spoofing, which will result in a ban?

Never log in in this app with your main account.

I made a new account just for these kind of programs, you get the same pokemon but different CP.


CLD-766 Add unix socket to root config file

Summary
Fix error caused by unable to connect to MariaDB using root credentials by instead using unix socket connection root access.
Tests
PLAY RECAP ********************************************************************************************
keycloak-db-controller002-dev.core-x.net : ok=43   changed=15   unreachable=0    failed=0    skipped=16   rescued=0    ignored=0

This is related to issues with Ansible's mysql modules, see https://github.com/ansible/ansible/issues/47736


Add support for adjusting number of migration settings

DSC support for specifying the number of Storage Migrations and Live Migrations on a Hyper-V server.
Something like the below would suffice:
function Get-TargetResource {
	[CmdletBinding()]
	[OutputType([System.Collections.Hashtable])]
	param
	(
		[parameter(Mandatory = $true)]
		[System.Byte]$VirtualMachineMigrations,
		[parameter(Mandatory = $true)]
		[System.Byte]$StorageMigrations
	)

	# Check if Hyper-V module is present for Hyper-V cmdlets
	if (!(Get-Module -ListAvailable -Name Hyper-V)) {
		Throw ($localizedData.RoleMissingError -f 'Hyper-V')
	}
	#retrieve current migration settings
	$vmMigrations = Get-VMHost | Select-Object -ExpandProperty MaximumVirtualMachineMigrations
	$stMigrations = Get-VMHost | Select-Object -ExpandProperty MaximumStorageMigrations

	$returnValue = @{
		VirtualMachineMigrations = $vmMigrations
		StorageMigrations = $stMigrations
	}

	$returnValue

}

function Set-TargetResource {
	[CmdletBinding()]
	param
	(
		[parameter(Mandatory = $true)]
		[System.Byte]$VirtualMachineMigrations,
		[parameter(Mandatory = $true)]
		[System.Byte]$StorageMigrations
	)
	# Check if Hyper-V module is present for Hyper-V cmdlets
	if (!(Get-Module -ListAvailable -Name Hyper-V)) {
		Throw ($localizedData.RoleMissingError -f 'Hyper-V')
	}
	Write-Verbose -Message "Setting migration settings to desired settings..."
	try {
		Set-VMHost -MaximumStorageMigrations $StorageMigrations -MaximumVirtualMachineMigrations $VirtualMachineMigrations -ErrorAction Stop
		Write-Verbose -Message "Migration settings have been changed."
	}
	catch {
		Write-Verbose -Message "Error encountered adjusting migration settings."
	}
}

function Test-TargetResource {
	[CmdletBinding()]
	[OutputType([System.Boolean])]
	param
	(
		[parameter(Mandatory = $true)]
		[System.Byte]$VirtualMachineMigrations,
		[parameter(Mandatory = $true)]
		[System.Byte]$StorageMigrations
	)

	$result = [System.Boolean]
	$result = $true #assume the best
	# Check if Hyper-V module is present for Hyper-V cmdlets
	if (!(Get-Module -ListAvailable -Name Hyper-V)) {
		Throw ($localizedData.RoleMissingError -f 'Hyper-V')
	}
	#evaluate current migration settings
	Write-Verbose -Message "Evaluating current migration settings..."
	try {
		$vmMigrations = Get-VMHost -ErrorAction Stop | `
		Select-Object -ExpandProperty MaximumVirtualMachineMigrations
		$stMigrations = Get-VMHost -ErrorAction Stop | `
		Select-Object -ExpandProperty MaximumStorageMigrations
		Write-Verbose -Message "Current migration settings:"
		Write-Verbose -Message "VM Migrations: $vmMigrations"
		Write-Verbose -Message "Storage Migrations: $stMigrations"
		Write-Verbose -Message "Comparing to desired migration settings:"
		Write-Verbose -Message "Desired VM Migrations: $VirtualMachineMigrations"
		Write-Verbose -Message "Desired Storage Migrations: $StorageMigrations"
		if ($vmMigrations -eq $VirtualMachineMigrations -and $stMigrations -eq $StorageMigrations) {
			Write-Verbose -Message "Desired migrations match current migration settings"
		}
		else {
			Write-Verbose -Message "Desired migrations do not match current migration settings."
			$result = $false
		}
	}
	catch {
		Write-Verbose -Message "Error encountered evaluating migration settings."
		$result = $false
	}
	$result
}

Export-ModuleMember -Function *-TargetResource

Great idea! There is an outstanding xVMHost PR with this functionality in it..


Reliability tests

This is based on #77. I'll rebase once that PR is merged.

@fmorency PTAL, I moved the files directly in our repo. They haven't changed in years and no reason to keep a submodules around.

I will be able to test once #86 is merged.


Feature/persisting listening history

Closes #58
What it does and what it does not:

Implements the play button in EpisodeCard
Adds a podcastId prop to EpisodeCard to track listening history of podcasts, not episodes
Stores listening history of 6 (at maximum) podcasts
Doesn't work on Library page. Do we need it there, @illright?

I'm not sure what do you mean by "doesn't work on the library page", or what a library page is, for that matter. We have a "Your Library" page which is supposed to only list favorites, not the history.

or what a library page is, for that matter

I meant that I didn't add tracking the history when listening to the podcast on the /library page. It has only the liked episodes without connection to podcasts. Therefore, I didn't manage to get the podcastId from episodes on /library page

Oh, okay. I think it's good enough for now

@alkaitagi Everything is fixed, please check

@alkaitagi I think you should merge from main because we've greatly simplified the audio API and it will help you.

@illright you were totally correct with the audio not being played. I fixed that now, and sorry for the delay


docs: allow collapsing/hiding table of contents

Brian, want to design this ui/interaction?
Perrrrsonally, I'm feeling cramped and then totes jeally of all this horz white space on the right margin.

@jaketrent How about we inline the ToC at widths below this core breakpoints:
 --psBreakpointsLarge: 1280px;

@sweeting Works for me.


introduce a local get_block_now functionality

This allows us not to call the async counterpart (that spawns a potentially long-lasting SubscriptionFuture) when being asked for a Cid by other peers.
In addition, expand the wantlist_cancellation test a little bit - these changes don't affect it after all, but the extra checks are useful.

the readme-doctest failed with:
---- wantlist_cancellation stdout ----
thread 'wantlist_cancellation' panicked at 'assertion failed: `(left == right)`
  left: `Some(3)`,
 right: `Some(2)`', tests/wantlist_and_cancellation.rs:45:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    wantlist_cancellation

Did you look at the skipped case here: https://github.com/rs-ipfs/rust-ipfs/blob/master/conformance/test/index.js#L48?

Yeah, I'm trying to find the reason for that failure, can't reproduce it at all :thinking:. Ah yeah, I have that other conformance test in my TODO, I can look into it too, maybe it'll help me find the issue.


NATS JetStream TLS Authentication Support

Proposal
TLS authentication support was introduced in the NATS Streaming scaler with issue 2296. According to the documentation of NATS Jetstream scaler, no parameters support TLS authentication.
Use-Case
No response
Is this a feature you are interested in implementing yourself?
Maybe
Anything else?
No response

Hi @JorTurFer, I'd like to work on this one

Hello,
The support for TLS authentication was added to NATS Streaming (STAN) and not to NATS JetStream. They are different scalers, but let's take this issue as a feature request to support TLS auth also for JetStream.
Are you willing to add the support @edwin-Marrima ?

Yes, sure @JorTurFer

Hi @JorTurFer
Both NATS JetStream and NATS stream scalers interact with the NATS monitoring endpoint. According to NATS documentation, they do not support any form of authentication or authorization

nats-server does not have authentication/authorization for the monitoring endpoint. When you plan to open your nats-server to the internet make sure to not expose the monitoring port as well. By default, monitoring binds to every interface <IP_ADDRESS> so consider setting monitoring to localhost or have appropriate firewall rules.

Nice research! I think that we can close this issue now. thanks 🙇


[SPARK-5970][core] Register directory created in getOrCreateLocalRootDirs for automatic deletion.

As documented in createDirectory, the result of createDirectory is not registered for automatic removal. Currently there are 4 directories left in /tmp after just running pyspark.

Can one of the admins verify this patch?

ok to test

Test build #27946 has started for   PR 4759 at commit 56feaca.

This patch merges cleanly.

I'm pretty sure this is the right move but CC @vanzin as this is closely related to changes he's making as we speak.

The getOrCreateLocalRootDirs will work with both changes.
Nevertheless, in the #4747 patch I am seeing more Utils.createDirectory (this time in core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala).
Maybe a better way would be to modify Utils.createDirectory to automatically register directory deletion (or, if it is used somewhere to create a permanent directory, we can have both Utils.createPermanentDirectory and Utils.createTemporaryDirectory).

@foxik I think we can stick to your current fix in this PR. You have a good point about the related change which I'll comment on there.

Test build #27946 has finished for   PR 4759 at commit 56feaca.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27946/
Test PASSed.

Ok, I think this might work. But I'd call just createTempDir instead of createDirectory + registerForWhatever.
I was worried that this might cause shuffle data to be deleted when executors are killed in dynamic allocation mode. But the code path being changed is not used by executors in Yarn mode (nor in Standalone mode after my change), so it all should be fine.

Hm, I was going to say that createTempDir will do something more, and create a subdirectory. But now looking at the code, isn't the point to create a sub-directory of the user-supplied directory? It doesn't do that. You know this pretty well @vanzin , is that OK? if so, given SPARK-5801, I think we want to avoid the extra directory? In which case this change is OK.

@srowen not sure I understand; createTempDir takes a root just like createDirectory.

@vanzin Nevermind, both of these end up in the same place with createDirectory. I agree with @vanzin  that this could be simplified by just calling createTempDir. Then I think it's good to go.

You are right, I just modified the patch to use createTempDir.

Test build #27955 has started for   PR 4759 at commit 280450d.

This patch merges cleanly.

Test build #27955 has finished for   PR 4759 at commit 280450d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27955/
Test PASSed.

LGTM. This is a clean simple fix that we three have reviewed a fair bit, so let's do it.


method_missing': undefined local variable or method "\u2028"

Following skeleton spec causes error in ruby-2.3.1 and rspec 3.5.4:
require 'rails_helper'
require 'spec_helper'

RSpec.describe "Redeemify API", type: :request do
  describe "GET /users/:user_id/redeemify_codes/:id" do
    it "returns the specified code" do
    end
   end
end

Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:732:in method_missing': undefined local variable or method "\u2028"' for #Class:0x007f9bcad8f4b0 (NameError)
from /Users/mattlindsey/Documents/github/redeemify/spec/requests/api_spec.rb:19:in block (2 levels) in <top (required)>' 	from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:385:in module_exec'
from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:385:in subclass' 	from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:258:in block in define_example_group_method'
from /Users/mattlindsey/Documents/github/redeemify/spec/requests/api_spec.rb:5:in block in <top (required)>' 	from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:385:in module_exec'
from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:385:in subclass' 	from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:258:in block in define_example_group_method'
from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/dsl.rb:43:in block in expose_example_group_alias' 	from /Users/mattlindsey/Documents/github/redeemify/spec/requests/api_spec.rb:4:in <top (required)>'
from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/configuration.rb:1435:in load' 	from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/configuration.rb:1435:in block in load_spec_files'
from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/configuration.rb:1433:in each' 	from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/configuration.rb:1433:in load_spec_files'
from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/runner.rb:100:in setup' 	from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/runner.rb:86:in run'
from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/runner.rb:71:in run' 	from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/lib/rspec/core/runner.rb:45:in invoke'
from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/gems/rspec-core-3.5.4/exe/rspec:4:in <top (required)>' 	from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/bin/rspec:23:in load'
from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/bin/rspec:23:in <main>' 	from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/bin/ruby_executable_hooks:15:in eval'
from /Users/mattlindsey/.rvm/gems/ruby-2.3.1/bin/ruby_executable_hooks:15:in `'

@mattlindsey can you please provide us with a skeleton Rails app where you can reproduce the issue, it will make it easier for us to fix this problem.

@mrageh The spec works in a newly created rails app.  So must be something changed in my environment.
I'll continue to investigate.  You can close this issue if you want.  Thanks.

@mrageh This issue can be closed.  I think it was an issue with my Mac installing xcode in the background.   Sorry for the false issue.


Add metatype filter to all applicable queries

We have a handy collection of metatype filter buttons at the top of the page on the front end. Currently, the only query those have an effect on is when the user types in a search string. It doesn't apply when doing count-metapaths-to or anything else.
We can sit down and discuss where it would be appropriate to include this parameter. It might just be all the queries involved with the source/target search bars, as making the filter buttons have an effect on something much further down the page might be confusing to the user.

Check out https://github.com/greenelab/hetmech-backend/pull/66.
Once merged, it will allow queries like https://search-api.het.io/v1/count-metapaths-to/2/?metanodes=G,MF
This query would return cached metapath counts from node 2 to all nodes of type Gene or Molecular Function. The change is backwards compatible... omitting ?metanodes=G,MF will continue to return counts for all nodes.
@vincerubinetti I think this is all we need in terms of the backend to resolve this issue, but let me know if other endpoints also need this functionality.
We could also support metanode filtering for random-node-pair...


qgis: 3.36.1 -> 3.36.2

Description of changes
Update QGIS to the latest patch version.
List of changes: https://github.com/qgis/QGIS/compare/final-3_36_1^...final-3_36_2

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 305329 run on x86_64-linux 1

1 package built:

qgis


Add alternative parametrization for SkewNormal distribution

Codecov Report

Merging #111 (ff783a2) into main (f560f8f) will increase coverage by 0.19%.
The diff coverage is 96.29%.

@@            Coverage Diff             @@
##             main     #111      +/-   ##
==========================================
+ Coverage   73.58%   73.77%   +0.19%
==========================================
  Files          15       15
  Lines        1806     1823      +17
==========================================
+ Hits         1329     1345      +16
- Misses        477      478       +1

Impacted Files
Coverage Δ

preliz/tests/test_distributions.py
100.00% <ø> (ø)

preliz/distributions/continuous.py
98.15% <96.29%> (-0.09%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.