Support dearmor GPG key for keyring

Use Case
Some provider we use like Google, Hashicorp only provide ASCII armored gpg key (if I did not miss anything), which seems to not work with the apt keyring mechanism implemented to fix #1034 as from my understanding apt expect binary gpg keys.
Describe the Solution You Would Like
Having a boolean parameter in apt::keyring to specify if the key is armored and dearmor it if necessary
Describe Alternatives You've Considered
Manually dearmore GPG keys and self host them either in puppet code or third party solution
Additional Context
n/a

ASCII armored keys are supported. You have to name them with a .asc extension if ASCII armored, and .gpg if binary. This is an APT thing, not an issue with this module. This is documented: https://github.com/puppetlabs/puppetlabs-apt/blob/0871cadcdcbc5f0e6540298fa11e9a3ebe884735/README.md?plain=1#L70

Also note that there is a bug in APT that causes it to fail if ASCII armored keys use Windows/DOS text format. You'll want to dos2unix the file first, if you find such a key. Reference: https://salsa.debian.org/apt-team/apt/-/merge_requests/309

Gotcha sorry for the trouble and thanks for the quick answer!


Refresh direct back to cart

After every couple of refreshes, the page goes back to the cart and I have to checkout again.

Hi Maddy! Thanks for your message and sorry you're having trouble. Did you ensure that your checkout page is listed in the compatible section and not the incompatible section? That behavior sounds like your checkout page is incompatible with the current version of this script. If you need help figuring out if your delivery slot selection page is compatible, feel free to share a screenshot of it. You can upload the image to imgur.com or any other file sharing service and then post the link here. Looking forward to getting this resolved for you!


enable alertforward case for e2e in kind.

enable alertforward case for e2e in kind, alertforward case  was added in https://github.com/stolostron/multicluster-observability-operator/pull/819
Signed-off-by: morvencao<EMAIL_ADDRESS>

/test test-e2e


VS Code black screen VMWare Fusion win 8 vm

VSCode Version: 1.11.2
OS Version: windows 8.1 virtual machine running on VMWare Fusion

Steps to Reproduce:

Disable Accelerate 3D Graphics in the vm display setting before booting the vm. Boot and Login.
Install VSCode
Run VSCode -- screen is black.
4.- Close VSCode.
add --disable-gpu.
Start VSCode -- screen is black.

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: http://code.visualstudio.com/Download#insiders

Ok,, I followed your link and downloaded the preview release. Steps Performed:
1.- Uninstalled the installed version 1.11.2.
2.- Restarted the vm.
3.- Installed the preview release.
4.- Opened Visual Studio Code.
Observations:
1.-  I went to File-Open Folder. This time, the window to select the folder opened (this was not happening before).
3.- I selected the folder, the main screen was black.
4.- I went to File and selected close Folder, and Exit Visual Studio Code.
5.- I reopened Visual Studio Code, the screen flickered as in Observation 1.  But the interesting thing,
the folder I had selected and closed opened and the screen shows contents of Release Notes:1.11.2.
6.- I selected two files for comparison, all displayed fine. I closed folder, I exited Visual Studio Code.
7.- Re opened Visual Studio Code., the screen flickered as in Observation 1.
8.- I selected File Open Folder and chose a different folder location , I experienced step 3 above. screen is black.
9. This time the screen remained black, folder contents did not display. I exited.
10.- I added the --disable-gpu, and opened again, the screen remained black.

Just got into the same issue with the following setup:
. Host: Windows 10 Pro.
. Hypervisor: Hyper-V
. Guest: Windows 8.1 Pro
. vCPU on guest: 1
. VSCode running on guest will start with a black screen.
. vCPUs increased to 2
. VSCode running on guest will start properly.

Since this issue is rather old, I am closing this and ask anyone to open a new issue if this is still happening.


Binding key to 'close tab'

I would like to bind a keystroke to close a tab but that option seems to be missing.

Do you want keystroke binding in general or just 'close tab'?

Do you want keystroke binding in general or just 'close tab'?

I have included a commit in my pull request that adds this option.

I have included a commit in my pull request that adds this option.

I don't need anything special. If I can make cmd-d close a tab, I'll be super happy. :)

I don't need anything special. If I can make cmd-d close a tab, I'll be super happy. :)

I don't need anything special. If I can make cmd-d close a tab, I'll be super happy. :)

You'll first need to unbind Cmd-D from "Add bookmark", but this is now possible in the dev branch.

I don't need anything special. If I can make cmd-d close a tab, I'll be super happy. :)

You'll first need to unbind Cmd-D from "Add bookmark", but this is now possible in the dev branch.


FX-1811: Updates styling on the filter button for collections

This PR just updates the styles on the filter button (and makes it show up a little later).
Gif

Screenshot

(Side note: the fact that these RN PRs now go to eigen makes me feel like a real mobile developer 😻)

Fails

:no_entry_sign:

🃏 FAIL in Collection-tests.tsx

renders a snapshot

Snapshot name: `Collection renders a snapshot 1`- Snapshot+ Received@@ -1206,19 +1206,19 @@ scrollEventThrottle={50} stickyHeaderIndices={Array []} updateCellsBatchingPeriod={50} viewabilityConfig={ Object {- "viewAreaCoveragePercentThreshold": 30,+ "viewAreaCoveragePercentThreshold": 75, } } viewabilityConfigCallbackPairs={ Array [ Object { "onViewableItemsChanged": [Function], "viewabilityConfig": Object {- "viewAreaCoveragePercentThreshold": 30,+ "viewAreaCoveragePercentThreshold": 75, }, }, ] } windowSize={21}

Full message

Error: expect(received).toMatchSnapshot()
Snapshot name: Collection renders a snapshot 1

Snapshot

Received

@@ -1206,19 +1206,19 @@
scrollEventThrottle={50}
stickyHeaderIndices={Array []}
updateCellsBatchingPeriod={50}
viewabilityConfig={
Object {

    "viewAreaCoveragePercentThreshold": 30,

    "viewAreaCoveragePercentThreshold": 75,
  }
}
viewabilityConfigCallbackPairs={
  Array [
    Object {
      "onViewableItemsChanged": [Function],
      "viewabilityConfig": Object {

        "viewAreaCoveragePercentThreshold": 30,

        "viewAreaCoveragePercentThreshold": 75,
      },
    },
  ]
}
windowSize={21}

at Object.toMatchSnapshot (/Users/travis/build/artsy/eigen/emission/src/lib/Scenes/Collection/tests/Collection-tests.tsx:46:31)
at Object.asyncJestTest (/Users/travis/build/artsy/eigen/emission/node_modules/jest-jasmine2/build/jasmineAsyncInstall.js:102:37)
at /Users/travis/build/artsy/eigen/emission/node_modules/jest-jasmine2/build/queueRunner.js:43:12
at new Promise ()
at mapper (/Users/travis/build/artsy/eigen/emission/node_modules/jest-jasmine2/build/queueRunner.js:26:19)
at /Users/travis/build/artsy/eigen/emission/node_modules/jest-jasmine2/build/queueRunner.js:73:41

  Generated by :no_entry_sign: dangerJS

Looks good!


Examples: Learning to rank - AssertionError

Python 3.8.5

Examples: Learning to rank from URL https://github.com/microsoft/FLAML generates
AssertionError: # rows in X_train must match length of y_train.
See attached

Thanks. Fixing it:
from sklearn.datasets import fetch_openml
from flaml import AutoML
X_train, y_train = fetch_openml(name="credit-g", return_X_y=True, as_frame=False)
y_train = y_train.cat.codes
# not a real learning to rank dataaset
groups = [200] * 4 + [100] * 2    # group counts
automl = AutoML()
automl.fit(
    X_train, y_train, groups=groups,
    task='rank', time_budget=10,    # in seconds
)


Go to definition shortcut F12 doesn't work on Ubuntu 20.04

VS Code Version:
Version: 1.53.2
Commit: 622cb03f7e070a9670c94bae1a45d78d7181fbd4
Date: 2021-02-11T11:48:44.518Z
Electron: 11.2.1
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.8.0-44-generic snap
OS Version:
Ubuntu 20.04
Steps to Reproduce:

Install fresh new Ubuntu 20.04
Install latest  stable VSCode editor
Add (erlang 0.6.9) Erlang language extension for Visual Studio Code extension (all extension settings are default).
Install erlang language.
Open any erlang project, place a cursor over some function call and press F12 key.

Expected behaviour:
jump to the function definition.
Currently observed behaviour:
jump to the end of the line where the cursor was placed before being pressed.
I checked that in VSCode Settings > Keyboard Shortcuts the Go to Definition is set to F12 and condition is editorHasDefinitionProvider && editorTextFocus && !isInEmbeddedEditor Default
Ctrl+(left mouse click) combo works the way F12 should (jumps to the function definition)
Ctrl+Shift+P and typing "Go to Definition" with selection of drop down option works as expected as well (the shortcut for the command in the list is displayed as F12).
In the upper menu under "Go" option the suboption "Go to Definition" is colored with more light grey (looks as disabled). The same is true for "Go to Declaration", "Go to Type Definition", "Go to Implementation". (But these list options look the same on my Windows 10 (64) machine and on that machine everything works fine).
The same installation (latest stable VSCode editor (I checked that is has the same version and commit hash), same erlang 0.6.9 extension, same extension settings) works as expected on my Windows 10 (64) machine. I can browse to the function definition by pressing F12 and I can get back by pressing Alt + leftArrow.
I also tested the F12 (Go to Definition) functionality on WSL2(Windows Subsystem Linux) with Ubuntu 20.04 Image with same VSCode settings and it works as expected.
I need the same to be working on my Ubuntu machine (Dell latitude 541, core I7). And it seems I can't get it to work.

Does this issue occur when all extensions are disabled?: Yes/No

Open any erlang project, place a cursor over some function call and press F12 key.

Instead, please right click and see of there is a  "Go to Definition" entry in the context menu

Open any erlang project, place a cursor over some function call and press F12 key.

Instead, please right click and see of there is a "Go to Definition" entry in the context menu

Yes, the right click on the function call opens a context menu with "Go to Definition" option and when I select this option it leads to the function definition. The problem is only that the shortcut F12 doesn't work.

Ok, please do this:

F1 > Set Log Level > Tace
F1 > Show Logs > Window
Press F12 on that symbol you are trying with
In the log output search for messages like this: [2021-03-05 16:50:45.876] [renderer2] [trace] KeybindingService#dispatch

I went through all the steps and all I got was this:
[2021-03-05 17:56:52.997] [renderer1] [trace] KeybindingService#dispatch End cursorEnd
[2021-03-05 17:56:52.997] [renderer1] [trace] CommandService#executeCommand cursorEnd

Let's first try to find out what command executes on your machine, with your settings and your keybindings when pressing F12. Could you please try the following:

Run the command Developer: Toggle Keyboard Shortcuts Troubleshooting
This will activate logging of dispatched keyboard shortcuts and will open an output panel with the corresponding log file.
Press exactly the keybinding you desire (F12)
check what VS Code detects and what command is invoked.

If you are having trouble understanding the output, please feel free to paste here and I can try to help.

Hey, Thanks, guys! I've got the problem - on my new laptop the  key is at the same time the  key, which is not the case for my windows machine. So I had to delete the default keymapping for the "end" and edit the keymapping for "Go to Definition", pressing F12 (which displayed the "end" in the popup's text field). And now I can't go to the end of the line by pressing the key but I can go to the function definition.

@IgorKoshnarov This does not appear to be something under the control of VS Code. It looks like your keyboard has a key labeled physically as "f12", but the key sends the "end" key to the OS. I suggest that you contact the manufacturer, as this appears to be a system wide issue which is not specific to vscode.


bulk sync to node  failed

What does <some-string> defines? I was never able to find a node with id equal to <some-string>. How to map <some-string> to an actual node in my cluster?
Example log from journalctl
Feb 12 13:19:47 my-srv01 dockerd[2615989]: time="2024-02-12T13:19:47.878730800Z" level=warning msg="bulk sync to node 993b5ed1c755 failed: failed to send a TCP message during bulk sync

docker info
Client:
 Context:    default
 Debug Mode: false
 Plugins:
  buildx: Docker Buildx (Docker Inc.)
    Version:  v0.10.2
    Path:     /usr/libexec/docker/cli-plugins/docker-buildx
  compose: Docker Compose (Docker Inc.)
    Version:  v2.16.0
    Path:     /usr/libexec/docker/cli-plugins/docker-compose
  scan: Docker Scan (Docker Inc.)
    Version:  v0.23.0
    Path:     /usr/libexec/docker/cli-plugins/docker-scan

Server:
 Containers: 10
  Running: 7
  Paused: 0
  Stopped: 3
 Images: 9
 Server Version: 20.10.14
 Storage Driver: overlay2
  Backing Filesystem: xfs
  Supports d_type: true
  Native Overlay Diff: true
  userxattr: false
 Logging Driver: gelf
 Cgroup Driver: cgroupfs
 Cgroup Version: 1
 Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
 Swarm: active
  NodeID: gn2rsb5a806r0u3t33b1qei2f
  Is Manager: false
  Node Address: <IP_ADDRESS>
  Manager Addresses:
   <IP_ADDRESS>:2377
 Runtimes: nvidia runc io.containerd.runc.v2 io.containerd.runtime.v1.linux
 Default Runtime: nvidia
 Init Binary: docker-init
 containerd version: 2456e983eb9e37e47538f59ea18f2043c9a73640
 runc version: v1.1.4-0-g5fd4c4d
 init version: de40ad0
 Security Options:
  apparmor
  seccomp
   Profile: default
 Kernel Version: 5.15.0-1030-aws
 Operating System: Ubuntu 20.04.2 LTS
 OSType: linux
 Architecture: x86_64
 CPUs: 16
 Total Memory: 62.02GiB
 Name: ip-10-0-3-105
 ID: I5Q6:MIY6:MCQD:4FXN:OBSB:GMUQ:E5ZO:A7WX:VSJK:B7EW:VA4L:33GZ
 Docker Root Dir: /mnt/docker/data
 Debug Mode: true
  File Descriptors: 137
  Goroutines: 195
  System Time: 2024-02-15T13:32:30.121115095Z
  EventsListeners: 3
 Registry: https://index.docker.io/v1/
 Experimental: true
 Insecure Registries:
  <IP_ADDRESS>/8
 Live Restore Enabled: false
 Default Address Pools:
   Base: <IP_ADDRESS>/12, Size: 20
   Base: <IP_ADDRESS>/16, Size: 24

https://github.com/moby/moby/discussions/47372

This log is probably generated by this line: https://github.com/moby/libnetwork/blob/3797618f9a38372e8107d8c06f6ae199e1133ae8/networkdb/cluster.go#L706

This answer explain how to map <some-string> to a particular swarm cluster Node. It would be nice to have it documented somewhere.
https://github.com/moby/moby/discussions/47372#discussioncomment-8539909


Migrate goodtables.io

See https://github.com/okfn/devops/issues/308
(creating an issue here for planning purposes only - use the other issue for discussion)

We also need a note on the old website

I'm cloding as DONE as we already have a devops issue for the following steps - https://github.com/okfn/devops/issues/310


External Loader requires

Hey! I have just started to try out to load loaders externally. This way big projects with a lot of loaders get's handed in a much more clean way. Here's what I think it should look like.
Screenshots from my Twitter

Also, the mapping function is wrong in every way, since it was a quick screenshot, but I liked the idea of having loaders in it's own env.


Merges release-3-0-0 into release-3-1-0

to: @ryandeivert  @jack1902
cc: @airbnb/streamalert-maintainers
Merges release-3-0-0 intorelease-3-1-0, bringing in all changes.
A rebase is cleaner, but will not work without a force push. This is the easiest way of doing it

Coverage decreased (-0.02%) to 96.178% when pulling 35980ca39da6a1145b91efdb3ef1b8ee7a4b3b97 on dw--merge-300-into-310 into e500e1d771e89ee005c302e2392eae40879da535 on release-3-1-0.


override create_or_update to include class-specific fields when using get_id to find existing objects

Did a once over of the zabbixapi classes and added create_or_update overrides for the ones that need additional fields for get_id queries against the API.
This is needed to support things like templated items, which are copied to all hosts linked to the template. The current create_or_update calls may pull one of the item copies instead because they are not specific enough with get_id.
Resolves #57

Hold on merging this, just found a set of broken tests when doing some regression testing - https://travis-ci.org/jrbeilke/zabbixapi/builds/184280573

Had to also add overrides for get_or_create() like I did with create_or_update() and now have a successful build - https://travis-ci.org/jrbeilke/zabbixapi/builds/184359876


Something is broken

I used truffle framework and it created migrations.sol
Contract is deploed successfully but Solium output:
✖ [Fatal error] An error occurred while linting over <path>\contracts\Migrations.sol: Line number 23 invalid or out of bounds.

@Gargo Could you produce your contract code here (even if it was the migrations.sol that caused the error)?

closing this, please re-open if you're facing the issue again with the code.


阿里云盘下载链接在携带referer情况下无法下载

Related https://github.com/ScoopInstaller/Scoop/issues/5280 https://github.com/ScoopInstaller/Scoop/discussions/5140
https://github.com/cscnk52/cetacea/blob/42f240e8fc063b6a60cff3c1659017089fd6477c/bucket/aliyundrive.json#L9

can be fixed by https://github.com/chawyehsu/dorado/commit/9ab4497813599d6ae2b06237f1c8da51b94a66a0
install via scoop install dorado/aliyundrive


fixed regex for backslash replacement to replace all backslashes

On windows I just had the error that a path looking like this:
someFolder\%someParam\index
got transformed to this
someFolder/%someParam\index
so only the first backslash got replaced by a normal slash.
The fix is to add a g flag to the replacement regex so that all backslashes match

Coverage remained the same at 100.0% when pulling 77e0ab0f8c95ce849a8bd35f78471a40ede9ca07 on kaoDev:fix/backslah-replace into ecc3bddf8ec89c717be95f0aa2c809d2ecaea38b on jesseditson:master.

Nice catch!

Released here: https://github.com/jesseditson/fs-router/releases/tag/v0.4.1


Azure DevOps git repos supported?

are Azure DevOps git repos supported?

Thank you for your question!
Keptn allows any derivative of Git.
May we ask you to test it and extend our documentation for Azure DevOps Git, i.e. make a PR?
https://keptn.sh/docs/0.6.0/manage/project/#select-git-based-upstream
https://github.com/keptn/keptn.github.io/blob/master/content/docs/0.6.0/manage/project/index.md
In the future, we ask you to put your questions into our Slack channel:
https://join.slack.com/t/keptn/shared_invite/enQtNTUxMTQ1MzgzMzUxLWMzNmM1NDc4MmE0MmQ0MDgwYzMzMDc4NjM5ODk0ZmFjNTE2YzlkMGE4NGU5MWUxODY1NTBjNjNmNmI1NWQ1NGY


bug: cannot run in the first start up of neovim

Steps to reproduce:

Open a project for the first time
Run build_workspace command to build
Run the main application

Expected:
User should be able to debug the application
Actual:
Application is not ran nor can be debug
Temporary solution:
Close and reopen neovim after the build_workspace


[WIP] Graph entropy algorithm

New implementation of the graph entropy clustering algorithm as described in E. C. Kenley and Y. Cho "Entropy-Based Graph Clustering: Application to Biological and Social Networks"

Codecov Report

Merging #1 (b40714e) into master (8fcf3ea) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #1   +/-   ##
=======================================
  Coverage   94.87%   94.87%
=======================================
  Files           6        6
  Lines         371      371
=======================================
  Hits          352      352
  Misses         19       19

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 8fcf3ea...b40714e. Read the comment docs.


libc is outdated

The libc dependency that magic-sys uses is very outdated and has invalid definitions. This makes it incredibly difficult to use magic-sys on ARM systems. See this build log to see an example (i8 vs u8).

I've updated the libc dependency to current latest version 0.2.13 and published version 0.2.0 of magic-sys, please give it a try.

Thank you very much!


Cimms dashboard

PR Details
Description
Adding CIMMs dashboard pages, including:

Assessments
Notifications
Data Selector
Reports
Configuration Parameters

How Has This Been Tested?
This was tested using the docker-compose.yml, which brings up many components, including the conflictmonitor, kafka, and mongodb. Please reference the updated sample.env for the required environment variables for these components.
These changes can be tested by running the docker-compose, and then tabbing over to the Intersection Dashboard page. After injecting sample data through the test-message-sender, you should see notifications, assessments, data in the data selector, and configuration parameters.
Types of changes

[ ] Defect fix (non-breaking change that fixes an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that cause existing functionality to change)

Checklist:

[ ] My changes require new environment variables:

[ ] I have updated the docker-compose, K8s YAML, and all dependent deployment configuration files.

[x] My changes require updates to the documentation:

[ ] I have updated the documentation accordingly.

[ ] My changes require updates and/or additions to the unit tests:

[ ] I have modified/added tests to cover my changes.

[x] All existing tests pass.

@drewjj @payneBrandon I am leaving this as a draft PR for now, I am having trouble with my DocumentDB data initialization. All other features are working, you would just have to run all of the conflictmonitor components to test right now. I will fix this up after I get back!
All of the page, model, feature, and component additions are good to review, except for the decoder page, which is still under development in CIMMS (and is not added to the intersections/dashboard page in this PR)


CXF-99867: Integrate Alibaba Terraform Provider with Fabric Connection Examples

Finalized steps page -

The screenshots should be in the same comment, along with the context of why they are being shared.


Make sure to convert tag strings to lower case before comparison

Related to https://github.com/opencollective/opencollective/issues/5702
Related Front-End PR: https://github.com/opencollective/opencollective-frontend/pull/8052
https://user-images.githubusercontent.com/12435965/184177680-7f2820ef-3644-4257-9436-9499af3ce9bb.mp4

This should not be required anymore after https://github.com/opencollective/opencollective-api/pull/8462

Closing this as https://github.com/opencollective/opencollective-api/pull/8462 is now merged, that will make sure all tags are lowercased in the database.


compliance operator scans result doesn't contain results for CIS control 4.2 Kubelet

I am using compliance operator to scan OpenShift cluster. There is a list of properties which are present in some of the cluster scan results while missing in other cluster scan results. Following is the list of missing controls:

1.2.8
1.2.10
1.2.14
1.2.15
1.2.16
1.2.17
1.2.19
1.3.6
1.4.2
4.2

Could anyone please help in understanding why some of the clusters would not be scanned for the above listed controls and how can I fix this?

Here is the evidence that the scan result of our cluster(refer the status corresponding to cluster mycluster-us-south-1-bx2.4x16 as this is the healthy cluster, do not refer to overall status as there are some other cluster which are not healthy or don't have compliance operator installed and affecting the overall status) contains following control results:


do not stop pipeline on not found resources

Pipeline was stopped on collections:

col26488
col12081

error message at col26488 on gdocify step:
+ exec
++ tee gdocified-book/stderr
+ cp -r disassembled-book/col26488 disassembled-book/stderr gdocified-book
++ cat book/collection_id
+ collection_id=col26488
+ book_dir=gdocified-book/col26488/disassembled
+ target_dir=gdocified-book/col26488/gdocified
+ book_slugs_file=gdocified-book/col26488/book-slugs.json
+ mkdir gdocified-book/col26488/gdocified
+ gdocify gdocified-book/col26488/disassembled gdocified-book/col26488/gdocified gdocified-book/col26488/book-slugs.json
Convert to RGB: ../resources/8a9dbdef9a6d7a62ea4da8fb4adb2f3f2d7566af
Convert to RGB: ../resources/2b721b0944b10dcfed1a88cb056a233b350838a9
Convert to RGB: ../resources/327c34078f002af621d6034ce19b6b1020691d97
Traceback (most recent call last):
  File "/usr/local/bin/gdocify", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.7/site-packages/bakery_scripts/gdocify_book.py", line 174, in main
    fix_jpeg_colorspace(doc, out_dir)
  File "/usr/local/lib/python3.7/site-packages/bakery_scripts/gdocify_book.py", line 142, in fix_jpeg_colorspace
    raise Exception('Error: Resource file not existing: ' + str(img_filename))
Exception: Error: Resource file not existing: /tmp/build/36dafd0d/gdocified-book/col26488/gdocified/CNX_Chem_20_01_alkanes.jpg

error message at col12081 on gdocify step:
+ exec
++ tee gdocified-book/stderr
+ cp -r disassembled-book/col12081 disassembled-book/stderr gdocified-book
++ cat book/collection_id
+ collection_id=col12081
+ book_dir=gdocified-book/col12081/disassembled
+ target_dir=gdocified-book/col12081/gdocified
+ book_slugs_file=gdocified-book/col12081/book-slugs.json
+ mkdir gdocified-book/col12081/gdocified
+ gdocify gdocified-book/col12081/disassembled gdocified-book/col12081/gdocified gdocified-book/col12081/book-slugs.json
Convert to RGB: ../resources/832b66982e7fe1b3aa3adbb1d44e90dd9a29a5c5
Traceback (most recent call last):
  File "/usr/local/bin/gdocify", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.7/site-packages/bakery_scripts/gdocify_book.py", line 174, in main
    fix_jpeg_colorspace(doc, out_dir)
  File "/usr/local/lib/python3.7/site-packages/bakery_scripts/gdocify_book.py", line 142, in fix_jpeg_colorspace
    raise Exception('Error: Resource file not existing: ' + str(img_filename))
Exception: Error: Resource file not existing: /tmp/build/36dafd0d/gdocified-book/col12081/gdocified/openstax.org/l/28collisionlab

@rnathuji I will close this issue and I've implemented your suggested fixes in https://github.com/openstax/output-producer-service/pull/268


[New Port Request] Compact Encoding Detection

Library name: CED
Library description: Compact Encoding Detection(CED for short) is a library written in C++ that scans given raw bytes and detect the most likely text encoding.
Source repository URL: https://github.com/google/compact_enc_det
Project homepage (if different from the source repository):
Anything else that is useful to know when adding (such as optional features the library may have that should be included):

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


Dockerfile

I was trying to create a docker file where I could run ink-wrapper. For reasons unknown to me, ink-wrapper produces wrong output. I am attaching sample.zip where you will find everything to reproduce the issue.

Thanks for the report, I'll look into it!

The problem seems to be with the version of rust you're using to install ink-wrapper. I believe this is related to an issue in a dependency of the library we use for code generation and the API it depends on, which is now being changed in nightly rust (https://github.com/udoprog/genco/issues/39). For now, I'd recommend installing with:
cargo +nightly-2023-04-16 install ink-wrapper --locked --version 0.5.0

to basically lock the toolchain you're using to compile ink-wrapper to one that's known to work (I checked, and this one indeed works). I'll add a note about this to the README, and in the future, I hope the relevant changes will just be released into rust proper, so we can set min rust version and it will just work automatically or fail with an appropriate message.

Installing ink-wrapper with fixed version nightly-2023-04-16, solved the issue. Thanks!


Allow setting certificate-authority-data with service account approach of configuration

We've added this capability in #3. This was supposed to be have been supported in this action from the get go, but there was an error in code and documentation. The k8s-secret input is expected to contain the contents of the secret associated with the service account of concern. This is then used to parse both the token and the cert...
For fetching Secret object required to connect and authenticate with the cluster, the following sequence of commands need to be run:
kubectl get serviceAccounts <service-account-name> -n <namespace> -o 'jsonpath={.secrets[*].name}'

kubectl get secret <service-account-secret-name> -n <namespace> -o json

The contents from the output of above command can be added as secret to the repository and later used in this action as shown below:
- uses: azure/k8s-set-context@v1
  with:
    method: service-account
    k8s-url: <URL of the clsuter's API server>
    k8s-secret: <secret associated with the service account>
  id: setcontext


Make InformationBuilder adhere the API guidelines.

Rust API guidelines state that methods of builders should return self to allow chaining.
Add missing documentation strings to builder functions.

Please add a changelog entry
You can create a new
## Unreleased
- foo

section in the changelog.

I added the respective changelog entry. Because the changes made it necessary to modify the tests, I think this qualifies as a breaking change.

Please fix the integration test

Thanks for your contribution!


Add Kyverno rules

Towards: https://github.com/giantswarm/giantswarm/issues/27333
Still a WIP, I am working on adding the opscrecipes

@Strigix , your rule seems to have a syntax error when trying to evaluate the tests. It seems that && is not admitted here.

Closes: https://github.com/giantswarm/giantswarm/issues/27834


add propdeps-plugin support for uploadArchives task.

Currently propdeps-plugin only support maven install task.
I added uploadArchives support.

The propdeps plugin has moved to the Spring Attic and is no longer maintained.


variables run as commands (first word)

I'm not sure how resholved should handle variables used as a command (i.e., the first word of a statement). I'm already aware of a number of patterns here and have found real-world examples in common programs where not handling these is "wrong", but I could use some feedback on how to handle it (and perhaps how often people run into it).
A few such patterns include:

Storing a frequently-used command in a variable name to make it easier to change or override and replacing uses of the command with the variable.
When a script has to do a little work to figure out what the right command is (i.e., it changes depending on user config, command-line options, platform, software versions, sanity checks of command behavior/options/output).
Running positional parameters as commands. This is done for multiple distinct reasons, like wrapping an execution with other logic, exposing script functions for external execution, etc. (I think most of these will be okay-ish, so they're currently exempted from the warning. For them to be robustly okay, resholved would probably have to figure out how and where they're being invoked, and then try to resolve the appropriate positional arg of each invocation.)

This pattern, at least with named variables, will now print a warning/request for feedback, such as:
  $GIT_PROGRAM status
  ^~~~~~~~~~~~
[ stdinNone ]:7: FEEDBACK WANTED: Letting dynamic command (first-word variable) through for now. Not sure if this is 'right' or not. ...

As the fact that resholved issues a warning here suggests, it isn't terribly hard to spot this behavior. Fixing it is less straightforward. Continuing the $GIT_PROGRAM example, resholved would either need to replace instances of the variable with a resolved program, or it needs to figure out where the variable is set and resolve whatever it's set to (but only if it's not something like a builtin/function/alias?)
Trying to do this automatically probably entails a jump in complexity and greater risk of breaking a resolved script in some way.
It is easy to throw an error here and just make/let the user override it, though I'm not sure how the override API should approach it...

It could enable the user to specify a replacement for the variable, but it seems tricky to do this robustly (do we just replace instances used as a command? also instances used inline? does the script build up the variable iteratively in multiple steps to add flags and options?)
The API could specify a target variable, and a string/identifier to try and resolve wherever it's defined. This might be a little more robust against the script iteratively constructing the command, but I can imagine it breaking down in other ways (the script uses the variable for flow control, or performs string operations like substitution or positional indexing/slicing).

I've made up my mind that this "should" be an error (because I've found a scenario where I personally missed identifying a dependency that I would have caught if this step had been an error that I had to fix instead of a warning.)
I'm not quite sure what the fix is yet...

This is a fairly common warning, so if it's not a silly amount of work, it would be nice to directly solve any straightforward cases.
If there's a reliable heuristic for splitting simple variables that are only set and read (I am doubtful), it could be productive to let the user specify imperative replacements/overwrites for this set of variables and save harder triage for steps for variables getting used in more complex ways.

Just wanted to add an example of when I saw it, and what I'm trying to do:
  newversion="$($newsyspath/sw/bin/nixos-version)"
                ^~~~~~~~~~~
[ stdinNone ]:26: FEEDBACK WANTED: Letting dynamic command (first-word variable) through for now. Not sure if this is 'right' or not. Weigh in @ https://github.com/abathur/resholved/issues/3

Here is my script:
#!/bin/sh

newsyspath=$(readlink -f "/var/cache/upgrade/staged")
prevsyspath=$(readlink -f /nix/var/nix/profiles/system)

if [ ! -e "$newsyspath" ]; then
	echo "No new system."
	exit 0
fi

if [ "$newsyspath" = "$prevsyspath" ]; then
	echo "System remains unchanged."
	exit 0
fi

prevversion="$(/run/current-system/sw/bin/nixos-version)"
newversion="$($newsyspath/sw/bin/nixos-version)"

echo "Upgrade from $prevversion to $newversion?"

Ooof. Thanks for adding this.

This is partially closed by the --keep $varname directive. I say partially because I think a more principled solution may be available eventually, but it'll entail paying more attention to variables than resholve currently does (and, by proxy, to all of the variable-setting commands, etc.)


Command + M (Minimize window) is overwritten by Continue on Mac

Describe the bug
Command + M is a system-wide hotkey to minimize the window of the current application (ie. the VSCode editor window). With the Continue extension installed, Command + M opens the extension's window within the editor instead.
I think it would be preferable for Continue to use a different hotkey, at least on mac.

I've been considering this as well, want to make a decisive and final decision at some point. Other possibilities are command k and command l, both ergonomic and not overly commonly used. Any other thoughts? If you were to (or already have) change the key binding what would you change it to?

Command + K is the "shortcut for more shortcuts" (aka "chords"). Look at the status bar when you press Command + K). Command + I triggers autocomplete in Python for me - I actually didn't know about this shortcut.
Two suggestions:

Control + M, which doesn't seem used (and would mean - I assume - Continue has the same shortcut across OSes)
Command + K, Command + M, which would then use the VSCode chords.

Not sure if this warrants a separate issue, but on the more general subject of keyboard shortcuts, the extension also seems to override some VSCode system shortcuts that I feel are pretty commonly used:

⇧+⌘+L - Select All Occurrences of Find Match / Search Editor: Select All Matches
⇧+⌘+M - View: Toggle Problems
⇧+⌘+Enter - Insert Line Above
⇧+⌘+R - Go to Symbol in Workspace / Search Editor: Search Again

For context, none of the other 80+ extensions I have installed overrides these shortcuts, so it would be nice if Continue didn't either. For now I've had to disable them to keep the system setting, but ideally that could be solved by using less general/less common hotkeys, using chords as suggested above, or maybe even scoping the shortcuts to when the Continue panel is active.

Continue is now using cmd+shift+L, cmd+L, and cmd+I. It won't be possible for us to select a shortcut that is both convenient and never used for other purposes, since there are so few keys to choose from, but we choose a convenient shortcut with the understanding that folks who are frequently using less common shortcuts are more likely to know how to change the keybindings for Continue. A note has also been added to our tutorial to make it clear that this is possible.
I'm going to close this issue as it's title doesn't make it the best candidate for discourse around keyboard shortcut choices (now that our defaults are cmd+L), but if this continues to be a concern please feel free to keep the conversation going elsewhere!


Deployment of signed images fails with Error: /app/connaisseur-config/v3g-beamman-val/.docker/config.json: illegal base64 data at input byte 72

Describe the bug
The auth secret_name in connaisseur values.yaml is configured to be the same secret used to pull the images, which works when connaisseur is not installed. Also, this secret was accepted by Kubernetes apiserver, implying it is properly formatted.
The format of this secret (obtained by getting the secret using kubectl and then base64 decoding the .dockerconfigjson value) is:
{
    "auths": {
        "registry.docker.artifactory.sativa.com": {
            "auth": "xxxxx",
            "username": "user"
        }
    }
}

The full error is:
Error from server: error when creating "bandslotmgr.yaml": admission webhook "connaisseur-svc.connaisseur.svc" denied the request: Unexpected Cosign exception for image "registry.docker.artifactory.sativa.com/repo/app:tag": Error: /app/connaisseur-config/validator/.docker/config.json: illegal base64 data at input byte 72
main.go:46: error during command execution: /app/connaisseur-config/validator/.docker/config.json: illegal base64 data at input byte 72

Expected behavior
Expected the image to be pulled and signature verified to be correct.
Optional: To reproduce

Optional: Versions (please complete the following information as relevant):

OS: Ubuntu 18.04
Kubernetes Cluster: GKE kubernetes v1.21
Notary Server:  N/A
Container registry: JFrog artifactory Enterprise Plus 7.19.12
Connaisseur: securesystemsengineering/connaisseur:v2.4.1
Other:

Optional: Additional context

@QWERTY92009 the error is surprising as it reports malformed lbase64. Have you reviewed how the base64 form looks like at byte 72? unfortunately it contains a secret such that it cannot be shared here for inspection.

has the same secret worked for you when using the dockerconfigjson directly with cosign on your local machine?
could you exec into the connaisseur pod and check the config.json at path /app/connaisseur-config/validator/.docker/config.json?

Otherwise, as Connaisseur reports this is an error originating from cosign. If this turns out to be a legit bug in reading a valid configjson, it would have to be reported there: https://github.com/sigstore/cosign . But I am not yet fully convinced of that.
Let me know if that helps!

Thanks @xopham. I'm not thoroughly convinced yet.
However:

I am able to add this secret to kubernetes without any complaint.
I'm able to extract the secret from kubernetes and base64 decode it as indicated above, to obtain the original creds.

I will definitely follow your troubleshooting tips and get back to you!

@xopham. When I exec in I see /app/connaisseur-config, containing
-rw-r--r--    1 0        0                2 Feb 21 18:44 config-secrets.yaml
-rw-r--r--    1 0        0             1116 Feb 21 18:44 config.yaml
drwxr-xr-x    3 0        0             4096 Feb 21 18:44 v3g-beamman-val
drwxr-xr-x    3 0        0             4096 Feb 21 18:44 v3g-phyman-val

Note the zero size of the yamls. Within v3g-beamman-val, there are no contents. (BTW, the name of the validator is correct.)
Apparently an error in installation? I didn't see any warnings, but maybe there is a log somewhere? Thanks!

@QWERTY92009 did you do an ls -la inside v3g-beaman-val because the path in your case would be /app/connaisseur-config/v3g-beaman-val/.docker/config.json and ls will not reveal hidden folders/files such as .docker. This had to be done as cosign only takes the dockerconfigjson from $HOME/.docker/config.json and the auth creds cannot be passed as input.
In your case, the following should work cat /app/connaisseur-config/v3g-beamman-val/.docker/config.json.
Does that work? If that does not yield any output than there would have to be something wrong with configuration or with the installation.
If the above won't help, I could try to reproduce the error. Would you be able to share a minimal example of the bug consisting of:

pseudonymized dockerconfigjson (auth secret replaced with non-secret value of same length and form)
a pseudonymized values.yaml
pseudonymized command you use to apply your deployment

Note that resources referenced there in would have to exist.

@xopham, you are correct of course. Using ls -la does reveal the file:
/app/connaisseur-config/v3g-beamman-val/.docker/config.json
The contents are exactly the contents of the config.json originally used, with the same form shown above.

@QWERTY92009 I was just checking if anything like this had been observed for cosign. Turns out cosign has a very unintuitive handling of a file being unavailable while this specific case has been fixed, I'd presume it may still be present in other configurations.
So, I rechecked the information you provided and noticed the following:

the error you are reporting points to /app/connaisseur-config/validator/.docker/config.json
the config.json you report above is located at /app/connaisseur-config/v3g-beamman-val/.docker/config.json

Was this two different configurations and typically error message and location of config.json are in accordance? Otherwise, this might indicate a misconfiguration in helm/values.yaml or a bug in authentication handling.
If possible, could you share your helm/values.yaml or a redacted version?

Thanks @xopham. The discrepancy was due to a clumsy redaction error.
Sharing values.yaml is an excellent idea! Please take a look and let me know what you think. Thanks!
(Note: added .txt extension due to github not liking the .yaml extension which seems strange.)
values-redacted.yaml.txt

@QWERTY92009 the helm/values.yaml looks unsuspicious. However, going back to the form of the secret that you provided above:
{
    "auths": {
        "registry.docker.artifactory.sativa.com": {
            "auth": "xxxxx",
            "username": "user"
        }
    }
}

I am very surprised of its form. I understand that it seems to be functional for k8s, but fails for the use case here, as cosign claims "illegal base64". I can only see 2 potential reasons rooted in cosign parsing the config.json different to kubernetes:

either the auth: has some broken base64 value as described here
or cosign does not do well with the username that is not base64 encoded to begin with

I would propose to copy that config.json to your local machine, make a backup of your ~/.docker/config.json and copy the config.json provided here to ~/.docker/config.json, download the cosign binary and attempt image verification to see whether it fails with the same issue and if so, see if that changes when removing 'username'...

Thanks @xopham. The auth json is created by docker, and accepted by k8s, but I think you are suggesting more closely examing its structure in case there is another incompatibility. I'll look into this. This is only a PAT, so I can send you the original and reset the PAT on my side if you are amenable to that.


connexion with odbc

using pyodbc or similar
principle:

using sqlite_bro GUI over an odbc source (or sqlalchemy ?) :
. connecting to "odbc" source, a source starting per odbc would be "odbc"
using sqlite_bro to import from odbc to sqlite
using sqlite_bro to  export from sqlite to odbc.

interest:

gui on anything in 1 file (if an odbc package)
keep it simple

limit:

not designed for performance
not standard
could be complex

example of how it could be
https://docs.microsoft.com/en-us/sql/connect/python/pyodbc/step-3-proof-of-concept-connecting-to-sql-using-pyodbc?view=sql-server-ver15

over here;: list of tables
crsr = cnxn.cursor()
table_names = [x[2] for x in crsr.tables(tableType='TABLE')]

bit of pyodbc usage https://stackoverflow.com/questions/41430458/load-csv-to-mdb-using-pyodbc-and-pandas

excel  https://www.red-gate.com/simple-talk/sql/database-administration/getting-data-between-excel-and-sql-server-using-odbc/
"Driver={Microsoft Excel Driver (*.xls, *.xlsx, *.xlsm, *.xlsb)};DBQ=MyPath/MyFile"

import pyodbc
access_string= "Driver={Microsoft Excel Driver (*.xls, *.xlsx, *.xlsm, *.xlsb)};DBQ=C:/tempo/test.xlsx"
con = pyodbc.connect(access_string, autocommit=True)
cursor = con.cursor()
print([x[2] for x in cur.tables()])
cursor.execute("select * from [Feuil1$]")
print(list(cursor.fetchall()))

['Feuil1$']
[(2.0, 1.0, 3.0), (54.0, 45.0, 56.0)]

$Connection.ConnectionString = 'Driver={Microsoft Excel Driver (*.xls, *.xlsx, *.xlsm, *.xlsb)};DBQ='+$ExcelFilePath+'; Extended Properties="Mode=ReadWrite;ReadOnly=false; HDR=YES"'

apparently:
# columns in table x
for row in cursor.columns(table='x'):
    print(row.column_name)


New maintainer

As I'm not really even tangentially related to the LTTP Rando community anymore, and I'm sure there are many other people more suited to running this project. If we can find someone interested who already has the technical skills to deploy a Firebase web project and manage the GitHub repo, I would like to transfer ownership of both to someone who can be a more active maintainer.
If anyone watching this repo has a suggestion, please let me know.

I'll admit I have my own tracker, and I have messed with firebase before for a separate test project. If necessary, I am willing to be on a fallback list of new owners. I'd rather prefer if someone else claims it first.

Although I'm also less involved in ALttP Rando (curse you OoT Rando), I'd be open to take care of this repository if you cannot find someone better suited.

I'm quite interested myself. I'm currently working on a voice activation addon for this tracker and intend on becoming VERY familiar with the project anyway


🌐 We need your help on translation.

I try to help as many developers as we can around the world.
Could we please get some help with the translation above? We are limited with members to support translation and using google translate sometimes is not that accurate as well.
small PR and improvements are welcome.

We can help developers by translating (I know Romanian for example), but that's giving someone a fish instead of teaching them to fish.
Learning English is the best investment a developer can make.

I will do some work on chinese version.

I'd like to help you with translation into Persian but I'm afraid if there's any support for RTL languages on the website or not. :(

I'd like to help you with translation into Persian but I'm afraid if there's any support for RTL languages on the website or not. :(

Thanks a lot @majid-amiri I don't think we have Persian set up just yet, I am hesitant to introduce more language because it slowly becoming a burden unless someone is dedicated to contributing. Thanks for very much considering to help tho.

I was actually thinking of refactoring the api code (to start) in such a manner that i18n is not too much of a burden. Ideally we could easily add multiple languages, and if there were no contributions for a specific section it would default to english (or whatever language would be best). May have a PR in the near future, but thumbs up if you support the idea (or thumbs down if u have doubts + reply).

I was actually thinking of refactoring the api code (to start) in such a manner that i18n is not too much of a burden. Ideally we could easily add multiple languages, and if there were no contributions for a specific section it would default to english (or whatever language would be best). May have a PR in the near future, but thumbs up if you support the idea (or thumbs down if u have doubts + reply).

That would be dream come true. ❤

I was actually thinking of refactoring the api code (to start) in such a manner that i18n is not too much of a burden. Ideally we could easily add multiple languages, and if there were no contributions for a specific section it would default to english (or whatever language would be best). May have a PR in the near future, but thumbs up if you support the idea (or thumbs down if u have doubts + reply).

I think we can seprate data and current languange from components' concern. For exmaple wrap Link with LanguageContext so no need to translateLink. In this way, page components will become static ui component, and file directories may look like
data
|__ api.en.ts
|__ api.zh.ts
|__ advanced.en.ts
|__ ...
pages
|__ api.tsx
|__ advanced.tsx
@pmaier983, please let me know if you need help.

I'm gonna help with the portuguese translation

That's awesome! Thank you @zeyuri.

Hi, @bluebill1049  i'm kind of puzzle  about term constraint-based validation  api during translation . I thought constraint-based is repetitive before word validation. Do you want to express more declarative validation api or something else?

@cpprookie what about 基于约束验证?

@cpprookie what about 基于约束验证?

better than current version. I decide use 校验 for word validation  during translation by following ant-design usage.
But 基于约束 is a little strange and confusing in chinese(at least for me). Could you give a simple comparison with other libs  for further unstanding？

Sounds good to me @cpprookie 👍
I don't have specific example, but as far as I know, we are the first one introduced field-level validation which aligned with HTML standard.

I get it! thanks for your patient explanation @bluebill1049 .


Adding OpenSSL support to FFMPEG, fixed Assmimp UWP build

Resolving #2012 with a fix made by @khouzam

Thanks for the PR!


Add support for Creator/contributor JSON content

Currently the Bolognese GenericWorkReader uses the display for creator and contributor. This is preventing me from getting data out of the reader to use when exporting in the ORCID XML writer. The only writer that currently uses the generic work reader is the RISWriter, so that has been updated and the tests fixed.

Failing specs are CI issues, they all pass locally.


Some of .md file dosen't have proper hyperlinking

For example in https://github.com/shogun-toolbox/shogun/blob/develop/doc/readme/INTERFACES.md, proper hyperlinking to install.md is not there.

just send in a PR :) thnx!

I sent a PR in #4859 (https://github.com/shogun-toolbox/shogun/pull/4859/commits/b0c9a4889d4bbed9f0daa8bace62d55456827ddb). Please verify it.

just send in a PR :) thnx!

I sent a PR in #4859 . Please verify it.


feat: HomeHeader has been rebuilt

This pull request addresses issue #402.
What changed?

Create HomeHeader component
Create LocationSelect component
Create Subtitle component

Taking some cues from this article about using custom select menus, I opted to use a native select menu in LocationSelect for this work. The arrow icon is also now an SVG instead of a transformed <div> element. Open to feedback, however - I know that rebuilding that component has its own issue now in #432 so we can address it then.
How did you test the changes?

Manually

Screenshots

@arnest00, thanks for all the work you did building the Home Header! I'll address the remaining issues and get it merged.


Cleaned up travis file

Removed redundant timezone environment variable
Update docker container to reference host memory per selenium guidance

See https://github.com/SeleniumHQ/docker-selenium#running-the-images for more details

You could remove the env section too, or we could do it later. That was for sauce labs setup that we aren't using.


Unable to create a resized volume when using LVM as storage pool (terraform crashing)

System Information
Linux distribution
 CentOS 7
Terraform version
v0.11.13
Provider and libvirt versions
Built from master (dmacvicar/terraform-provider-libvirt@46b0ba055a47587a8c0b4b2c76f5e47165b7c6c4) with go get as I wasn't able to find a recent RPM on the OBS server.
Compiled against library: libvirt 4.5.0
Using library: libvirt 4.5.0
Running hypervisor: QEMU 1.5.3
Running against daemon: 4.5.0

Description of Issue/Question
Setup
Scenario: creating a resized LVM volume from a small image.
As image was in qcow2 I had to convert it to raw and store it in default storage pool.
I've got an LVM guest_images_hddvg storage pool.
I've been looking at https://github.com/dmacvicar/terraform-provider-libvirt/issues/358 reported by @mjiao.
provider "libvirt" {
  uri = "qemu:///system"
}

variable "domain_name" {
  type    = "string"
  default = "test_resize"
}

# Create a LV from the raw ubuntu image as creating a LVM from the remote qcow2 fails
# https://cloud-images.ubuntu.com/bionic/current/bionic-server-cloudimg-amd64.img
# qemu-img convert /var/lib/libvirt/images/image-ubuntu-qcow2 /var/lib/libvirt/images/image-ubuntu-raw
resource "libvirt_volume" "base_vol" {
  name = "${var.domain_name}_base_vol"
  pool = "guest_images_hddvg"
  source = "/var/lib/libvirt/images/image-ubuntu-raw"
}

# Create a new resized LV from the base LV
resource "libvirt_volume" "resized_vol" {
  name = "${var.domain_name}_resized_vol"
  pool = "guest_images_hddvg"
  base_volume_id = "${libvirt_volume.base_vol.id}"
  base_volume_pool = "guest_images_hddvg"
  size =<PHONE_NUMBER>0
}

resource "libvirt_domain" "terraform_test" {
  name = "${var.domain_name}"
  disk {
    volume_id = "${libvirt_volume.resized_vol.id}"
    scsi = "true"
  }

  vcpu = "1"
  memory = "2048"

  network_interface = {
    hostname = "${var.domain_name}"
    network_name = "default"
    wait_for_lease = true
  }

  console {
    type        = "pty"
    target_port = "0"
    target_type = "serial"
  }

  console {
    type        = "pty"
    target_type = "virtio"
    target_port = "1"
  }

Steps to Reproduce Issue
terraform plan
terraform apply

Error: Error applying plan:

1 error(s) occurred:

* libvirt_volume.resized_vol: 1 error(s) occurred:

* libvirt_volume.resized_vol: unexpected EOF

Terraform does not automatically rollback in the face of errors.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.
panic: runtime error: invalid memory address or nil pointer dereference
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt: [signal SIGSEGV: segmentation violation code=0x1 addr=0x8 pc=0xc4bf71]
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt:
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt: goroutine 114 [running]:
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt: github.com/dmacvicar/terraform-provider-libvirt/libvirt.newDefBackingStoreFromLibvirt(0xc000312180, 0xc0002da960, 0x10, 0xe291e0, 0x115d
b10, 0x0, 0xc0003673c8)
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt:         /home/baptiste/go/src/github.com/dmacvicar/terraform-provider-libvirt/libvirt/volume_def.go:66 +0x281
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt: github.com/dmacvicar/terraform-provider-libvirt/libvirt.resourceLibvirtVolumeCreate(0xc000192a80, 0xdeb800, 0xc000459a90, 0x0, 0x0)
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt:         /home/baptiste/go/src/github.com/dmacvicar/terraform-provider-libvirt/libvirt/resource_libvirt_volume.go:200 +0x1631
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt: github.com/dmacvicar/terraform-provider-libvirt/vendor/github.com/hashicorp/terraform/helper/schema.(*Resource).Apply(0xc0000f87e0, 0xc0
0045a050, 0xc0002da880, 0xdeb800, 0xc000459a90, 0x427401, 0xc000367b80, 0x4d839c)
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt:         /home/baptiste/go/src/github.com/dmacvicar/terraform-provider-libvirt/vendor/github.com/hashicorp/terraform/helper/schema/resour
ce.go:225 +0x351
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt: github.com/dmacvicar/terraform-provider-libvirt/vendor/github.com/hashicorp/terraform/helper/schema.(*Provider).Apply(0xc0000f9030, 0xc0
0045a000, 0xc00045a050, 0xc0002da880, 0xc0002f2380, 0xc00030ccc0, 0x7f9a2fb49268)
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt:         /home/baptiste/go/src/github.com/dmacvicar/terraform-provider-libvirt/vendor/github.com/hashicorp/terraform/helper/schema/provid
er.go:283 +0x9c
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt: github.com/dmacvicar/terraform-provider-libvirt/vendor/github.com/hashicorp/terraform/plugin.(*ResourceProviderServer).Apply(0xc0004a3b0
0, 0xc0002da580, 0xc0004720f0, 0x0, 0x0)
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt:         /home/baptiste/go/src/github.com/dmacvicar/terraform-provider-libvirt/vendor/github.com/hashicorp/terraform/plugin/resource_prov
ider.go:527 +0x57
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt: reflect.Value.call(0xc00030aa80, 0xc000316058, 0x13, 0xfc9f08, 0x4, 0xc000367f18, 0x3, 0x3, 0xc000098040, 0x0, ...)
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt:         /usr/lib/golang/src/reflect/value.go:447 +0x454
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt: reflect.Value.Call(0xc00030aa80, 0xc000316058, 0x13, 0xc0004c0f18, 0x3, 0x3, 0x0, 0x0, 0x0)
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt:         /usr/lib/golang/src/reflect/value.go:308 +0xa4
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt: net/rpc.(*service).call(0xc00031e840, 0xc000324230, 0xc0002ce840, 0xc0002ce850, 0xc00031a000, 0xc0002da100, 0xdf0080, 0xc0002da580, 0x16
, 0xdf00c0, ...)
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt:         /usr/lib/golang/src/net/rpc/server.go:384 +0x14e
2019-03-20T19:14:16.956+0100 [DEBUG] plugin.terraform-provider-libvirt: created by net/rpc.(*Server).ServeCodec
2019-03-20T19:14:16.957+0100 [DEBUG] plugin.terraform-provider-libvirt:         /usr/lib/golang/src/net/rpc/server.go:481 +0x47e
2019/03/20 19:14:16 [TRACE] root: eval: *terraform.EvalWriteState
2019/03/20 19:14:16 [TRACE] root: eval: *terraform.EvalApplyProvisioners
2019/03/20 19:14:16 [TRACE] root: eval: *terraform.EvalIf
2019/03/20 19:14:16 [TRACE] root: eval: *terraform.EvalWriteState
2019/03/20 19:14:16 [TRACE] root: eval: *terraform.EvalWriteDiff
2019/03/20 19:14:16 [TRACE] root: eval: *terraform.EvalApplyPost
2019/03/20 19:14:16 [ERROR] root: eval: *terraform.EvalApplyPost, err: 1 error(s) occurred:

* libvirt_volume.resized_vol: unexpected EOF
2019/03/20 19:14:16 [ERROR] root: eval: *terraform.EvalSequence, err: 1 error(s) occurred:

* libvirt_volume.resized_vol: unexpected EOF
2019/03/20 19:14:16 [TRACE] [walkApply] Exiting eval tree: libvirt_volume.resized_vol
2019/03/20 19:14:16 [TRACE] dag/walk: upstream errored, not walking "libvirt_domain.terraform_test"
2019-03-20T19:14:16.959+0100 [DEBUG] plugin: plugin process exited: path=/home/baptiste/.terraform.d/plugins/terraform-provider-libvirt
2019/03/20 19:14:16 [TRACE] dag/walk: upstream errored, not walking "provider.libvirt (close)"
2019/03/20 19:14:16 [TRACE] dag/walk: upstream errored, not walking "meta.count-boundary (count boundary fixup)"
2019/03/20 19:14:16 [TRACE] dag/walk: upstream errored, not walking "root"
2019/03/20 19:14:16 [TRACE] Preserving existing state lineage "083dbc9b-b3fc-78af-5640-6fa2d7046b43"
2019/03/20 19:14:16 [TRACE] Preserving existing state lineage "083dbc9b-b3fc-78af-5640-6fa2d7046b43"
2019/03/20 19:14:16 [TRACE] Preserving existing state lineage "083dbc9b-b3fc-78af-5640-6fa2d7046b43"
2019/03/20 19:14:16 [DEBUG] plugin: waiting for all plugin processes to complete...
2019-03-20T19:14:16.961+0100 [WARN ] plugin: error closing client during Kill: err="connection is shut down"

!!!!!!!!!!!!!!!!!!!!!!!!!!! TERRAFORM CRASH !!!!!!!!!!!!!!!!!!!!!!!!!!!!

Terraform crashed! This is always indicative of a bug within Terraform.
A crash log has been placed at "crash.log" relative to your current
working directory. It would be immensely helpful if you could please
report the crash with Terraform[1] so that we can fix this.

When reporting bugs, please include your terraform version. That
information is available on the first line of crash.log. You can also
get it by running 'terraform --version' on the command line.

[1]: https://github.com/hashicorp/terraform/issues

!!!!!!!!!!!!!!!!!!!!!!!!!!! TERRAFORM CRASH !!!!!!!!!!!!!!!!!!!!!!!!!!!!

Additional information:
I'am able to get a resized volume when using the defaut pool storage.

@gwarf thx for issue ! :+1:  we will need to investiagate on it

@gwarf can you retry with the latest master branch? we did a fix in this direction

Hi @MalloZup,
Thanks for looking at this!
So I've tried to clean and reinstall it but now I've got a weird version report:
~/go/bin/terraform-provider-libvirt -version
/home/baptiste/go/bin/terraform-provider-libvirt was not built correctly
Compiled against library: libvirt 4.5.0
Using library: libvirt 4.5.0
Running hypervisor: QEMU 1.5.3
Running against daemon: 4.5.0

I tried to run a deployment with a resize and I'm still getting unexpected EOF and SIGSEV:
Error: Error applying plan:

1 error(s) occurred:

* libvirt_volume.base_vol_resized: 1 error(s) occurred:

* libvirt_volume.base_vol_resized: unexpected EOF

Terraform does not automatically rollback in the face of errors.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.

panic: runtime error: invalid memory address or nil pointer dereference
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt: [signal SIGSEGV: segmentation violation code=0x1 addr=0x8 pc=0xc4ab31]
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt:
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt: goroutine 102 [running]:
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt: github.com/dmacvicar/terraform-provider-libvirt/libvirt.newDefBackingStoreFromLibvirt(0xc00031e330, 0xc00044ee20, 0x26, 0xc00031e330, 0x0, 0x0, 0xc0000853e8)
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt:         /home/baptiste/go/src/github.com/dmacvicar/terraform-provider-libvirt/libvirt/volume_def.go:66 +0x281
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt: github.com/dmacvicar/terraform-provider-libvirt/libvirt.resourceLibvirtVolumeCreate(0xc000196540, 0xdec500, 0xc00047d250, 0x0, 0x0)
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt:         /home/baptiste/go/src/github.com/dmacvicar/terraform-provider-libvirt/libvirt/resource_libvirt_volume.go:202 +0x17b0
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt: github.com/dmacvicar/terraform-provider-libvirt/vendor/github.com/hashicorp/terraform/helper/schema.(*Resource).Apply(0xc0001845b0, 0xc0004d45a0, 0xc00044ed40, 0xdec500, 0xc00047d250, 0x427401, 0xc000085b80, 0x4d839c)
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt:         /home/baptiste/go/src/github.com/dmacvicar/terraform-provider-libvirt/vendor/github.com/hashicorp/terraform/helper/schema/resource.go:225 +0x351
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt: github.com/dmacvicar/terraform-provider-libvirt/vendor/github.com/hashicorp/terraform/helper/schema.(*Provider).Apply(0xc000184d90, 0xc0004d4550, 0xc0004d45a0, 0xc00044ed40, 0xc0000a8000, 0xc000085b78, 0x7f641d1066c0)
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt:         /home/baptiste/go/src/github.com/dmacvicar/terraform-provider-libvirt/vendor/github.com/hashicorp/terraform/helper/schema/provider.go:283 +0x9c
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt: github.com/dmacvicar/terraform-provider-libvirt/vendor/github.com/hashicorp/terraform/plugin.(*ResourceProviderServer).Apply(0xc00030e040, 0xc00044ece0, 0xc0003c0020, 0x0, 0x0)
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt:         /home/baptiste/go/src/github.com/dmacvicar/terraform-provider-libvirt/vendor/github.com/hashicorp/terraform/plugin/resource_provider.go:527 +0x57
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt: reflect.Value.call(0xc00030a4e0, 0xc0003dd310, 0x13, 0xfcace8, 0x4, 0xc000085f18, 0x3, 0x3, 0xc0002272c0, 0x42ee67, ...)
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt:         /usr/lib/golang/src/reflect/value.go:447 +0x454
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt: reflect.Value.Call(0xc00030a4e0, 0xc0003dd310, 0x13, 0xc0003e1f18, 0x3, 0x3, 0x12a05f200, 0xc0003e1f10, 0xc0003e1fb8)
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt:         /usr/lib/golang/src/reflect/value.go:308 +0xa4
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt: net/rpc.(*service).call(0xc000309340, 0xc0000ca690, 0xc0002ce798, 0xc0002ce7b0, 0xc0003d4080, 0xc0003c9280, 0xdf0d80, 0xc00044ece0, 0x16, 0xdf0dc0, ...)
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt:         /usr/lib/golang/src/net/rpc/server.go:384 +0x14e
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt: created by net/rpc.(*Server).ServeCodec
2019-04-04T10:40:24.382+0200 [DEBUG] plugin.terraform-provider-libvirt:         /usr/lib/golang/src/net/rpc/server.go:481 +0x47e
2019/04/04 10:40:24 [TRACE] root: eval: *terraform.EvalWriteState
2019-04-04T10:40:24.386+0200 [DEBUG] plugin: plugin process exited: path=/home/baptiste/.terraform.d/plugins/terraform-provider-libvirt
2019/04/04 10:40:24 [TRACE] root: eval: *terraform.EvalApplyProvisioners
2019/04/04 10:40:24 [TRACE] root: eval: *terraform.EvalIf
2019/04/04 10:40:24 [TRACE] root: eval: *terraform.EvalWriteState
2019/04/04 10:40:24 [TRACE] root: eval: *terraform.EvalWriteDiff
2019/04/04 10:40:24 [TRACE] root: eval: *terraform.EvalApplyPost
2019/04/04 10:40:24 [ERROR] root: eval: *terraform.EvalApplyPost, err: 1 error(s) occurred:

* libvirt_volume.base_vol_resized: unexpected EOF
2019/04/04 10:40:24 [ERROR] root: eval: *terraform.EvalSequence, err: 1 error(s) occurred:

* libvirt_volume.base_vol_resized: unexpected EOF
2019/04/04 10:40:24 [TRACE] [walkApply] Exiting eval tree: libvirt_volume.base_vol_resized
2019/04/04 10:40:24 [TRACE] dag/walk: upstream errored, not walking "libvirt_domain.terraform_test"
2019/04/04 10:40:24 [TRACE] dag/walk: upstream errored, not walking "meta.count-boundary (count boundary fixup)"
2019/04/04 10:40:24 [TRACE] dag/walk: upstream errored, not walking "provider.libvirt (close)"
2019/04/04 10:40:24 [TRACE] dag/walk: upstream errored, not walking "root"
2019/04/04 10:40:24 [TRACE] Preserving existing state lineage "9fba15a8-985f-9514-3b38-4dbf7e0dc466"
2019/04/04 10:40:24 [TRACE] Preserving existing state lineage "9fba15a8-985f-9514-3b38-4dbf7e0dc466"
2019/04/04 10:40:24 [TRACE] Preserving existing state lineage "9fba15a8-985f-9514-3b38-4dbf7e0dc466"
2019/04/04 10:40:24 [DEBUG] plugin: waiting for all plugin processes to complete...
2019-04-04T10:40:24.389+0200 [WARN ] plugin: error closing client during Kill: err="connection is shut down"
2019-04-04T10:40:24.390+0200 [DEBUG] plugin.terraform-provider-template_v2.1.0_x4: 2019/04/04 10:40:24 [ERR] plugin: plugin server: accept unix /tmp/plugin454517233: use of closed network connection
2019-04-04T10:40:24.392+0200 [DEBUG] plugin: plugin process exited: path=/home/baptiste/repos/Ops-Hacks/Deployment/Ops-server/Terraform/resize_ubuntu_lvm/.terraform/plugins/linux_amd64/terraform-provider-template_v2.1.0_x4

!!!!!!!!!!!!!!!!!!!!!!!!!!! TERRAFORM CRASH !!!!!!!!!!!!!!!!!!!!!!!!!!!!

Terraform crashed! This is always indicative of a bug within Terraform.
A crash log has been placed at "crash.log" relative to your current
working directory. It would be immensely helpful if you could please
report the crash with Terraform[1] so that we can fix this.

When reporting bugs, please include your terraform version. That
information is available on the first line of crash.log. You can also
get it by running 'terraform --version' on the command line.

[1]: https://github.com/hashicorp/terraform/issues

!!!!!!!!!!!!!!!!!!!!!!!!!!! TERRAFORM CRASH !!!!!!!!!!!!!!!!!!!!!!!!!!!!

@gwarf thx for reply.
OK i suspect that we don't recognize the type.
Could you give me a step by step guide for how-to-reproduce? i'm not familiar with lvm but i can learn :) thx for your help :cupid:

OK, so for LVM this [CentOS7 LVM tutorial] may provide some guidance.
My setup is the following:

a Volume Group named hddvg (running on a Physical Volume that is a single partition /dev/sdb1)
a guest_images_hddvg  storage pool form that Volume Group (you can check redhat documentation for some info on how to manage it using virt-manager or this doc that I used about managing LVM storage pool with virsh
a default directory storage pool in /var/lib/libvirt/images to contain OS images

So you should have a LVM physical volume, a LVM volume group and a storage pool, using something like this:
# Create a partition for LVM
sudo gdisk /dev/sdb
sudo gdisk -l /dev/sdb
# Create a LVM physical volume with the disk partition
sudo pvcreate /dev/sdb1
sudo pvdisplay
sudo pvs
# No existing Volume Groups yet
sudo vgdisplay
sudo vgs
# Create storage pool using a volume group
sudo virsh pool-define-as guest_images_hddvg logical - - /dev/sdb1 hddvg /dev/hddvg
sudo vgcreate hddvg /dev/sdb1
sudo virsh pool-start guest_images_hddvg
sudo virsh pool-autostart guest_images_hddvg
# Check what we have
sudo virsh pool-list --all
sudo virsh pool-info guest_images_hddvg
sudo vgdisplay
sudo vgs

Then you can create volumes in the storage pool:
# Create a volume
sudo virsh vol-create-as guest_images_hddvg volume_hdd_1 8G
# Check volumes
sudo virsh vol-list guest_images_hddvg
sudo lvdisplay
sudo lvscan
sudo lvs
# Clean volume
sudo virsh vol-delete volume_hdd_1 guest_images_hddvg

As I'm not able to create a volume from a qcow2 (it wants to know the image size), I had to convert an image to raw:
cd /var/lib/libvirt/images
wget https://cloud.centos.org/centos/7/images/CentOS-7-x86_64-GenericCloud.qcow2
qemu-img convert -f qcow2 -O raw CentOS-7-x86_64-GenericCloud.qcow2 CentOS-7-x86_64-GenericCloud.raw

And with this I'm able to use the raw image from the default pool to create volume on the guest_images_hddvg pool. The VMs work as expected.
resource "libvirt_volume" "volume-root-deploy" {
  name = "root-deploy"
  pool = "guest_images_hddvg"
  source = "/var/lib/libvirt/images/CentOS-7-x86_64-GenericCloud.raw"
}

And to break everything I try to do this:
resource "libvirt_volume" "base_vol" {
  name = "lvm-ubuntu"
  pool = "guest_images_hddvg"
  source = "/var/lib/libvirt/images/image-ubuntu-raw"
}

resource "libvirt_volume" "base_vol_resized" {
  name = "lvm-ubuntu-resized"
  base_volume_id = "${libvirt_volume.base_vol.id}"
  pool = "guest_images_hddvg"
  # Size in bytes (20 GB)
  size =<PHONE_NUMBER>0
}

Let me know if something is missing/unclear.
Thanks!

Just as a quick note: This also happens on Debian 9 & 10!

Is there any update on this?
As far as I understood libvirt/volume_def.go:66 crashes trying to access baseVolumeDef.Target.Format.Type because Format is nil.
According to libvirt documentation, lvm volumes does not use format type element.
This is my example volume the plugin fails to use as base volume
<volume type='block'>
  <name>debian-buster-201908131525-amd64</name>
  <key>Ms9YcN-LWqh-MSOb-uiJQ-HQaq-ueTU-3Cbjir</key>
  <source>
    <device path='/dev/sda3'>
      <extent start='129922760704' end='132070244352'/>
    </device>
  </source>
  <capacity unit='bytes'>2147483648</capacity>
  <allocation unit='bytes'>2147483648</allocation>
  <physical unit='bytes'>2147483648</physical>
  <target>
    <path>/dev/kvm-vg/debian-buster-201908131525-amd64</path>
    <permissions>
      <mode>0660</mode>
      <owner>0</owner>
      <group>6</group>
    </permissions>
    <timestamps>
      <atime>1565770634.242217437</atime>
      <mtime>1565770634.242217437</mtime>
      <ctime>1565770634.242217437</ctime>
    </timestamps>
  </target>
</volume>

This is an example of valid lvm snapshot
<volume type='block'>
  <name>debian-buster-201908131525-amd64-s</name>
  <key>L2gseq-g9L6-1GE4-OoZj-0NDY-BrbI-GT2lqY</key>
  <source>
    <device path='/dev/sda3'>
      <extent start='146028888064' end='148188954624'/>
    </device>
  </source>
  <capacity unit='bytes'>2147483648</capacity>
  <allocation unit='bytes'>2160066560</allocation>
  <physical unit='bytes'>2147483648</physical>
  <target>
    <path>/dev/kvm-vg/debian-buster-201908131525-amd64-s</path>
    <permissions>
      <mode>0660</mode>
      <owner>0</owner>
      <group>6</group>
    </permissions>
    <timestamps>
      <atime>1565786914.351383181</atime>
      <mtime>1565786914.351383181</mtime>
      <ctime>1565786914.351383181</ctime>
    </timestamps>
  </target>
  <backingStore>
    <path>/dev/kvm-vg/debian-buster-201908131525-amd64</path>
    <permissions>
      <mode>0660</mode>
      <owner>0</owner>
      <group>6</group>
    </permissions>
    <timestamps>
      <atime>1565786914.319383462</atime>
      <mtime>1565786914.319383462</mtime>
      <ctime>1565786914.319383462</ctime>
    </timestamps>
  </backingStore>
</volume>

ok thx i will put it in the backlog since I see the community need it.

So, here's a workaround.
Just edit /etc/lvm/lvm.conf and change: wipe_signatures_when_zeroing_new_lvs = 1 to wipe_signatures_when_zeroing_new_lvs = 0.
Read the config comments to understand. It hasn't caused me any issues so far.
The solution to this is to add --yes to the command. There are some related issues in Red hat's bugzilla about this. I think some other project suffered of the same. Just sayin'


feat(docs): updated link to have latest reference

Signed-off-by: Felipe Avelar<EMAIL_ADDRESS>Proposed Changes

Now docs will have a valid route /latest instead of redirecting to the last version

I submit this contribution under the Apache-2.0 license.

Does this fix #3036 ?

@kaplanlior not yet. Due to legacy pages pointing to old documentation


Added contributors on README

Title
Added Contributors on READMe
Description of what's changing
Contributors are now reflecting on README
...
Which issue does this PR relate to?
#108
...
Checklist
[ ] Tests are written and maintain or improve code coverage
[ ] I've tested this in my application using yarn link (if applicable)
[ ] You consent to and are confident this change can be released with no regression

Looks good! Thank you for contributing :)

Looks good! Thank you for contributing :)

Thanks for helping :)

@all-contributors please add @Ayushisood for docs


Add exercise: Greatest Common Divisor

Codecov Report

Merging #3 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #3   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          12     13    +1
  Lines         225    231    +6
  Branches       39     41    +2
=====================================
+ Hits          225    231    +6

Impacted Files
Coverage Δ

src/greatest-common-divisor.js
100% <100%> (ø)

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 522de9c...2dc0d44. Read the comment docs.


RxJS 5 "extensions" - where should they live?

So I've found myself needing recently a couple of the extensions from earlier versions of Rx.DOM and Rx.Node (stuff like requestAnimationFrameScheduler and Observable.fromStream) - do you want PR's here for these things, or should there be a @reactivex/rxjs-extensions repo or?
I'd err on the side of them living in here, as you can (theoretically) pluck the things you want out, but wanted to check first...

@robwormald I should have the new RAF Scheduler finished this weekend. Since everything's modular now, I'm fine committing it to the main project and importing it as necessary.

Awesome, thanks.

As I understood RAF scheduler is going to be available in this repo. What about others things such as Rx.DOM.ajax?

@roganov that's the plan at this point. Since everything is modular, we can include other RxJS utilities in this repo. Ajax, web sockets, rxify helpers, etc.

I've been working to update the rx-node package that @mattpodwysocki broke out during the rx v2 days, and I can submit a PR for this.  One question, is if I should add these operators to a node folder, at the same level as the DOM operators, or if they should just go in with the normal operators?


[DOCFIX] Update POSIX-API.md

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.5 please


Remove all component-specific tokens in react

The following components are using the old --component-specific tokens:

CheckboxRadioTemplate
InputWrapper
Checkbox
CheckboxGroup
FieldSet
LegacyAccordion and LegayAccordionHeader
List
ListItem
RadioButton
RadioGroup
Select
TextArea
ToggleButtonGroup

This issue is no longer needed as we are doing this as part of our "improvements" issues for the components when going from "Altinn" to "Felles" components


ft: configurable number of search results

This pull request...

[ ] Fixes a bug
[x] Introduces a new feature
[ ] Improves an existing feature
[ ] Boosts code quality or performance

Description
This adds a config key, maxssearchresults which defaults to 4.
Purpose
Sometimes songs are obscure and don't appear in the first 4 results. A configurable limit alleviates this.
Relevant Issue(s)
This PR closes issue #758

Apologies for the whitespace changes 😬

Completed superseded by #1006


[Eurotech] Models update to support asset command response.

Update models to include complete asset command response. Fix data types for diagnostic service.
Signed-off-by: David Woodard<EMAIL_ADDRESS>Company Info
Eurotech
https://www.eurotech.com/en
IoT Plug and Play Device Info
ReliaGate 10-12-6x
https://www.eurotech.com/en/products/iot/multi-service-iot-edge-gateways/reliagate-10-12
Model Submission Goals
Update model to include internal modules.
Code Owners & Reserved Names

@rido-min Synced with main branch.


[ENHANCEMENT] (Provide a feasible and upload channel for adding software icons and screenshots from Chinese Mainland and other restricted regions)

Please confirm these before moving forward

[X] I have searched for my feature proposal and not found a work-in-progress/duplicate/resolved/discarded issue.
[X] This improvements refers to an existing feature. If you want to suggest a new feature, please use this template
[X] This improvements is not a bug. If you want to report a bug, please use this template

Describe the improvement
Due to the use of translation software, my expression may be inaccurate
As a Chinese, I would like to help add more software icons and screenshots, but due to network access restrictions in China Mainland, I am unable to access Google Spreadsheet and I do not have a VPN. Therefore, is it allowed for us to use other methods (such as Github) to collect data
THANKS
Describe how this improvement could help users.
Increase the data of software in China, and help more Chinese people use WingetUI

In addition, it is impossible to display whether the image information of any software is related to Google Spreadsheet
I chose Visual Studio Code because I guess someone should have uploaded images for it, but there is no display here

I am planning to add a proper interface to publish new icons


The assets directory

All files will in assets directory except page files when build command finished.

Maybe some loan companies will name the assets(资产/资金) as a page
Sure, that's my last company😂

I considering about _dvan or _assets😂

Decide __assets with 1333a9


Memory leak issue in RtspSnapshotMaker from previous closed issue #40

          Hi @BogdanovKirill

I am also wondering about this memory leak issue.  I am using the decoding github project "rtspcapture - working" which gives me the SnapshotMaker example. I changed it to a Winforms project so I could stream to a picture box, and then take a snapshot, and put it in another picture box.
I commented out the ToImage() function so I took out all displaying of Images in my picture boxes. The memory seems more stable.
When calling the ToImage function, my memory spikes up when taking shapshots, even though I am clearing my Picture Boxes.
As I take more snapshots, the memory climbs higher.
It never come down, even when calling the GC explicitly.
I am not sure how to clean up ?  Can you help?
(This code comes from my crude test program)
    private void OnFrameReceived(object sender, RawFrame rawFrame)
    {
        if (!(rawFrame is RawVideoFrame rawVideoFrame))
            return;

        FFmpegVideoDecoder decoder = GetDecoderForFrame(rawVideoFrame);
        if (!decoder.TryDecode(rawVideoFrame, out DecodedVideoFrameParameters decodedFrameParameters))
            return;

        int bufferSize;

        bufferSize = decodedFrameParameters.Height *
                     ImageUtils.GetStride(decodedFrameParameters.Width, RawFramesDecoding.PixelFormat.Abgr32);

        if (_decodedFrameBuffer.Length != bufferSize)
            _decodedFrameBuffer = new byte[bufferSize];

        var bufferSegment = new ArraySegment<byte>(_decodedFrameBuffer);

        var postVideoDecodingParameters = new PostVideoDecodingParameters(RectangleF.Empty,
            new Size(decodedFrameParameters.Width, decodedFrameParameters.Height),
            ScalingPolicy.Stretch, RawFramesDecoding.PixelFormat.Bgr24, ScalingQuality.Bicubic);

        IDecodedVideoFrame decodedFrame = decoder.GetDecodedFrame(bufferSegment, postVideoDecodingParameters);

        //ArraySegment<byte> frameSegment = decodedFrame.DecodedBytes;
        //string snapshotName = decodedFrame.Timestamp.ToString("O").Replace(":", "_") + ".jpg";

        //ToImage(decodedFrameParameters.Width, decodedFrameParameters.Height, snapshotName, System.Drawing.Imaging.PixelFormat.Format24bppRgb, frameSegment.Array);

    }

    private void ToImage(int Width, int Height, string imgName, System.Drawing.Imaging.PixelFormat pixelFormat, byte[] rgbValues)
    {
        Bitmap bitMap = new Bitmap(Width, Height, pixelFormat);

        Rectangle BoundsRect = new Rectangle(0, 0, Width, Height);

        BitmapData bitmapData = bitMap.LockBits(BoundsRect, ImageLockMode.WriteOnly, bitMap.PixelFormat);

        IntPtr _dataPointer = bitmapData.Scan0;

        int _byteData = bitmapData.Stride * bitMap.Height;

        System.Runtime.InteropServices.Marshal.Copy(rgbValues, 0, _dataPointer, _byteData);

        bitMap.UnlockBits(bitmapData);

        if (pictureBox1.Image != null)
        {
            pictureBox1.Image.Dispose();
            pictureBox1.Image = null;
        }

        pictureBox1.Image = bitMap;
        pictureBox1.Refresh();

        if (MustTakeSnapshot)
        {
            Thread t = new Thread(new ParameterizedThreadStart(TakeSnapshot));
            t.Start(bitMap);
        }
    }

    private void TakeSnapshot(object bitMap)
    {

        Bitmap b = (Bitmap)bitMap;

        ImageCodecInfo jgpEncoder = GetEncoder(ImageFormat.Jpeg);

        System.Drawing.Imaging.Encoder encoder = System.Drawing.Imaging.Encoder.Quality;

        EncoderParameters encoderParams = new EncoderParameters(1);

        EncoderParameter myEncoderParameter = new EncoderParameter(encoder, 50L);

        encoderParams.Param[0] = myEncoderParameter;

        string filename = @"c:\capture\frame" + DateTime.Now.Millisecond.ToString();
        b.Save(filename, jgpEncoder, encoderParams);

        if (pictureBox2.Image != null)
        {
            pictureBox2.Image.Dispose();
            pictureBox2.Image = null;
        }

        pictureBox2.Image = Image.FromFile(filename);

        b.Dispose();
        b = null;
        bitMap = null;

        MustTakeSnapshot = false;
    }

Originally posted by @zoekatakuzinos in https://github.com/BogdanovKirill/RtspClientSharp/issues/40#issuecomment-1914546992

I have made it work.
I did not start a new thread from the ToImage function (which was called from OnFrameReceived)
I rather put the code directly into ToImage function and the memory seems stable now.
It was perhaps not a good design to start a new thread from within an async function


Updated gms:play-services to be more specific on requiring only GCM

This change is more specific & accurate for this library, as we only require GCM. Else, a user is required to install more packages than needed from 'Extras' in packages just to compile.
This can also mislead some users (me as well) to put in 'multidex true' in their build.gradle which is discouraged, and also causes Android versions under 5.0 to crash upon boot.

Thanks!


Wb input

Preliminary Watts input structure for Workbench workflow management. To discuss together. This is very preliminary and a first step with partial input definition and logic to show how to extract data in python script.

@zhieejhia93 - please update your local development branch so that you can focus on the changes I made!


Wapiti library not supported on Windows

See fork with JNI: https://github.com/kermitt2/Wapiti

See #74


Add option to remove tab close buttons

There should be a preference for removing tab close buttons (it just needs to apply the css style ".close-editor-action {display:none;}" to the interface) - tab close buttons make it too easy to accidentally close a tab, and you can use middle-click instead anyway.

@bpasero @hashhar As this has now been implemented by you when will it get checked into the main codebase?

@jez9999 @hashhar I am not seeing a PR yet?

@bpasero @jez9999 I am only able to hide and show the close buttons but they don't refresh automatically. You would have to at least drag the tab to make the setting take effect.
I am unable to get it to repaint the window at my will.
I'll look into it again but right now I have campus placement sessions coming up so I'm a little too busy.

@hashhar Your patch modifies the code where the tabs are created, but is there a way to apply a global stylesheet to the UI?  If so, you could just add or remove a CSS styling that hides all the tab close buttons.  That seems the most straightforward way to do it.

@hashhar also keep in mind that you do not want to hide the dirty decorator which is drawn on top of the close action.

Thanks for the input. I'll take it up again after the placements are over.

@hashhar How long will the placements take?  This seems like a very small patch and I'd really like to get rid of those tab close buttons as soon as possible. :-)

You are welcome to try and write a patch if you would like. I think it should be over by 28th of this month.

@hashhar Bump!  Any chance of progressing on this issue now? :-)

I am not able to figure out how tabs are drawn. I think I have a long week ahead of me of stepping through the entire codebase to learn how stuff works.

@hashhar Did you get anywhere with this?  I just closed another tab accidentally!  Grrrr!

Pushed this behind a new setting workbench.editor.showTabCloseButton. The dirty indicator when visible serves as action to save the file:

I am sorry to maybe sound a little negative, but IMO the reason why people wanted to remove the close button is because they don't want to accidentally push any buttons while switching tabs, just being free to press anywhere on the tab, without worrying what will happen - at least I do. So to replace a close button with another button is a bit counter productive. Maybe add settings for it too?

Exactly.
@bpasero While having the indicator save the document is at least a lot less destructive that closing it, I'm still disappointed you didn't just use my code.  The idea is that we want to be able to click the tab button with no side effects apart from selecting or moving the tab.  Your solution gives the side effect of saving.  I still suggest you back this out and implement my patch.

@PunchyRascal thanks for your comment on this, I see the issue and pushed a fix for it.

Nice.    Works great!


Bot To Handle Approvers AND Reviewers In Owners File

Here's the proposed workflow as described by @bgrant0607
Proposed workflow:
a reviewer, who isn't necessarily an approver, would be assigned
the reviewer would eventually type "LGTM" into a comment
the author would (often) squash commits and/or rebase
the reviewer would then type I approve if they were an approver, or would notify an approver to do that if they were not; the author could self-approve if they were an approver
I believe @hongchaodeng is working on tooling to automatically generate an owners file with two components, an Reviewers List and an Owners List.
We will need to implement our own ACLs using the bot to verify that only approvers can make the final "approvals".
@apelisse @foxish let me know if this sounds reasonable, and if so, I'll start working on it.

@grodrigues3
/lgtm now
reviewers aka assignees
owners aka approvers
see also #1429
cc @hongchaodeng @philips @apelisse @foxish

and the existing issue is #1389

I do have the reviews list generated. It appends to existing OWNERS or creates new OWNERS with a separate section called "reviewers":
diff --git a/build/OWNERS b/build/OWNERS
index acc600a..e10e94b 100644
--- a/build/OWNERS
+++ b/build/OWNERS
@@ -3,3 +3,9 @@ assignees:
   - ixdy
   - lavalamp
   - zmerlynn
+
+reviewers:
+ - thockin
+ - mikedanese
+ - luxas
+ - andyzheng0831

Even though the program is ready and file is generated, I'm not sure about the current state yet.. Maybe I should also merge the "assignees" and "reviewers".
The other point I'm confused is about approver. Can someone help create approver list? Once we have that we can continue on #1429 to implement the approval system.

@hongchaodeng yes, reviewers and assignees are same thing and therefore can be merged.   Similarly, Owners and Approvers are the same thing.   Reviewers should be a pretty broad pool of people who have some minimum level of familiarity with the code (e.g have touched a related file).   The approvers list may be harder to generate automatically, but perhaps we can figure out how to generate that at the community meeting tomorrow.

Just to sync up here while I'm updating  https://github.com/kubernetes/kubernetes/pull/31567. Are we OK with promoting the current list of GitHub handles in the existing OWNERS files to fall under an approvers key. New potential reviewers identified by the tooling that @hongchaodeng has been working on will listed under a reviewers key. My understanding of the discussion from the contrib-x meeting on 25 August was that we are going to address automatic promotion of reviewers to approvers in a separate track of work. Does that all sound correct to you @grodrigues3, @bgrant0607, @apelisse and @hongchaodeng?

yeah, sounds right to me.  we can address automatic promotion of reviewers into approvers later.  and we can initially use the existing owners as approvers.


Partial database

Import plugin should skip empty tables and use only those populated by data. I have tested a small portions of data just to check how it works.
Import tool stops working when there is no forum admin and members in MySQL database (only a real table structure with a partial data).
Let say I have populated MySQL rows with all categories and something more which is executed by import script after members table. Only categories will be imported and there is no way to import posts and private messages.
It can be fixed by manually adding more records to the database. Not a real bug and will not affect those who use a whole database dump.
Tested on IP Board 2.3.x > NodeBB 0.6.1.

i see your point, but sometimes, some data records depend on others to exist as well.
For example:
Each topic depends on a parent-category and a user
Each post depends on a parent-topic and a user


Hydration when using fetch on client side breaks the nuxt site

The module seem to prevent the app from executing fetch api calls on client side.
Here is a reproduction of the issue:
https://codesandbox.io/s/nuxt-delay-hydrate-fetch-api-bug-ucd2o
@harlan-zw advise on such a scenario

Not sure what exactly is going on here, seems like the issue is that you don't have the module in manual mode but you are using it as if it was in manual mode.
Using the latest version and setting to manual mode seems to have the fetch work correctly. The component should be automatically imported for you via nuxt/components
 delayHydration: {
    mode: "manual"
  }

If you still have issues let me know


Adding repositories/id to URL_VALIDATOR

This is a nondocumented public way of getting a repository by ID.

I noticed GitHub began canonicalizing their repository URLs a while back, thanks for adding this!

No problem!
On Wed, May 10, 2017 at 5:47 PM Philip Schatz<EMAIL_ADDRESS>wrote:

Merged #168 https://github.com/philschatz/octokat.js/pull/168.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/philschatz/octokat.js/pull/168#event-1076691192, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA3loVDJDHIZAsCWhgsuhw7E_N6AMCRDks5r4dwpgaJpZM4NVW_l
.


[WIP] Check the current state of warnings

This is not meant to be merged yet, I want to evaluate how many warnings exist on each platform

Thanks @sstamenova for doing this, much appreciated.

This fill fix almost all remaining warnings on Windows: https://github.com/onnx/onnx-mlir/pull/1657


Passing large schema in as string causes pattern too long error

Issue workflow progress

Progress of the issue based on the
Contributor Workflow

[ ] 1. The issue provides a reproduction available on GitHub, Stackblitz or CodeSandbox

For example, you can start off by editng the
'basic' example on Stackblitz.

Please make sure the graphql-eslint version under package.json matches yours.

[ ] 2. A failing test has been provided

[ ] 3. A local solution has been provided

[ ] 4. A pull request is pending review

Describe the bug

When passing a large schema of over 8kb as a string instead of file path the loader attempts to pattern match the raw string and will throw and eslint pattern too long error.
To Reproduce Steps to reproduce the behavior:
Programatically extend and call the ESLint class and load the schema as a string into parserOptions. Next use the lintText method of the class to lint the GraphQL schema subset against the entire schema. Here an error will be thrown about the pattern being too long.

Environment:

OS: Linux/MacOS
@graphql-eslint/eslint-plugin: 3.20.1
Node.js:

Additional context
This appears to be worked around in version 4 of the linter here by building the schema differently in the browser for use in the playground.

Hey @dimaMachina I am working on it, Accidentally submitted before wrapping up and I am editing it now

@dimaMachina I have updated the bug with a description. I will see if I can put a sandbox together as an example. Our schema cannot be shared but I think I can create a similar way to reproduce if it would help

@cernst11 thank you, some reproduction will be very helpful

Hey @dimaMachina I have created an example here. I simplified our code down to the bare minimum and generated a mock schema.

@cernst11 so the error says Parsing error: [graphql-eslint] pattern is too long which goes from graphql-config's minimatch dependency
https://github.com/isaacs/minimatch/blob/ef8f2672bdbbf6a632ea815636659fb31b5169aa/src/assert-valid-pattern.ts#L1-L12
where value const MAX_PATTERN_LENGTH = 1024 * 64 is hardcoded to 65536, your schema SDL contains 73892 characters

fixed in<EMAIL_ADDRESS>


Specify canonical user for gcp instance exec

Previously we supplied the user registered with GCP metadata as a FallbackUser, which allowed a user's SSH config to change the user. However this behavior is inconsistent with gcloud ssh and resulted in default user entries (Host *) in SSH config breaking this functionality.
Switch to using the user from GCP metadata as the canonical user for login. If we later need to support changing that user, it should be a new feature of the wexec command/API.
Fixes #725.
Signed-off-by: Michael Smith<EMAIL_ADDRESS>

If it's empty then it will behave as before. And I think it will always be non-empty.


Create DMTanks-SphericalTanks.netkan

newly released

SphericalTanks_v1_0_0.zip
I don't understand the error? Zip File Attached... appears to have the folder that's supposedly missing.

Seems like another case of https://github.com/KSP-CKAN/NetKAN/pull/3824.
I extracted the contents and created a new .zip and had no issues. By chance did you use the right-click -> compress -> "to zip file" option in windows explorer to create the .zip?

I did, I think.
Sent from my iPhone

On May 24, 2016, at 8:25 PM, Matthew<EMAIL_ADDRESS>wrote:
Seems like another case of #3824.
I extracted the contents and created a new .zip and had no issues. By chance did you use the right-click -> compress -> "to zip file" option in windows explorer to create the .zip?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

This time I created a new .zip file from the file menu in file explorer, then copied the contents into it.... does this one work?
SphericalTanks_v1_0_0.zip

seems to work fine now. Hoping for a swift merge.

That's looking fine as far as I can see. Merging with many thanks!