text:
Still blacked text and background color on tooltips.

[BUG]
Hi. On my project, tooltip text and background color are both blacked. Texts are not readable, and any of my changes could not solve the problem.
I've checked different solutions mentioned in #4766, #4741, #4783, and ... But none of them could not solve my problem. I am using the latest version of Chart.js. Also I've test it in both Chrome and Firefox.

@mbsmt could you provide a jsfiddle that demonstrates the example?

@jcopperfield it's really strange. It worked well without any issue out of my project.
https://jsfiddle.net/a6uqexmw/2/
What might be happened that caused the problem?
Compare the jsfiddle above with this new snapshot ...

@mbsmt I think you might have a local problem with your browsers.

@jcopperfield I tested it with Chrom, Firefox, IE. All of them have that problem!

What version of Chart.js are you using, I think it's a bug fixed in 2.7.1 (#4783)

@mbsmt Might be a driver problem. Your fiddle doesn't recreate the bug for me.

@simonbrunel It's 2.7.1, the latest version. I tested it outside my project in a standalone html file, and it worked without any problem. I had to remove all included CSS and JS files one by one from my project and finally I found a conflict between Chart.js and Persianumber.js.

@mbsmt it seems like Persianumber.js overwrites the parseInt and parseFloat functions. This is not good practice. However if you the problem is that the parseInt implementation doesn't include the radix argument, which causes the problem.
You can either remove the parseInt and parseFloat functions from the Persianumber.js or you should add the radix argument to parseInt, like shown below.
origParseInt = parseInt;
parseInt = function(str, radix) {
    str = str && str.toString().replace(/[\u06F0\u06F1\u06F2\u06F3\u06F4\u06F5\u06F6\u06F7\u06F8\u06F9]/g, function(v){return String.fromCharCode(v.charCodeAt(0) - 1728)}).replace(/[\u0660\u0661\u0662\u0663\u0664\u0665\u0666\u0667\u0668\u0669]/g, function(v){return String.fromCharCode(v.charCodeAt(0) - 1584)}).replace(/[\u066B]/g, '.');
    return origParseInt(str, radix);
};

@jcopperfield That's it... You solved my problem. Thank you and all other guys.

authors: jcopperfield mbsmt simonbrunel


text:
Bug: After connecting to the DB2 database, the table list cannot be displayed

Chat2DB Version
3.0.14
Describe the bug
After connecting to the db2 database, all tables under the current schema cannot be displayed in the table list on the left.
db2 version v11

try new version

source: gharchive/issue

authors: batshelltan miahemu

license_type: permissive

license_source: github-api


text:
portable solution

I'm trying to use pandoc-latex-color. But I'd really like to have a portable solution that works via Dropbox on multiple computers. This is how I use latex templates across computers as well. It would be nice if there wasn't a machine-dependent "installation" process.
Is such a thing possible?

there is not machine dependent process in the installation. pip is a standard way for installing python package

Sorry. Let me explain again. If I use pip I need to install the python package on each computer (they're all Macs BTW). What I'd like to do is be able to put the pandoc-latex-color.py and any associated scripts in the ~/.pandoc/filters/ directory and have it work across multiple computers without needing to install using pip on each computer.
Is that possible?

You can use
$ pip install --user pandoc-latex-color

and add the $USER/.local/bin folder (on a MacOS, it could be different) to the $PATH variable.

Thanks.

repo: chdemko/pandoc-latex-color


text:
Docker needs to be installed before successful chef-client run

Cookbook version
2.8.0
Chef-client version
12.7.2
Platform Details
CentOS 7.2 - Vagrant or DigitalOcean
Scenario:
Basic Cookbook to install docker, pull an image, start the container fails.
Steps to Reproduce:
Create a new cookbook and use this recipe:
imgvars  = node['docker']['images']
contvars = node['docker']['containers']

directory '/data' do
  action :create
end

docker_service 'default' do
  action [:create, :start]
end

imgvars.each do |dimages|
  docker_image dimages[:name] do
    tag dimages[:tag]
    action :pull
  end
end

contvars.each do |containers|
 docker_container containers[:name] do
   repo containers[:repo]
   tag containers[:tag]
   port containers[:port]
   network_mode containers[:network_mode]
   links containers[:link]
   volumes containers[:volumes]
   env containers[:env]
   restart_policy 'always'
 end
end

And this attributes/default.rb
default['docker']['images']    = [
  { :name=>"ghost", :tag=>"latest" },
  { :name=>"abiosoft/caddy", :tag=>"latest" }
]

default['docker']['containers'] = [
  { :name=>"ghost", :repo=>"ghost", :tag=>"latest", :port=>'2368:2368', :volumes=>"/data/ghost:/var/lib/ghost" },
  { :name=>"caddy", :repo=>"abiosoft/caddy", :tag=>"latest", :port=>['80:80', '443:443'], :volumes=>"/data/caddy/Caddyfile:/etc/Caddyfile" }
]

Expected Result:
Docker should download the 'ghost' official image and start the container.
Actual Result:
$ vagrant provision
==> default: Loading Berkshelf datafile...
==> default: Sharing cookbooks with VM
==> default: Updating Vagrant's Berkshelf...
==> default: Resolving cookbook dependencies...
==> default: Fetching 'dockertest' from source at .
==> default: Using compat_resource (12.10.6)
==> default: Using docker (2.8.0)
==> default: Using dockertest (0.1.0) from source at .
==> default: Vendoring compat_resource (12.10.6) to /Users/cpressland/.berkshelf/vagrant-berkshelf/shelves/berkshelf20160613-91532-b5ofu0-default/compat_resource
==> default: Vendoring docker (2.8.0) to /Users/cpressland/.berkshelf/vagrant-berkshelf/shelves/berkshelf20160613-91532-b5ofu0-default/docker
==> default: Vendoring dockertest (0.1.0) to /Users/cpressland/.berkshelf/vagrant-berkshelf/shelves/berkshelf20160613-91532-b5ofu0-default/dockertest
==> default: Chef 12.7.2 Omnibus package is already installed.
==> default: Running provisioner: chef_solo...
==> default: Detected Chef (latest) is already installed
==> default: Generating chef JSON and uploading...
==> default: Running chef-solo...
==> default: [2016-06-13T14:09:26+00:00] INFO: Forking chef instance to converge...
==> default: Starting Chef Client, version 12.7.2
==> default: [2016-06-13T14:09:26+00:00] INFO: *** Chef 12.7.2 ***
==> default: [2016-06-13T14:09:26+00:00] INFO: Chef-client pid: 14509
==> default: [2016-06-13T14:09:27+00:00] INFO: Setting the run_list to ["recipe[dockertest::default]"] from CLI options
==> default: [2016-06-13T14:09:27+00:00] INFO: Run List is [recipe[dockertest::default]]
==> default: [2016-06-13T14:09:27+00:00] INFO: Run List expands to [dockertest::default]
==> default: [2016-06-13T14:09:27+00:00] INFO: Starting Chef Run for vagrant-3712a224
==> default: [2016-06-13T14:09:27+00:00] INFO: Running start handlers
==> default: [2016-06-13T14:09:27+00:00] INFO: Start handlers complete.
==> default: Compiling Cookbooks...
==> default:
==> default: ================================================================================
==> default: Recipe Compile Error in /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/dockertest/recipes/default.rb
==> default: ================================================================================
==> default:
==> default: Excon::Errors::SocketError
==> default: --------------------------
==> default: No such file or directory - connect(2) for /var/run/docker.sock (Errno::ENOENT)
==> default:
==> default: Cookbook Trace:
==> default: ---------------
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/docker/libraries/helpers_base.rb:44:in `api_version'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/docker/libraries/helpers_container.rb:61:in `default_network_mode'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/docker/libraries/docker_container.rb:65:in `block in <class:DockerContainer>'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/compat_resource/files/lib/chef_compat/copied_from_chef/chef/property.rb:631:in `instance_exec'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/compat_resource/files/lib/chef_compat/copied_from_chef/chef/property.rb:631:in `exec_in_resource'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/compat_resource/files/lib/chef_compat/copied_from_chef/chef/property.rb:649:in `stored_value_to_output'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/compat_resource/files/lib/chef_compat/copied_from_chef/chef/property.rb:365:in `get'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/compat_resource/files/lib/chef_compat/copied_from_chef/chef/property.rb:278:in `call'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/compat_resource/files/lib/chef_compat/copied_from_chef/chef/property.rb:521:in `network_mode'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/dockertest/recipes/default.rb:33:in `block (2 levels) in from_file'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/resource_builder.rb:90:in `instance_eval'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/resource_builder.rb:90:in `build'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/dockertest/recipes/default.rb:29:in `block in from_file'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/dockertest/recipes/default.rb:28:in `each'
==> default:   /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/dockertest/recipes/default.rb:28:in `from_file'
==> default:
==> default: Relevant File Content:
==> default: ----------------------
==> default: /tmp/vagrant-chef/2ebbc2eb034dc4b094e5584b61f5f05d/cookbooks/docker/libraries/helpers_base.rb:
==> default:
==> default:  37:        end
==> default:  38:
==> default:  39:        ################
==> default:  40:        # Helper methods
==> default:  41:        ################
==> default:  42:
==> default:  43:        def api_version
==> default:  44>>         @api_version ||= Docker.version(connection)['ApiVersion']
==> default:  45:        end
==> default:  46:
==> default:  47:        def connection
==> default:  48:          @connection ||= begin
==> default:  49:                            opts = {}
==> default:  50:                            opts[:read_timeout] = read_timeout if read_timeout
==> default:  51:                            opts[:write_timeout] = write_timeout if write_timeout
==> default:  52:
==> default:  53:                            if host =~ /^tcp:/
==> default:
==> default:
==> default: Running handlers:
==> default: [2016-06-13T14:09:28+00:00] ERROR: Running exception handlers
==> default: Running handlers complete
==> default: [2016-06-13T14:09:28+00:00] ERROR: Exception handlers complete
==> default: Chef Client failed. 0 resources updated in 01 seconds
==> default: [2016-06-13T14:09:28+00:00] FATAL: Stacktrace dumped to /var/chef/cache/chef-stacktrace.out
==> default: [2016-06-13T14:09:28+00:00] FATAL: Please provide the contents of the stacktrace.out file if you file a bug report
==> default: [2016-06-13T14:09:28+00:00] ERROR: No such file or directory - connect(2) for /var/run/docker.sock (Errno::ENOENT)
==> default: [2016-06-13T14:09:28+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)
Chef never successfully completed! Any errors should be visible in the
output above. Please fix your recipes so that they properly complete.

Workaround
Comment out the part of the recipe that performs docker related tasks, in this example:
imgvars.each do |dimages|
  docker_image dimages[:name] do
    tag dimages[:tag]
    action :pull
  end
end

contvars.each do |containers|
 docker_container containers[:name] do
   repo containers[:repo]
   tag containers[:tag]
   port containers[:port]
   network_mode containers[:network_mode]
   links containers[:link]
   volumes containers[:volumes]
   env containers[:env]
   restart_policy 'always'
 end
end

Allow docker to successfully install and then remove the comments and re-run chef-client.

After doing some further testing this only seems to occur when network_mode containers[:network_mode] is called. If I comment this from the above recipe everything seems to work fine.

Did you try setting :network_mode in default['docker']['containers'] for each container?
network_mode - Sets the networking mode for the container. One of bridge, host, container.
attributes/default.rb
default['docker']['images']    = [
  { :name=>"ghost", :tag=>"latest" },
  { :name=>"abiosoft/caddy", :tag=>"latest" }
]

default['docker']['containers'] = [
  { :name=>"ghost", :repo=>"ghost", :tag=>"latest", :port=>'2368:2368', :volumes=>"/data/ghost:/var/lib/ghost", :network_mode=>"host" },
  { :name=>"caddy", :repo=>"abiosoft/caddy", :tag=>"latest", :port=>['80:80', '443:443'], :volumes=>"/data/caddy/Caddyfile:/etc/Caddyfile", :network_mode=>"host" }
]

Confirmed api_version method requires docker to be running... it should never be used before then.
I also found another one of these in the docker_version method.

Is this still an issue? I don't see the method in your stack trace actually being used anywhere anymore.
$ grep -ir api_version *
CHANGELOG.md:- _ Adding @api_version instance variable
libraries/helpers_base.rb:      def api_version
libraries/helpers_base.rb:        @api_version ||= Docker.version(connection)['ApiVersion']
spec/spec_helper.rb:    attr_accessor :api_version
spec/spec_helper.rb:    @api_version = '1.18'
$

Closing this for now. Let me know if I need to re-open it.

authors: cpressland gmreburn someara

repo: chef-cookbooks/docker

license_source: bigquery


text:
yum_repository support for :enable and :disable repositories

I have a use case where I am using a third party cookbook which installs a rpm package via a yum repository on a RHEL 7 OS in AWS.  The problem I am experiencing is that the rpm package is in a RHEL optional repository that is not enabled by default.  I figured with this cookbook, I could own the AWS RHEL repository and create definitions for all of the various AWS RHEL repositories.
However instead of doing that, it may be an interesting use case (since this cookbook effectively owns the yum_repository namespace :smile: ) if yum_repository could have two actions to enable and disable a repository without having to own the entire repository definition.  I figured behind the scenes it would perform a yum-config-manager --enable or yum-config-manager --disable behind the scenes.
If this is something that you think would fit into the purview of this cookbook, I would be more than happy to whip something up.
Just wanted to run this past you before doing so.
Thanks!

+1

+1

+1

+1

+1

We've moved yum_repository to core chef. Any feature requests should be done there now

authors: acholt jmaziarz kierandelaney netflash st-isidore-de-seville stephan2012 tas50

repo: chef-cookbooks/yum


text:
Setting memory reservation

What are you trying to do and what's not working?
Im trying to set the memory reservation. You can set the ram with the option
--cram 4
I would like to set the memory reservation aswell. Maybe something like:
--cram_reservation 2
Can this be done?

I think so... Look for memoryAllocation at the Config Spec documentation. There are a few things we can set in that structure. What would be helpful?

Seems like the reservation can indeed be set with:
https://www.vmware.com/support/developer/converter-sdk/conv50_apireference/vim.ResourceAllocationInfo.html
Can it already be done with a featue of knife-vpshere (like something knife vsphere vm config set ... )?

Not as-is, as config doesn't support nested attributes.

This is now merged, can be closed?

Yes, thanks for the reminder!

authors: petericebear swalberg

repo: chef-partners/knife-vsphere


text:
Output from client.rb doesn't get logged to the log file.

Description
Output from client.rb doesn't get logged to the log file.
Clearly this isn't an easy fix, since the client.rb file contains details about what log file you'd like chef to log to.
What I'd like to do is setup a logger before client.rb is executed, and encourage people to use this logger for meaningful messages originating in client.rb.
The logger will be responsible for writing any messages to stdout as per the current behavior, as well as saving them.
Once the client.rb file is executed and the actual chef logger initialized, the saved log messages can be written to the log file normally.
Chef Version
$ chef --version
Chef Development Kit Version: 0.17.17
chef-client version: 12.13.37
delivery version: master (f68e5c5804cd7d8a76c69b926fbb261e1070751b)
berks version: 4.3.5
kitchen version: 1.11.1

Platform Version
Arch linux, centos7
Replication Case
$ cat client.rb
cookbook_path ['/home/zeal/dev/pub/chef-pub/cookbooks']

puts 'Inside client.rb'
$ cat cookbooks/sample/recipes/default.rb
log 'Inside chef' do
  level :warn
end
$ cat run_list.json
{
  "run_list": [
    "recipe[sample]"
  ]
}

Run chef with: chef-client -z -j run_list.json -c client.rb --logfile chef.log -F null
Client Output
$ chef-client -z -j run_list.json -c client.rb --logfile chef.log -F null
Inside client.rb

$ cat chef.log
# Logfile created on 2016-11-07 15:10:15 -0800 by logger.rb/44203
[2016-11-07T15:10:17-08:00] WARN: Inside chef
# Logfile created on 2016-11-07 15:10:15 -0800 by logger.rb/44203

I like this idea ( @zfjagann  and I discussed it before he wrote it up ) - the question I have is, does this require an RFC before he takes a whack at doing it? The border is fuzzy to me. CC @thommay  and @lamont-granquist

This seems broad enough that an RFC would be called for. Not sure if this is a thing we should encourage though, feels off putting for reasons I can't really quantify. Feels very foot-gun-y but that alone isn't a reason to not do it.

It seems reasonably fine as long as you're aware of the sharp edges.  I wonder if it would be possible to have a callback fire any time the logger was set via either the CLI or in config or anywhere else in the codebase to reinitialize the class?  Probably too much of a stretch goal, but it would make the code flow a little better.  If you get down to fighting with when the CLI options are parsed vs when the config file is parsed thats more when I'd throw the brakes on the idea -- right now I've had insufficient caffeine to recall if existing parse order supports this use case or not.

Created an RFC

authors: coderanger jaymzh lamont-granquist zfjagann

repo: chef/chef


text:
Lcg/ruby 2.2.3

Massive GC improvements in 2.2.x
Massive perf improvements in 2.2.x (chef spec tests are 2x faster than 2.1.6)

@chef/client-core @chef/client-maintainers 2.2.x has been out since December, lets start using it.

:+1:

IIRC Ruby has committed to following SemVer since 2.1 so we should be good here. From the announcement https://www.ruby-lang.org/en/news/2014/12/25/ruby-2-2-0-released/  there is a "Remove deprecated C APIs" entry. As long as everything builds we should be good though.
:+1:

So far I just did one build on my Ubuntu 14.04 workstation and it was fine.

We've also had ruby 2.2 in the chef/chef test matrix for a long time, and i've been using it on my workstation with rvm for chef dev stuff for months.

That's probably fine, I'd expect it to be a binary thing where gems w/ C exts either fail to build or they don't.

The Halite unit tests have also been running on 2.2 (and 2.1, 2.0) forever with no problems.

i am running 2.2 backed chef client for quiet sometime.. at least with 10 community cookbooks. so those works fine for sure.
https://github.com/ranjib/PiChef/releases/download/0.0.1/chef-12.2.1.deb

:rocket:

source: gharchive/pull-request

authors: coderanger danielsdeleo lamont-granquist mcquin ranjib

repo: chef/omnibus-chef


text:
the best model for EN

@chen0040  hi,
I want to train my own model in my EN data. Which your variants models u recommend me to use?

I trained rnn1 and he doesn't suit me
data has ~160k raws

license: MIT


text:
🛑 Homepage is down

In 330b596, Homepage (https://larrychen.tech) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Homepage is back up in d929b70.

repo: chendachao/uptime


text:
🛑 Homepage is down

In f10a0e1, Homepage (https://larrychen.tech) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Homepage is back up in 57a312d.


text:
🛑 Homepage is down

In 2887eba, Homepage (https://larrychen.tech) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Homepage is back up in c05d1ee.


text:
🛑 Homepage is down

In 8633db5, Homepage (https://larrychen.tech) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Homepage is back up in a334655.


text:
🛑 tt is down

In 4756370, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 500
Response time: 3427 ms

Resolved: tt is back up in 2a82318 after 24 minutes.

repo: chenhaonanniubi/upptime


text:
🛑 tt is down

In 651848e, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 500
Response time: 3153 ms

Resolved: tt is back up in 7ca3ba5 after 21 minutes.


text:
🛑 tt is down

In e9fdfe5, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 504
Response time: 10203 ms

Resolved: tt is back up in 7248501 after 21 minutes.


text:
Unable to implement in my project

I want to implement the same in my project but I don't understand how to import the plugin.
Please guide me

Hi there - the usage directions in the README should be pretty straightforward.
Take a look in the dist folder. There are two files you need:

checkbox-dropdown-plugin.min.js
checkbox-dropdown-style.css

Make sure these files are imported into your project.
You will also need to import the bootstrap library into your project. Take a look at their website if you arent sure how to do that.

license: mit


text:
Use :has() selector for selected Tablist__Tab

https://github.com/chharvey/xmeter/blob/4d0a6ac72f6dcd4348418ecc7ae3b749e674b169/css/src/_o-Tablist.less#L120
Instead of using javascript to add class .o-Tablist__Tab--js-selected, use the css :has() pseudo-class to select a tab that contains a checked input element. Similarly, use the same concept for a selected tab’s corresponding panel.
.o-Tablist__Tab:has(.o-Tablist__Check:checked) {}
.o-Tablist__Tab:has(.o-Tablist__Check:checked) + .o-Tablist__Panel {}

note that at the time of this writing, browser support is null: http://caniuse.com/#feat=css-has

closing, as the Tablist Object is moving to https://github.com/chharvey/aria-patterns with [role="tablist"]

repo: chharvey/xmeter


text:
Added proxy status and onStart and onStop event listener

When the proxy is run through an async process, we need notifications when the server has completely started and stopped along with a status.

@chhsiao90 Is there a way to push this to a new release on maven central?

I will publish a new release asap

I just published 1.2.1, it should be available soon (maybe hours later) in maven central.

Thanks!


text:
Last-Modified header is not reliable

Race conditions, where several triggers flicker in one second, are possible.
We should consider moving to ETag.

v1.2.1 and up support ETags

repo: chillum/jsonmon


text:
Denote PL0 PAUSER in the firmware manifest

The PL0 PAUSER (described in https://github.com/chipsalliance/caliptra-sw/pull/103) should be communicated in the firmware manifest.
This needs to be updated in the firmware manifest spec and implementation.

This is done in #103

repo: chipsalliance/caliptra-sw


text:
GaloisLFSR: Note on taps === LSb-to-MSb polynomial

Also fixed a README link issue in CONTRIBUTING.md

Related Issue: https://github.com/chipsalliance/chisel3/issues/2483

Due to shifting towards LSb, the polynomials are currently implemented with a LSb to MSb notation
Most scrambling polynomials are notated MSb-to-LSb
This can lead to user trouble.
It's still confusing even with the note, but I hope it helps. (The notation for these polynomials is a mess everywhere)

I don't think anyone cares about this quirk in the notation for the FibonacciLFSR.
AFAIW, the GaloisLFSR is the only one used to implement specific scrambling polynomials.
A more proper fix requires left shifting since the Seq of Bools is MSb to LSb (need to shift towards MSb).
I triple-checked this and I'm ~95% sure I have MSb to LSb the right way around. I still appreciate someone quadruple-check ing me before merge.
Contributor Checklist

[ ] Did you add Scaladoc to every public function/method?
[ ] Did you add at least one test demonstrating the PR?
[ ] Did you delete any extraneous printlns/debugging code?
[ ] Did you specify the type of improvement?
[ ] Did you add appropriate documentation in docs/src?
[ ] Did you state the API impact?
[ ] Did you specify the code generation impact?
[ ] Did you request a desired merge strategy?
[ ] Did you add text to be included in the Release Notes for this change?

Type of Improvement

API Impact

Backend Code Generation Impact

Desired Merge Strategy

Release Notes

Reviewer Checklist (only modified by reviewer)

[ ] Did you add the appropriate labels?
[ ] Did you mark the proper milestone (Bug fix: 3.4.x, [small] API extension: 3.5.x, API modification or big change: 3.6.0)?
[ ] Did you review?
[ ] Did you check whether all relevant Contributor checkboxes have been checked?
[ ] Did you do one of the following when ready to merge:

[ ] Squash: You/ the contributor Enable auto-merge (squash), clean up the commit message, and label with Please Merge.
[ ] Merge: Ensure that contributor has cleaned up their commit history, then merge with Create a merge commit.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I'll ping @seldridge.
My understanding summarized: the shift for the multiplication of the Galois Polynomial is towards the msb when using MSb-to-LSb notation.

Yeah, I think that's a good practice to get into with PRs. I figured I'd fix that link since it's a small change, but I've now separated the two changes. That PR is now #2596
The main goal of this PR is to add some a note to help #2483 while I find time to work on a more permanent patch.

authors: CLAassistant metzkorn


text:
Making contact form more attractive.

Hey @chiraag-kakar @harshita2216 @amanjha8100 ,since the contact page looks too simple and old school, can I make it more attractive so that UI will look much better than now? I'm GSSOC'21 participant can I work on this?

One pr is under consideration for that issue, if it is not solved ,we will assign it to you.

Ok. Thanks.

One pr is under consideration for that issue, if it is not solved ,we will assign it to you.

@amanjha8100  Is pr got merged that you were talking about?

@dhananjayd1729 The PRs will be merged by the project admin. We will create the issue again if the PR under review is not accepted.

repo: chiraag-kakar/sharenlearn


text:
Missing JS files in package

Hi! the download of master in the zip file, dosen't has the javascript files under the folder "dist" (which is missing)

Hello,
You're right. The dist folder is available in the dist.tar file that's attached to every release.

If someone finds this thread and is looking for dist.tar.
Nowadays, in a non-module environment, you can download and use these:
CSS: https://cdn.jsdelivr.net/npm/flatpickr/dist/flatpickr.min.css
JS: https://cdn.jsdelivr.net/npm/flatpickr

authors: Bert-de-Jong chmln vousys

repo: chmln/flatpickr


text:
minDate disables wrong dates in other months, today is highlighted in other month

Expected Behavior
minDate should disable dates before the specified one.
Current Behavior
It disables random dates in other months in addition to that. See January for example, 1-18 is disabled.
Steps to Reproduce (for bugs)
https://jsfiddle.net/L93kuaeo/1/
Your Environment

Version used: 2.2.3. From the CDN.
Browser Name and version: Chrome 55.
Operating System and version (desktop or mobile): Linux (desktop).

I've added another issue I have noticed just now with the default theme, the 19 of January is highlighted as today and it is probably the same issue that disables wrong dates.

Yep, same here..

Hello,
Already fixed as of https://github.com/chmln/flatpickr/commit/196401b942920e5c7cb82e89a1f8504e20bbe84d
Feel free to roll back to previous version, and a new release will be coming soon

authors: chmln ernestoalejo yoyek


text:
(python) [URGENT] 3.9.0-b5 is installed through 'upgrade all', despite --pre not being set

For some reason, 3.8.5 appeared in the python3 package but not in the python package. I ran an upgrade all operation.
'python' initially stayed at 3.8.4
'python3' then runs its upgrade operation, upgrading to 3.8.5, its dependency is 'python' and so it tries to match version numbers, but since python did not have 3.8.5, it instead then installed 3.9.0-b5 as well.
I ended up with Python 3.8.4, 3.8.5, and 3.9.0-b5 install data in my Windows Apps & features section. It's a mess. This needs to be fixed ASAP.
Steps to replicate, starting with no python packages installed:

choco install python --version 3.8.4 -y
choco upgrade all -y OR choco upgrade python python3 -y

Someone in chat just replied to me that the metadata for python --version 3.8.5 exists: https://chocolatey.org/packages/python/3.8.5
But it does not show up in the Version History:

I have no clue what's going on. Someone in chat was unable to reproduce it. I'm just going to close this.
3.8.5 still does not appear in the version history of these packages on the site, however. I have to think it's somehow related. This seems like an issue with the site itself.

Someone in chat just replied to me that the metadata for python --version 3.8.5 exists:

That someone would be me, but as I mentioned on gitter there isn't anything wrong from a package perspective, but could be a bug in the choco client itself, or an intermittent error.

Well, this might be a clue as to why it happened. Or this might just be a red herring.
WARNING: Provided python InstallDir was ignored by the python installer
WARNING: Its probable that you had pre-existing python installation
WARNING: Installed to: C:\Program Files (x86)\Streamlink\Python
WARNING: Installation folder is not the default. Not changing permissions. Please ensure your installation is secure.

(streamlink is installed through chocolatey as well)

authors: AdmiringWorm ImportTaste


text:
Add option to sort available repository packages by release date to match the existing Chocolatey http interface options

Propose a feature request to match the online web portal for the main Chocolatey package repository (and other chocopackages web repos) and add a 3rd option to the following UI sort list:
https://1drv.ms/i/s!AhSqjTYUuHPKgb95yiPGE6gbVFJaLg
In addition to A-Z alphabetical, and by "popularity" # of installs, also supply "Newest" as an option.  This would add incredibly invaluable usability to this already excellent GUI, as the newest updated packages I would assert is just as valuable as overall total installations.
Thanks!

This would be great to have!!!  +1 for this request

Hi @ssnyder @CollinChaffin is this feature still open to take on? I'm up for it :)

authors: CollinChaffin abc516 ssnyder

repo: chocolatey/ChocolateyGUI


text:
Added chocolateyUninstall.ps1 file for Firefox

I made the chocolateyUninstall.ps1 for Firefox choco package, please review. I tested the package with the new file using my local folder/repository and the uninstalling pocedure is now silent. I also tested through Ansible (that needs Firefox to be silently uninstalled otherwise it does not work) and it works too.

In the registry I could find only the path to the firefox.exe file. How would I get to the helper.txt path through the registry?

Weird, I am going to call no dice on this one.

The only reason Chocolatey can't automatically uninstall it is that it is not able to detect the installer type (yet).

So this is interesting...

FirefoxInstall.exe: installer: 7-Zip(1.0); overlay: 7-zip Installer data(-)

Unzipped there is a setup file:

/FirefoxInstall/setup.exe: installer: Nullsoft Scriptable Install System

Yep, I found the location of the Uninstall string in the registry. Sorry but I just started with powershell, so.. how would I change my script to get to uninstall location from the registry?

@sparkvilla first choco upgrade chocolatey -pre and then run choco new firefox and take a look at the uninstall script that is generated to adjust this one.

@ferventcoder I have used the template you suggested to make up a second version of chocolateyuninstall.ps1 file. It works but Firefox is not uninstalled silently (things that was achieved with the script of this pull request). Something wrong. How can I upload the new script so that you can have a look at it? or, Shell I open a new pull request?

The script generated for uninstall is a template result which still requires some editing. The idea is to look at that to get an idea how you can adjust this PR to use the registry instead. You still need to make the adjustments to it for the silent arguments.

You can always paste script text into these messages as another alternative way of looking at what you have questions about.

` # If this is an exe, change installerType and silentArgs
$ErrorActionPreference = 'Stop'; # stop on all errors
$packageName = 'firefox'
$softwareName = 'Mozilla Firefox' #part or all of the Display Name as you see it in Programs and Features. It should be enough to be unique
$installerType = 'EXE'
#$silentArgs = '/qn /norestart'
https://msdn.microsoft.com/en-us/library/aa376931(v=vs.85).aspx
$validExitCodes = @(0, 3010, 1605, 1614, 1641)
if ($installerType -ne 'MSI') {
The below is somewhat naive and built for EXE installers
Uncomment matching EXE type (sorted by most to least common)
$silentArgs = '/S'           # NSIS
#$silentArgs = '/VERYSILENT /SUPPRESSMSGBOXES /NORESTART /SP-' # Inno Setup
#$silentArgs = '/s'           # InstallShield
#$silentArgs = '/s /v"/qn"' # InstallShield with MSI
#$silentArgs = '/s'           # Wise InstallMaster
#$silentArgs = '-s'           # Squirrel
#$silentArgs = '-q'           # Install4j
#$silentArgs = '-s -u'        # Ghost
Note that some installers, in addition to the silentArgs above, may also need assistance of AHK to achieve silence.
$silentArgs = 'none'             # none; make silent with input macro script like AutoHotKey (AHK)
#       https://chocolatey.org/packages/autohotkey.portable
$validExitCodes = @(0)
}
$uninstalled = $false
$local_key     = 'HKCU:\Software\Microsoft\Windows\CurrentVersion\Uninstall*'
$machine_key   = 'HKLM:\SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall*'
$machine_key6432 = 'HKLM:\SOFTWARE\Wow6432Node\Microsoft\Windows\CurrentVersion\Uninstall*'
$key = Get-ItemProperty -Path @($machine_key6432,$machine_key, $local_key)                         -ErrorAction SilentlyContinue
| ? { $_.DisplayName -like "$softwareName" }
if ($key.Count -eq 1) {
$key | % {
$file = "$($_.UninstallString)"
if ($installerType -eq 'MSI') {
  # The Product Code GUID is all that should be passed for MSI, and very
  # FIRST, because it comes directly after /x, which is already set in the
  # Uninstall-ChocolateyPackage msiargs (facepalm).
  $silentArgs = "$($_.PSChildName) $silentArgs"

  # Don't pass anything for file, it is ignored for msi (facepalm number 2)
  # Alternatively if you need to pass a path to an msi, determine that and
  # use it instead of the above in silentArgs, still very first
  $file = ''
}

Uninstall-ChocolateyPackage -PackageName $packageName `
                            -FileType $installerType `
                            -SilentArgs "$silentArgs" `
                            -ValidExitCodes $validExitCodes `
                            -File "$file"

}
} elseif ($key.Count -eq 0) {
Write-Warning "$packageName has already been uninstalled by other means."
} elseif ($key.Count -gt 1) {
Write-Warning "$key.Count matches found!"
Write-Warning "To prevent accidental data loss, no programs will be uninstalled."
Write-Warning "Please alert package maintainer the following keys were matched:"
$key | % {Write-Warning "- $_.DisplayName"}
} `

@sparkvilla @ferventcoder I just wanted to touch base on this...
Is there still work required here?

@sparkvilla pinging again about this...
Is there still work required?

@gep13 @ferventcoder  @sparkvilla  Taking a look at this now will be back with a PR soon as work permits.

I have found a solution for @sparkvilla problem; This borrows some code from the original install file. Created a new PR #262

Closing this in favour of new PR: https://github.com/chocolatey/chocolatey-coreteampackages/pull/262

repo: chocolatey/chocolatey-coreteampackages


text:
Added vcredist2015 package

This adds Visual C++ Redistributable Packages for Visual Studio 2015
Newer version of #75

@pocki thank you for doing this!  it makes a lot of sense for this package to reside in this repository!

Yes, it makes more sense. Thanks for contacting me and give me the hint to add this package here.


text:
Document version number normalization

What New Or Updated Would You Like To See?
Add a page to the documentation explaining Version number normalization.
Why Is It Needed?
With the upgrade to V2, version numbers are normalized. Currently we only explain this on the migration guide, but we should explain this on it's own page as it will exist beyond the initial migration.
Additional Context?
https://github.com/chocolatey/docs/pull/764#discussion_r1268279821

We should update the link mentioned in that comment, once we create the page.

authors: corbob pauby

repo: chocolatey/docs


text:
(GH-100) Fix Choco package source folder copy criteria

Changes the copy path to copy from the files folder to the root of the destination.
Fixes #100

@pauby thanks for fixing this!

repo: chocolatey/package-verifier


text:
Integrate new atom mappers

We should be able to integrate other atom mappers such as the TimeMachine mapper or similar, that we could use in perses for doing the mapping.

Wwe could also try https://github.com/michellab/BioSimSpace/blob/devel/python/BioSimSpace/Align/_align.py#L719

Time machine mapper is in https://github.com/proteneer/timemachine/blob/master/timemachine/fe/atom_mapping.py

cc'ing @richardjgowers

@ijpulidos you should probably think about Licenses if you're looking in BSS.  And yes if you refactor so that the AtomMapping is an input then it will be relative easy to get this from Lomap/Kartograf/etc, I think wrapping up TimeMachine's approach is interesting, it's a more modern look at what rdFMCS can do compared to other options.

authors: IAlibay ijpulidos richardjgowers

repo: choderalab/perses


text:
Checkpointing feature and pH fix [WIP]

This PR adds a checkpointing mechanism. This involves serialization of proton drives into xml.
It also includes a fixed version of the pH correction feature which I ran into during serialization. It contained a bug that resulted in NaN values that weren't picked up.
Tasks:

[x] Serialization of _TitratableResidue data structure
[x] Serializaton of _TitrationState data structure
[x] Deserialization of _TitratableResidue data structure
[x] "Running the code" tests
[x] Deserialization of _TitrationState data structure
[ ] (De)serialization of protondrive as a whole, including properties, and titration states and residues.
[ ] Unit tests for before/after serialization consistency of objects, ensuring that the simulation properly gets resumed

Also includes

[x] bugfix for pH code that is necessary to get deserialization working.

Still need to settle on the appropriate precision for float serialization.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0cb18e5). Click here to learn what that means.
The diff coverage is 91.46%.

@@            Coverage Diff            @@
##             master     #112   +/-   ##
=========================================
  Coverage          ?   64.79%
=========================================
  Files             ?       37
  Lines             ?     4570
  Branches          ?        0
=========================================
  Hits              ?     2961
  Misses            ?     1609
  Partials          ?        0

Impacted Files
Coverage Δ

protons/app/titrationreporter.py
87.37% <ø> (ø)

protons/tests/test_pkas.py
100% <100%> (ø)

protons/tests/test_explicit.py
97.05% <100%> (ø)

protons/app/driver.py
86.26% <87.85%> (ø)

protons/app/pka.py
98.07% <95.65%> (ø)

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 0cb18e5...ab5a009. Read the comment docs.

Things seem to be working. What I need to do tomorrow is make a minor adjustment to the ConstantPHSimulation/Calibration classes.  Using that, I will make sure tomorrow that I can successfully start, end, and resume a simulation. Should have this PR wrapped up if that works.
In that case, the long simulations we want to run will be up and running by end of day tomorrow.

authors: bas-rustenburg codecov-io

repo: choderalab/protons


text:
Bump deps

Ignore font stuff, this should only change package.json

Remove sass-loader and node-sass from package json (currently breaking stuff)
npm outdated --parseable=true --depth=0 | awk -F'[:|@]' '{print $2}'
Update result

@psimyn Thanks! Was just about to bump mongoose up as you suggested, but you beat me to it!

authors: choonkending psimyn

repo: choonkending/react-webpack-node


text:
Added test for one arg

#1 Adds test for functions with one parameter.

Alright I changed it.

Thanks, you forgot to change it on the next line :)

My bad. Now it should be right.

authors: FelixDavidson chrfrasco


text:
Randomize button

A randomize button for colors & settings

Closing. Evaluated but not really possible since streamlit doesn't allow buttons within a form, and having two parallel mechanisms would be a ton of effort.

repo: chrieke/prettymapp


text:
no audio no sound,probook 4540s,big Sur

hi Chris
I installed Big Sur(11,2,1) using your guide.
every thing work perfect except aud
EFI.zip
io

![Screen Shot 1400-08-11 at 14 03 39](https://user-images.githubusercontent.com/69670261/139830859-2bd8f33e-0de9-4d0e-9890-b17f5f85fbb0.png

)

hi Chris
I installed Big Sur(11,2,1) using your guide.
every thing work perfect except aud
EFI.zip
io

Solved
I change layout ID from 12 to 13 and it worked.
please change layout id for Probook 4540s to 13 in for guide and package.

And another one:
for Probook 4540s bios version F.68 Rev.A works perfect.


text:
🛑 The Holding Company is down

In b7531f4, The Holding Company (https://holdingco.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: The Holding Company is back up in 752e6f2 after 29 minutes.

repo: chrisbergeron/upptime


text:
added the font Lotion : https://gitlab.com/nefertiti/lotion-dist

https://font.nina.coffee/
I'll email mx. belikova if this font goes live to see if they have a better description. I just copy-pasted the existing description on their site.
Rendered their name in lower case since they do that on their website.

How unique! Thanks so much! Looks good to me. (But yeah, a better description at some point would be great).

authors: aljedaxi chriscoyier

repo: chriscoyier/coding-fonts


text:
Mutually exclusive radio buttons require double click on Mac to trigger

Mutually exclusive radio option buttons don't trigger when clicked - they need to be double clicked (weird, I know) to work.
Mac High Sierra, Python 3.6.4 python.org,  Gooey Version 1.0.0
Expected Behaviour
When I create two mutual exclusive options, the Gooey UI correctly displays two radio buttons.  I expect that clicking on a radio button will select it and deselect the other (mutually exclusive behaviour).
Actual Behaviour
Nothing happens when I click on either radio button.  I need to double click on a radio button for the state to change.  The mutual exclusive behaviour works ok - its the need to double click that is not right.
Actual full runnable example:
    import argparse
    from gooey import Gooey
    from gooey import GooeyParser

    @Gooey
    def main():
        parser = GooeyParser(description="Convert python to rpn for hp41s/free42/dm42")

        group = parser.add_mutually_exclusive_group()
        group.add_argument("-v", "--verbose", action="store_true")
        group.add_argument("-q", "--quiet", action="store_true")

        parser.add_argument('filename', help="name of the file to process", widget='FileChooser')
        parser.add_argument("-n", "--nolinenum", action='store_true', help="do not generate line numbers")
        parser.add_argument("-c", "--comments", action='store_true', help="generate commants")
        args = parser.parse_args()

        def run():
            print('work done here...')

            if args.quiet:
                print(rpn)
            elif args.verbose:
                print('source is', source)
                print('rpn is', rpn)
            else:
                line_count = len(rpn.split('\n'))
                print(f'Generated {line_count} lines.')
                print(rpn)

        run()

    main()

Perhaps not a Gooey bug otherwise it would have been noticed. Maybe a weird interaction with Mac wxPython?  Installing Gooey installed wxPython (4.0.0b1). Apologies if I'm doing something wrong with driving argparse, but it seems to work ok on the command line.

Perhaps not a Gooey bug otherwise it would have been noticed.

Ha! If only...
Give this branch a quick run and let me know if it fixes your issue? It was an issue with the select/deselect logic when the Mutex groups were in an optional section.
pip install -U git+https://github.com/chriskiehl/Gooey/@issue-272-optional-radio-group-behavior

authors: abulka chriskiehl

repo: chriskiehl/Gooey


text:
Display screen messes up ASCII tables.

Hello everyone. First of all, thanks to chris for his great work with Gooey.
I've been using Gooey to create a simple GUI for an script that does statistical calculation.
As an output, I need to show a table and the correspondient results of the rest of the calculations.
However, the display screen messes up the ASCII tables, not allowing them to be properly configured. Showing me this:

I've tried using Tabulate & PrettyTable, however, neither work. They do work on the CLI version though.
Is there any way to fix this? Is there any other library that allows me to show a table that works with Gooey?
Thank you for your time.

It can be done!
I just need to add an option to allow different fonts in that box. WxPython's default font uses a proportional typeface rather than a mono-space one (hence it breaking your ascii table formatting).
Should be a quick fix!

That explains it!
Thanks a lot for the quick response, and help.
I'll be waiting patiently for the fix.

Alright! Done! 5 minute hack job before I step out for some lunch. :)
The decorator now supports an additional boolean arg called monospace_display. Set that to true to enable mono-spaced output in the display panel.
e.g.
Gooey(program_name="blah", monospace_display=True)

It should support arbitrary fonts, but.. that'll have to come later..!

It works perfectly! Thanks a lot for your help. You rock!

authors: Datkros chriskiehl


text:
Silence Python 3.8 runtime warning

This patch removes the buffer size paraemter in calls to subprocess.Popen() so that the default value is assumed. This silences this Python 3.8 runtime warning:
RuntimeWarning: line buffering (buffering=1) isn't supported in binary mode, the default buffer size will be used```

Checkout CONTRIBUTING.md. These should target the release branch.

authors: bje- chriskiehl


text:
tests are missing from pypi sdist tarball

Hi! I'm packaging python-identify for Arch Linux.
We usually use the pypi sdist tarballs for packaging, as pypi.org offers a unique naming interface for tarballs (which is important in a build environment).
It would be great, if the tests could be included in an upcoming release of the sdist on pypi.org, as they ensure, that users can run tests against their environment before using a package.

hi, please use the github source tgz -- thanks!

authors: asottile dvzrv

repo: chriskuehl/identify


text:
Handle indented sections

According to the ConfigParser docs, sections can be indented: https://docs.python.org/3/library/configparser.html
The code at the moment will treat an indented section as a continued line from the previous key value pair.

Thanks so much for the patch.  There were a couple of other minor bugs that I found during testing so I pushed my own changes as well.  You'll find the update in version configparser-0.1.4.
Again, thank you so much.

This is not fixed till now, 22-July-2020 , or this bug came again in the configparser?

Vickygum, it past all my tests and appears to be working fine.  Could you give me an example of a configuration where the parsing varies between python and ruby?

authors: Vickygum ccooke chrislee35


text:
roundrobin  instruction can support N<->M dataset shards

add new NetworkType AllShardTOAllShard
and
roundrobin instruction can support N<->M dataset shards

Codecov Report

Merging #78 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #78   +/-   ##
=======================================
  Coverage   55.37%   55.37%
=======================================
  Files           9        9
  Lines         186      186
=======================================
  Hits          103      103
  Misses         81       81
  Partials        2        2

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 7fd7d7c...d575a1a. Read the comment docs.

repo: chrislusf/gleam


text:
no fid

yes

There are no file id any more when uploading to filer. File id is a concept of object store, not for file.

authors: chengshihard chrislusf

repo: chrislusf/seaweedfs


text:
doesn't work when installed with yarn

Strangely none of the tests run and no browsers open when this repo is installed using yarn. When installing using npm it opens the browser and runs two tests that fail (i think on purpose) and then it just sits there.
❯ yarn test
yarn run v1.9.4
$ wdio
2018-09-14T02:30:20.721Z DEBUG wdio-config: wdio-sync found, running tests synchronous
2018-09-14T02:30:21.354Z DEBUG wdio-cli:run: Run suite with config /Users/dgreene/playground/webdriverio-performance-testing/wdio.conf.js and params { _: [],
  '$0':
   '/Users/dgreene/medallia/playground/webdriverio-performance-testing/node_modules/.bin/wdio' }

 RUNNING  0-0 in chrome - /tests/pageWeight.e2e.js
... 1 pending tests

Test Suites:	 0 passed, 2 total (0% completed)
Time:		 🕐  13.78s

 RUNNING  0-0 in chrome - /tests/pageWeight.e2e.js
 RUNNING  0-1 in chrome - /tests/scriptBlocking.e2e.js

Test Suites:	 0 passed, 2 total (0% completed)
Time:		 🕙  14.61s

 RUNNING  0-1 in chrome - /tests/scriptBlocking.e2e.js

Test Suites:	 0 passed, 1 failed, 2 total (50% completed)
Time:		 🕚  14.64s

Test Suites:	 0 passed, 2 failed, 2 total (100% completed)
Time:		 🕛  14.64s

Test Suites:	 0 passed, 2 failed, 2 total (100% completed)
Time:		 🕐  14.64s

Test Suites:	 0 passed, 2 failed, 2 total (100% completed)
Time:		 🕑  14.74s

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
previous command took 16s

I'm guessing this is a bug with v5 expecting modules to be in a certain location, but the lack of any errors getting logged is suggesting nothing is missing or in an unexpected location.

@dylang have you tried to run it with NPM?

authors: christian-bromann dylang


text:
RG10 Max Support

Hi - I see you support the RGB10, do you have any plans to support the RGB10 Max?
Thanks
Rik

If I receive a requested dev unit, I may look into this.  However, RetroOZ is a fork of ArkOS and functions very similarly.  You can find more information about it here: https://forum.odroid.com/viewtopic.php?f=193&t=42007&sid=5048586e98bfbb3c1cfbca2d7dc21af8.  Be aware that support questions for RetroOZ used with the RGB10Max should not be asked on the odroid forum.  Your best means for support will be via the RGB10Max channel on the RGHandheld discord.  The link to join the discord is available here

repo: christianhaitian/arkos


text:
Building instructions: most RG503 links are broken

In the "Building" document - https://github.com/christianhaitian/arkos/wiki/Building - the following links (related to the RG503) are broken:

kernel; https://github.com/christianhaitian/RG503_kernel.git
kernel building instructions; https://github.com/christianhaitian/RG503_kernel/blob/main/README.md
u-boot; https://github.com/christianhaitian/RG503_uboot

The broken links are due to the fact that neither the https://github.com/christianhaitian/RG503_kernel nor https://github.com/christianhaitian/RG503_uboot repositories exist.
Would it be possible for those repositories to be uploaded to github? Is there a version of the RG503 building instructions that doesn't require those repositories?
Thanks.

Yea those are forks from Anbernic's repo and they still haven't made them public for some reason.  I've went ahead and copied the forks to new repositories and made them public so they're available now.  The links in the build instructions have been updated to reflect this change.

Thank you, Christian. It's much appreciated.

authors: christianhaitian mjkaye


text:
Documentation

Hi there,
We would be interested by this module and saw it has been dowloaded several times already.
Could you tell us where to find the documentation ?
Many thanks,
Best,
Sylvain

Hi Sylvain the module was written just for the mongodb driver but was then extended by others to allow for more general kerberos usage. However I've no experience in using it outside of the node driver. you might be able to find more of what you need here
https://www.npmjs.com/package/passport-negotiate
As they've used the kerberos module to implement http auth it seems. If you do use the module I would love any contributions about how to use it for whatever use case you are planning.

authors: SylvainEstevez christkv

repo: christkv/kerberos


text:
[Feature Request]: Do not automatically persist chromadb.db.duckdb.PersistentDuckDB

Describe the problem
Once I've populated a collection I would like to be able to load it in a read-only fashion such that it is not auto-persisted on exit.
Describe the proposed solution
Add a flag in Settings to disable auto-persist.
Alternatives considered
Extend the class and nerf methods but I don't think this would work
Importance
nice to have
Additional Information
chromadb==0.3.25

Chroma with 0.4 has now dropped duckdb. Closing this issue.

authors: jeffchuber riklopfer

repo: chroma-core/chroma


text:
Screencast flow not working as expected

What versions are you running?
$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.7.4
$ google-chrome --version
Version 94.0.4606.61 (Official Build) (x86_64)
$ go version
go version go1.17 darwin/amd64

What did you do? Include clear steps.
package main

import (
	"context"
	"encoding/base64"
	"fmt"
	"image/png"
	"log"
	"os"
	"strings"
	"time"
	"github.com/chromedp/cdproto/cdp"
	"github.com/chromedp/cdproto/page"
	"github.com/chromedp/chromedp"
)

var webpage = "https://github.com"

func main() {
	opts := []chromedp.ExecAllocatorOption{
		chromedp.NoFirstRun,
		chromedp.NoDefaultBrowserCheck,
	}

	// Init Browser (not headless)
	initCtx, _ := chromedp.NewExecAllocator(context.Background(), opts...)
	ctx, cancel := chromedp.NewContext(initCtx)
	defer cancel()

	chromedp.ListenTarget(ctx, func(ev interface{}) {
		switch ev := ev.(type) {
		case *page.EventScreencastFrame:
			c := chromedp.FromContext(ctx)
			err := page.ScreencastFrameAck(ev.SessionID).Do(cdp.WithExecutor(ctx, c.Target))
			if err != nil {
				log.Printf("ScreencastFrameAck: %v", err)
			}

			// Filename
			name := fmt.Sprintf("frame-%d.png", ev.Metadata.Timestamp.Time().Unix())

			// Save the screenshot
			go func() {
				p, err := png.Decode(base64.NewDecoder(base64.StdEncoding, strings.NewReader(ev.Data)))
				if err != nil {
					fmt.Println(err)
				}

				f, err := os.Create(name)
				if err != nil {
					log.Println(err)
				}
				defer f.Close()

				if err = png.Encode(f, p); err != nil {
					log.Printf("failed to encode: %v", err)
				}
			}()

		}
	})

	// navigate
	if err := chromedp.Run(ctx,
		page.Enable(),
		page.StartScreencast().WithFormat(page.ScreencastFormatPng),
		chromedp.Navigate(webpage),
		chromedp.Sleep(10*time.Second),
		page.StopScreencast(),
		chromedp.Stop(),
	); err != nil {
		fmt.Errorf("could not execute screencast session: %v", err)
	}
}

What did you expect to see?
I know the github.com page contains an animation. With the code above I'm expecting to receive multiple EventScreencastFrame events during the 10 seconds session.
What did you see instead?
I'm only receiving 1 event.
I checked chromedp/examples and I googled it but I could not find any example using the screencast functions from this project.
I tried another example (using another cdp library and the flow was working as expected. Meaning I don't think the issue is related to my setup).
I'm wondering if the way I'm acknowledging the screencast frame could be the issue.

You should execute err := page.ScreencastFrameAck(ev.SessionID).Do(cdp.WithExecutor(ctx, c.Target)) in a separate goroutine.  See the comment of chromedp.ListenTarget:
https://github.com/chromedp/chromedp/blob/3f81ff8463df931d8f1c2574610b62660c06a01d/chromedp.go#L676-L680
Change the code like this:
    chromedp.ListenTarget(ctx, func(ev interface{}) {
        switch ev := ev.(type) {
        case *page.EventScreencastFrame:
-           c := chromedp.FromContext(ctx)
-           err := page.ScreencastFrameAck(ev.SessionID).Do(cdp.WithExecutor(ctx, c.Target))
-           if err != nil {
-               log.Printf("ScreencastFrameAck: %v", err)
-           }
-
-           // Filename
-           name := fmt.Sprintf("frame-%d.png", ev.Metadata.Timestamp.Time().Unix())
-
            // Save the screenshot
            go func() {
+               c := chromedp.FromContext(ctx)
+               err := page.ScreencastFrameAck(ev.SessionID).Do(cdp.WithExecutor(ctx, c.Target))
+               if err != nil {
+                   log.Printf("ScreencastFrameAck: %v", err)
+               }
+
+               // Filename
+               name := fmt.Sprintf("frame-%d.png", ev.Metadata.Timestamp.Time().Unix())
                p, err := png.Decode(base64.NewDecoder(base64.StdEncoding, strings.NewReader(ev.Data)))
                if err != nil {
                    fmt.Println(err)

That was it! Thanks @ZekeLu !

authors: ZekeLu catalan-adobe catalandavid

repo: chromedp/chromedp


text:
PhotoViewer for Google Photos -  Photo location/source

I'm a  PhotoViewer for Google Photos user. I use ROKU to view Google Photos thru PhotoViewer.    I noticed that after I log off my Google account,  my Google photos remain viewable on PhotoViewer.   Question 1:  Why is this?
Question 2:  How can I discontinue/stop PhotoViewer access to my Google Photo library ?

When you grant the Roku application access, this is outside your current logged in web session. To unregister your account from the device:
Main screen > Settings (Gear Icon) > Unregister account from device. This will remove all tokens and account from the device.


text:
Небольшой дожд…

Видимо где-то закралась ошибка с > и >=
(если там, конечно, было не слово «дождик») )

Там может быть например "Небольшой дождь со снегом" или ещё что-нибудь.
Какой это был город?

Москва на 3 марта

Да, там как раз "Небольшой дождь со снегом". Видно по картинке + я проверил, какие там данные реально стоят. Но я заметил другую ошибку: на английском и немецком указан град (sleet на английском, Graupel на немецком, grésil на французском)

А тут?
Локация Минск

Это всё же баг.
Спасибо @alex-liberty и @romanesko
Надо фиксить

authors: alex-liberty chubin romanesko


text:
wttr.in server is down

Hi, I tried accessing wttr.in more times today, but I received "The web page at https://wttr.in/ may be temporarily down or has been permanently moved to a new web address." error. Can you please check why the server is down? And could you tell me what is the uptime?

same. i couldnt use curl to get weather info.

it's down :(
Second time this month... what's going on?


text:
IAP sample crash on android 5.x devices

Probably just need to upgrade IAP with 1.1.4 version
http://discuss.cocos2d-x.org/t/iap-sample-run-crash/22483/1

user feedback:
I updated sdkbox to version 1.1.5 and intergrate iap plugin into my project. It work on my android device

so, please close the issue.

authors: darkdukey yinjimmy

repo: chukong/sdkbox-samples


text:
Alignment issues when indicators are not all the same size.

I have a list of indicators, where all aren't exactly the same size. There appears to be some distortion in the line connector as shown below:

Any ideas how to resolve?

resolved.
Had to set the space parameter in the ConnectorThemeData to the size of my biggest Indicator.

repo: chulwoo-park/timelines


text:
🛑 91家纺 is down

In 2e8a26d, 91家纺 (https://www.91jf.com/) was down:

HTTP code: 403
Response time: 1242 ms

Resolved: 找家纺网 is back up in 059bcb8 after .

repo: chwang-team/status-hao


text:
Await for release 1.2.0

Hi Simon, any chance to release version 1.2.0 soon? The release date defined in milestones was 31. January 2018. There are some minor fixes which would help. Thanks.

1.2.0 published !

authors: chymz kzimny


text:
Does this tool even work?

Behavior
I am trying to use this tool to prune old images from ghcr repository: I run this command:
$ /bin/drprune gh images -n $GH_ORG_NAME -t "$GH_TOKEN" -c $IMAGE_NAME

██╗███╗   ███╗ █████╗  ██████╗ ███████╗███████╗
██║████╗ ████║██╔══██╗██╔════╝ ██╔════╝██╔════╝
██║██╔████╔██║███████║██║  ███╗█████╗  ███████╗
██║██║╚██╔╝██║██╔══██║██║   ██║██╔══╝  ╚════██║
██║██║ ╚═╝ ██║██║  ██║╚██████╔╝███████╗███████║
╚═╝╚═╝     ╚═╝╚═╝  ╚═╝ ╚═════╝ ╚══════╝╚══════╝

INFO[2022-07-27_18:17:31] ✅ Package: xxxxxx/xxxxxxx
INFO[2022-07-27_18:17:31] ✅ We have 232 versions in this package

No matter what args I run this with, the number 232 does not change. So I'm wondering, does this tool even do what it's supposed to, or is it just a prototype?
Expected behavior
Old images should be deleted from GHCR
Actual behavior
Nothing is deleted
Environment
I am using the latest main version, built using make build on Ubuntu 20.04 (snap version of golang, v1.18.3)

Hi @ukch!
This CLI is under development, I haven't had the opportunity to gen a stable version yet :(.
If you want to help, feel free!
Thx! \o/

authors: ci-monk ukch

repo: ci-monk/drprune


text:
Moving cursor grabbing functionality into behavior along with a setting to enable it.

On some of the projects we've integrated this, we've added functionality over the top on attached, which listens to touch start and end, and applies the grabbing cursor. This should be moved into the behavior to reduce duplication, with a setting to enable it.

@WillDonohoe Could you submit a PR based on the code you previously wrote for this functionality?

Closing this as per PR #3

authors: WillDonohoe ciampo

repo: ciampo/swiper-behavior


text:
Bug: Can't publish apk

I just tried the release 1.3.0 and the sample project here...
 ./gradlew assembleDemoRelease
 ./gradlew publishHuaweiAppGalleryDemoRelease

Build file /Users/julio.buenocotta/git/huawei-publish-gradle-plugin/sample-groovy/build/outputs/apk/demo/release has wrong file extension that doesn't match with announced buildFormat(APK) plugin extension param.

@cosic are you there ?

@BugsBunnyBR Hi! Yes, I'm here. I'm going to  figure out it next weekend.

I found a couple issues which blocked me https://github.com/gradle/gradle/issues/16777 and https://github.com/gradle/gradle/issues/16775. I'm trying to figure it out.

For the first issue...
have you tried this workaround https://github.com/gradle/gradle/issues/16777#issuecomment-817303964 ?
It is odd, someone commented that is was fixed in gradle 7.1.1... and ktlint was able to workaround it as well... maybe looking at the code or their changelog would help ?

@BugsBunnyBR Sorry for long reaction. But seems I fixed this issue. Could you check 1.3.1-SNAPSHOT version or into snapshot-1.3.1 branch?

@BugsBunnyBR Hi! Could you check again 1.3.1-SNAPSHOT version please?

@BugsBunnyBR Is /builds/veepee/offerdiscovery/products/front-mobile/android/android-vente-privee/app/build/outputs/apk/veepeeAll/release correct path for build apk file folder? Does it contain any apk file?

It seems that compiling the app in a different command resulted in the upload task to work!
This works
./gradlew clean assembleRelease
./gradlew :app:publishHuaweiAppGalleryRelease

This doesn't work
./gradlew clean assembleRelease :app:publishHuaweiAppGalleryRelease

@BugsBunnyBR Yes i got the same feedback here (issue#26) but haven't tackled the issue yet. Do you have any idea why?

Fixed at v1.3.1

authors: BugsBunnyBR cosic

repo: cianru/huawei-publish-gradle-plugin


text:
Singularity branch

Implement a branch with nf-ncov-voc using singularity instead of conda

Removing high priority status because we got nf-ncov-voc working with docker, which provides a significant runtime improvement over conda

repo: cidgoh/VIRUS-MVP


text:
Apply the current code status of Off-policy to On-policy

Off-policy版で修正されていた以下の点についてOn-policy版に反映

Gumbel分布パラメータの外部設定化
initial_finished を変数化
finished 更新のタイミングを最後に

ありがとうございますー、了解しました :+1:

repo: ciela/reinforce-joey


text:
NightmareJS Support

This project uses semver, please check the scope of this pr:

[ ] #none# - documentation fixes and/or test additions
[ ] #patch# - backwards-compatible bug fix
[x] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

Add NightmareJS support

Visual Acceptance tests:
New
Electron: Simple

Electron: Error

Electron: svg

Electron: svg-experimental

Electron: capture right

Electron: fixed

Electron: fixed-more


text:
update API for better support of custom handling of submit and close

#minor#
CHANGELOG

Added support to use the built in modalClose function only if needed

As this stands this would be considered a breaking change (the default close behavior would change) - I'm going to add some comments for refinement to avoid this.

👍 Awesome, thanks!

repo: ciena-frost/ember-frost-modal


text:
refactor word module data

Move all module data into Data.h and Data.cpp next to .cpp, use std::array, std::string_view, std::unordered_map for data definitions, reference Airline module data.

I want to work on this one .I'm going to start today

ok, assigned

any progress on it?

I hope he won't mind but he hasn't done anything in a while, if possible I'd like to be assigned

Sure, go ahead

assigned

Oh someone else already did it while I was sleeping, bravo.

@eric-bodhi data still needs to be moved to one file - WordData as other modules, so you can work on that.

if not I can do it

authors: cieslarmichal eric-bodhi ruxixa

repo: cieslarmichal/faker-cxx


text:
add missing tests for StringHelper

https://app.codecov.io/github/cieslarmichal/faker-cxx/blob/main/src%2Fcommon%2FStringHelper.cpp

I can take this, if its not been worked.


text:
Connectivity test fails with Envoy as DaemonSet

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When deploying Cilium on an EKS cluster AND configuring it with Envoy as DaemonSet the cilium connectivity test command fails.
Here is an extract of the output (let me know if you need the whole output):
  ℹ  📜 Applying CiliumNetworkPolicy 'client-egress-to-fqdns-one.one.one.one' to namespace 'cilium-test'..
  [-] Scenario [to-fqdns/pod-to-world]
  [.] Action [to-fqdns/pod-to-world/http-to-one.one.one.one-0: cilium-test/client-6b4b857d98-b2mts (10.0.12.196) -> one.one.one.one-http (one.one.one.one:80)]
  ❌ command "curl -w %{local_ip}:%{local_port} -> %{remote_ip}:%{remote_port} = %{response_code} --silent --fail --show-error --output /dev/null --connect-timeout 2 --max-time 10 --retry 3 --retry-all-errors --retry-delay 3 http://one.one.one.one:80" failed: command terminated with exit code 22
  ℹ  curl output:

  📄 No flows recorded for peer cilium-test/client-6b4b857d98-b2mts during action http-to-one.one.one.one-0
  📄 No flows recorded for peer one.one.one.one-http during action http-to-one.one.one.one-0
  [.] Action [to-fqdns/pod-to-world/https-to-one.one.one.one-0: cilium-test/client-6b4b857d98-b2mts (10.0.12.196) -> one.one.one.one-https (one.one.one.one:443)]
  [.] Action [to-fqdns/pod-to-world/https-to-one.one.one.one-index-0: cilium-test/client-6b4b857d98-b2mts (10.0.12.196) -> one.one.one.one-https-index (one.one.one.one:443)]
  [.] Action [to-fqdns/pod-to-world/http-to-one.one.one.one-1: cilium-test/client2-646b88fb9b-xsb7z (10.0.12.156) -> one.one.one.one-http (one.one.one.one:80)]
  ❌ command "curl -w %{local_ip}:%{local_port} -> %{remote_ip}:%{remote_port} = %{response_code} --silent --fail --show-error --output /dev/null --connect-timeout 2 --max-time 10 --retry 3 --retry-all-errors --retry-delay 3 http://one.one.one.one:80" failed: command terminated with exit code 22
  ℹ  curl output:

  📄 No flows recorded for peer cilium-test/client2-646b88fb9b-xsb7z during action http-to-one.one.one.one-1
  📄 No flows recorded for peer one.one.one.one-http during action http-to-one.one.one.one-1
  [.] Action [to-fqdns/pod-to-world/https-to-one.one.one.one-1: cilium-test/client2-646b88fb9b-xsb7z (10.0.12.156) -> one.one.one.one-https (one.one.one.one:443)]
  [.] Action [to-fqdns/pod-to-world/https-to-one.one.one.one-index-1: cilium-test/client2-646b88fb9b-xsb7z (10.0.12.156) -> one.one.one.one-https-index (one.one.one.one:443)]
  [-] Scenario [to-fqdns/pod-to-world-2]
  [.] Action [to-fqdns/pod-to-world-2/https-cilium-io-0: cilium-test/client-6b4b857d98-b2mts (10.0.12.196) -> cilium-io-https (cilium.io:443)]
  [.] Action [to-fqdns/pod-to-world-2/https-cilium-io-1: cilium-test/client2-646b88fb9b-xsb7z (10.0.12.156) -> cilium-io-https (cilium.io:443)]
  ℹ  📜 Deleting CiliumNetworkPolicy 'client-egress-to-fqdns-one.one.one.one' from namespace 'cilium-test'..

📋 Test Report
❌ 4/42 tests failed (4/284 actions), 13 tests skipped, 0 scenarios skipped:
Test [client-ingress]:
Test [client-egress-l7]:
  ❌ client-egress-l7/pod-to-world/http-to-one.one.one.one-1: cilium-test/client2-646b88fb9b-xsb7z (10.0.12.156) -> one.one.one.one-http (one.one.one.one:80)
Test [client-egress-l7-named-port]:
  ❌ client-egress-l7-named-port/pod-to-world/http-to-one.one.one.one-1: cilium-test/client2-646b88fb9b-xsb7z (10.0.12.156) -> one.one.one.one-http (one.one.one.one:80)
Test [to-fqdns]:
  ❌ to-fqdns/pod-to-world/http-to-one.one.one.one-0: cilium-test/client-6b4b857d98-b2mts (10.0.12.196) -> one.one.one.one-http (one.one.one.one:80)
  ❌ to-fqdns/pod-to-world/http-to-one.one.one.one-1: cilium-test/client2-646b88fb9b-xsb7z (10.0.12.156) -> one.one.one.one-http (one.one.one.one:80)
connectivity test failed: 4 tests failed

Please note that the same test works fine when disabling the following values of the Helm chart:
envoy:
  enabled: true

The results when disabling Envoy as DaemonSet:
✅ All 42 tests (284 actions) successful, 13 tests skipped, 0 scenarios skipped.

Cilium Version
cilium version
cilium-cli: v0.15.7 compiled with go1.21.0 on linux/amd64
cilium image (default): v1.14.1
cilium image (stable): v1.14.1
cilium image (running): 1.14.1
Kernel Version
Latest bottlerocket AMI
Kubernetes Version
v1.27.4
Sysdump
cilium-sysdump-20230910-102356.zip
Relevant log output
No response
Anything else?
The whole code used is here
Code of Conduct

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

@Smana can you try the test again? Since the connectivity was to 1.1.1.1 it might have failed because it was making connections to the outside world.

@aanm Of course, do I have to change anything? upgrade the cilium CLI version?

I just run the command again and I still get the same errors. I don't understand how it would work without changing anything?
I'm gonna double check again without running envoy as daemonset

@SmaineTF1 Just because we have had CI flakes for connectivity tests done to the outside world.

authors: SmaineTF1 Smana aanm

repo: cilium/cilium


text:
Transparent encryption changes pod network behavior on k8s foreground delete

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
With enabled encryption, if a pod is deleted with foreground deletion (background deletion does not cause this), the terminating pod is immediately isolated from pods running on other worker nodes.
The expected standard behavior would be that the pod is still able to connect to other pods during the shutdown grace period. This is necessary in some cases to gracefully complete or offload work.
To replicate this, you can create some k8s cluster with and without encryption enabled and a pod setup, where one pod continuously tries to connect to another pod running on a different node. Then delete the pod which connects to the other pod with foreground deletion and observe that the terminating pod is not able to reach the other pod anymore.
This was my test setup:
---
apiVersion: apps/v1
kind: StatefulSet
metadata:
  name: curl-test-deployment
  labels:
    app: curl-test
    curl-test: curl-test
spec:
  replicas: 1
  selector:
    matchLabels:
      app: curl-test
  template:
    metadata:
      labels:
        app: curl-test
    spec:
      affinity:
          podAntiAffinity:
            requiredDuringSchedulingIgnoredDuringExecution:
            - labelSelector:
                matchExpressions:
                - key: curl-test
                  operator: In
                  values:
                  - curl-test
              topologyKey: "kubernetes.io/hostname"
      securityContext:
        runAsUser: 1000
        runAsGroup: 1000
        fsGroup: 1000
      terminationGracePeriodSeconds: 40
      containers:
        - name: ping-container
          ports:
          - containerPort: 8080
          image: quay.io/cilium/alpine-curl:v1.5.0@sha256:7b286939730d8af1149ef88dba15739d8330bb83d7d9853a23e5ab4043e2d33c
          env:
            - name: CURL_TARGET
              value: nginx-service.dev-kafka.svc:80
          command: ["/bin/sh"]
          args: ["-c", "while true; do curl -sS --fail --connect-timeout 5 $CURL_TARGET; sleep 1;  done;"]
          resources:
            requests:
              cpu: 500m
              memory: 500Mi
            limits:
              cpu: 500m
              memory: 500Mi
          securityContext:
            runAsNonRoot: true
            allowPrivilegeEscalation: false
            seccompProfile:
              type: RuntimeDefault
            capabilities:
              drop: ["ALL"]
---
apiVersion: v1
kind: Pod
metadata:
  name: nginx
  labels:
    app.kubernetes.io/name: proxy
    curl-test: curl-test
spec:
  affinity:
      podAntiAffinity:
        requiredDuringSchedulingIgnoredDuringExecution:
        - labelSelector:
            matchExpressions:
            - key: curl-test
              operator: In
              values:
              - curl-test
          topologyKey: "kubernetes.io/hostname"
  containers:
  - name: nginx
    image: nginx:stable
    ports:
      - containerPort: 80
        name: http-web-svc
    resources:
      requests:
        cpu: 500m
        memory: 500Mi
      limits:
        cpu: 500m
        memory: 500Mi
---
apiVersion: v1
kind: Service
metadata:
  name: nginx-service
spec:
  selector:
    app.kubernetes.io/name: proxy
  ports:
  - name: name-of-service-port
    protocol: TCP
    port: 80
    targetPort: http-web-svc

Cilium Version
1.14.6
Kernel Version
Linux 6.5.0-14-generic #14~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Mon Nov 20 18:15:30 UTC 2 x86_64 x86_64 x86_64 GNU/Linux
Kubernetes Version
v1.26.10 & v1.27.3
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Code of Conduct

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

@khartld can you share how you are installing Cilium?

In my local tests I created a kind cluster with config
kind: Cluster
apiVersion: kind.x-k8s.io/v1alpha4
#networking:
  disableDefaultCNI: true   # do not install kindnet
  kubeProxyMode: none       # do not run kube-proxy
nodes:
- role: control-plane
- role: worker
- role: worker
- role: worker

And installed Cilium via helm via helm with these values:
image:
 pullPolicy: IfNotPresent
ipam:
 mode: kubernetes

# required for runnig without kube-proxy
kubeProxyReplacement: strict
k8sServiceHost: kind-control-plane # use master node in kind network
k8sServicePort: 6443               # use api server port

l7Proxy: false # does not cause problem, but is required for wireguard
encryption:
 enabled: true
 type: wireguard

Not completely sure about our production setup, but there we also use the helm chart in combination with ArgoCD

Okay, this is actually expected behavior (I know, it seems weird, but bear with me).
Routing and policy enforcement in Cilium is actually done with CiliumEndpoints, which are separate objects that are created by Cilium to manage details about how to pass traffic to and from an associated Pod. Importantly, the CiliumEndpoint is associated with the Pod by an ownerReference on the CiliumEndpoint.
When you do a background deletion, the Pod is deleted first, then Kubernetes will garbage collect everything that has an ownerReference to that Pod afterwards.
Contrast that with foreground deletion, where the Pod is marked as deleting, then the dependent objects (including CiliumEnpodints in this case) are deleted, then the Pod is deleted.
(See https://kubernetes.io/docs/concepts/architecture/garbage-collection/#cascading-deletion for the details).
In this case, when you foreground delete a Pod, the underlying resources that allow Cilium to forward traffic are deleted while the pod is still running, so you see the traffic start dropping.
It sounds like using background deletion and/or using preStopHooks would be better choices for what you're trying to do.
Please feel free to discuss more if this is unclear.

Okay, that makes sense. I just want to mention (for other people maybe encountering this issue) that this behavior makes transparent encryption incompatible with operators that rely on foreground deletion like Strimzi

Does the ownerReference to the Pod set the blockOwnerDeletion flag?  It sounds like it should be set to false in this case, which would allow the Pod to be garbage collected before the CiliumEndpoint, and if necessary the CiliumEndpoint can use a finalizer to ensure that it is not deleted before the owning Pod is deleted.

That's good - so it's at least possible, if unlikely, that the Pod will be garbage collected before the CiliumEndpoint.  I think the CiliumEndpoint should have a finalizer on it so that the controller can make sure the Pod is deleted before the CiliumEndpoint.

It looks like Cilium deliberately avoided finalizers + blockOwnerDeletion in order to reduce the permissions available to specific nodes (with commits like https://github.com/cilium/cilium/commit/900f66879ad4c66e62eaa334fe7bd6ab2119e5b1). What exactly would the finalizer do?

A finalizer on the CiliumEndpoint will prevent kubernetes garbage collection from deleting the resource as long as the finalizer exists.  The deletion/cleanup code for the CiliumEndpoint can detect the deletionTimestamp and wait for the associated Pod to be deleted before it removes it's resources and the finalizer.  That would ensure that the CiliumEndpoint resource remains viable as long as the Pod exists, and once the Pod is removed the CiliumEndpoint can be safely deleted.
I think removing blockOwnerDeletion: true was probably correct, since the Pod owns the CiliumEndpoint it would require the CiliumEndpoint to be deleted before the Pod which appears to cause problems.  However the finalizer on the CiliumEndpoint is likely still needed as long as the Pod requires the CiliumEndpoint to exist during it's cleanup phase.

authors: aanm bobh66 joestringer khartld youngnick


text:
In the iPam mode of CRD, the pod may allocate an IP that has already been released

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When using IPAM in CRD mode, I encountered an issue where the IP address already assigned to the pod was marked as released. After analysis, it was found that there are bugs in the current version of Cilium.
The steps and related codes to reproduce the issue are as follows:

The operator triggered 'PrepareIPRelease()' and marked an IP in 'CiliumNode. status. release.ips' as 'marked-for-release'. node.go:L789
The agent received a new 'CiliumNode' update event and triggered the 'updateLocalNodeResource()` function. allocator.go#L196
Concurrent with 2., create a new pod and allocate a new IP.
Finally, the newly allocated IP for the third part will be marked as' ready-for-release 'in the 2. step, and the IP address release will be completed.  The IP address of the Pod that has already been running was released..

The detailed code related to the problem is as follows:
https://github.com/cilium/cilium/blob/73b3704bb96bbd90ba76992d9ad978d87e402716/pkg/ipam/crd.go#L428-L443
Cilium Version
1.14.6
Kernel Version
5.15.54-051554-generic
Kubernetes Version
1.24.4
Regression
No response
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Cilium Users Document

[ ] Are you a user of Cilium? Please add yourself to the Users doc

Code of Conduct

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

Thanks for the comment; this certainly looks like an issue.

reopen

I reopen it since @weiwei02 has been working on a fix https://github.com/cilium/cilium/pull/34489.

authors: doniacld squeed weiwei02


text:
question: Assign a specific IP address to cilium-ingress service

Is there an existing issue for this?

[x] I have searched the existing issues

Version
equal or higher than v1.16.4 and lower than v1.17.0
What happened?
I'm trying to assign a specific IP address to a cilium-ingress type service. I initially thought the IP address assigned to a LoadBalancer service type will be automatically assigned to cilium-ingress service, but that is not the case. I end-up with two CiliumLoadBalancerIPPool addresses being used. The goal is to assign the specific 192.168.4.30 IP address to cilium-ingress service and keep the origin service type as ClusterIP.
How can we reproduce the issue?
Service vmalertmanager-vmks-victoria-metrics-k8s-stack is using 192.168.4.30 IP address, which I specifically assign:
apiVersion: v1
kind: Service
metadata:
  name: vmalertmanager-vmks-victoria-metrics-k8s-stack
  namespace: kube-system
  uid: 3f7b3ac2-0abe-4938-b9dd-fd0773b0fe2f
  resourceVersion: '15971'
  creationTimestamp: '2024-11-30T01:50:19Z'
  labels:
    app.kubernetes.io/component: monitoring
    app.kubernetes.io/instance: vmks-victoria-metrics-k8s-stack
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/name: vmalertmanager
    app.kubernetes.io/version: v1.106.1
    helm.sh/chart: victoria-metrics-k8s-stack-0.28.4
    managed-by: vm-operator
  annotations:
    io.cilium/lb-ipam-ips: 192.168.4.30
    meta.helm.sh/release-name: vmks
    meta.helm.sh/release-namespace: kube-system
  ownerReferences:
    - apiVersion: operator.victoriametrics.com/v1beta1
      kind: VMAlertmanager
      name: vmks-victoria-metrics-k8s-stack
      uid: 432f7aa5-45fb-425c-958b-5a67be8bfc15
      controller: true
      blockOwnerDeletion: true
  finalizers:
    - apps.victoriametrics.com/finalizer
  selfLink: >-
    /api/v1/namespaces/kube-system/services/vmalertmanager-vmks-victoria-metrics-k8s-stack
status:
  loadBalancer:
    ingress:
      - ip: 192.168.4.30
        ipMode: VIP
  conditions:
    - type: cilium.io/IPAMRequestSatisfied
      status: 'True'
      lastTransitionTime: '2024-11-30T01:50:19Z'
      reason: satisfied
      message: ''
spec:
  ports:
    - name: http
      protocol: TCP
      port: 9093
      targetPort: web
      nodePort: 31516
    - name: tcp-mesh
      protocol: TCP
      port: 9094
      targetPort: 9094
      nodePort: 31949
    - name: udp-mesh
      protocol: UDP
      port: 9094
      targetPort: 9094
      nodePort: 31949
  selector:
    app.kubernetes.io/component: monitoring
    app.kubernetes.io/instance: vmks-victoria-metrics-k8s-stack
    app.kubernetes.io/name: vmalertmanager
    managed-by: vm-operator
  clusterIP: 10.43.30.115
  clusterIPs:
    - 10.43.30.115
  type: LoadBalancer
  sessionAffinity: None
  externalTrafficPolicy: Cluster
  ipFamilies:
    - IPv4
  ipFamilyPolicy: SingleStack
  allocateLoadBalancerNodePorts: true
  internalTrafficPolicy: Cluster

While cilium-ingress-vmalertmanager-vmks-victoria-metrics-k8s-stack is using 192.168.4.21 IP address:
apiVersion: v1
kind: Service
metadata:
  name: cilium-ingress-vmalertmanager-vmks-victoria-metrics-k8s-stack
  namespace: kube-system
  uid: 86140058-d4d9-41e2-8d8c-6f712e80adab
  resourceVersion: '24925'
  creationTimestamp: '2024-11-30T01:49:47Z'
  labels:
    cilium.io/ingress: 'true'
  ownerReferences:
    - apiVersion: networking.k8s.io/v1
      kind: Ingress
      name: vmalertmanager-vmks-victoria-metrics-k8s-stack
      uid: 0b8c615e-634a-4438-8b5c-9e2da1f7981d
      controller: true
  selfLink: >-
    /api/v1/namespaces/kube-system/services/cilium-ingress-vmalertmanager-vmks-victoria-metrics-k8s-stack
status:
  loadBalancer:
    ingress:
      - ip: 192.168.4.21
        ipMode: VIP
  conditions:
    - type: cilium.io/IPAMRequestSatisfied
      status: 'True'
      lastTransitionTime: '2024-11-30T01:49:48Z'
      reason: satisfied
      message: ''
spec:
  ports:
    - name: http
      protocol: TCP
      port: 80
      targetPort: 80
      nodePort: 30184
    - name: https
      protocol: TCP
      port: 443
      targetPort: 443
      nodePort: 31545
  clusterIP: 10.43.16.144
  clusterIPs:
    - 10.43.16.144
  type: LoadBalancer
  sessionAffinity: None
  externalTrafficPolicy: Cluster
  ipFamilies:
    - IPv4
  ipFamilyPolicy: SingleStack
  allocateLoadBalancerNodePorts: true
  internalTrafficPolicy: Cluster

Cilium custom chart settings:
$ helm get values cilium -n kube-system
USER-SUPPLIED VALUES:
autoDirectNodeRoutes: true
bandwidthManager:
  enabled: true
bpf:
  datapathMode: netkit
  lbExternalClusterIP: false
  masquerade: true
  tproxy: true
  vlanBypass:
  - 1
  - 4
bpfClockProbe: true
cgroup:
  resources:
    limits:
      cpu: null
      memory: 128Mi
    requests:
      cpu: 10m
      memory: 128Mi
ciliumEndpointSlice:
  enabled: true
cni:
  resources:
    limits:
      cpu: null
      memory: 128Mi
    requests:
      cpu: 10m
      memory: 128Mi
dashboards:
  enabled: true
directRoutingSkipUnreachable: true
envoy:
  enabled: true
  prometheus:
    serviceMonitor:
      enabled: true
      namespace: kube-system
  resources:
    limits:
      cpu: null
      memory: 128Mi
    requests:
      cpu: 10m
      memory: 128Mi
  rollOutPods: true
envoyConfig:
  enabled: true
externalIPs:
  enabled: true
gatewayAPI:
  enableAlpn: true
  enabled: true
hubble:
  metrics:
    dashboards:
      enabled: true
    enabled:
    - dns:query;ignoreAAAA
    - drop
    - flow
    - httpV2
    - icmp
    - port-distribution
    - tcp
    serviceMonitor:
      enabled: true
      namespace: kube-system
  peerService:
    clusterDomain: cluster.local
  relay:
    enabled: true
    podDisruptionBudget:
      enabled: true
    prometheus:
      enabled: true
      serviceMonitor:
        enabled: true
        namespace: kube-system
    replicas: 2
    resources:
      limits:
        cpu: null
        memory: 128Mi
      requests:
        cpu: 10m
        memory: 128Mi
    retryTimeout: 10s
    rollOutPods: true
  tls:
    auto:
      certManagerIssuerRef:
        group: cert-manager.io
        kind: ClusterIssuer
        name: hubble-cluster-issuer
      method: certmanager
  ui:
    backend:
      resources:
        limits:
          cpu: null
          memory: 128Mi
        requests:
          cpu: 10m
          memory: 128Mi
    enabled: true
    frontend:
      resources:
        limits:
          cpu: null
          memory: 128Mi
        requests:
          cpu: 10m
          memory: 128Mi
    podDisruptionBudget:
      enabled: true
    replicas: 1
    rollOutPods: true
ingressController:
  enabled: true
ipam:
  operator:
    clusterPoolIPv4PodCIDRList: 10.42.0.0/16
ipv4NativeRoutingCIDR: 10.42.0.0/16
k8sClientRateLimit:
  burst: 100
  qps: 50
k8sServiceHost: 127.0.0.1
k8sServicePort: 6444
kubeProxyReplacement: true
l2announcements:
  enabled: true
  leaseDuration: 15s
  leaseRenewDeadline: 5s
  leaseRetryPeriod: 2s
l2podAnnouncements:
  enabled: true
  interface: eth0
loadBalancer:
  algorithm: maglev
  l7:
    backend: envoy
logOptions:
  level: warn
maglev:
  hashSeed: BOr1gpQOdI3oUFuN
  tableSize: 16381
operator:
  prometheus:
    enabled: true
    serviceMonitor:
      enabled: true
      namespace: kube-system
  resources:
    limits:
      cpu: null
      memory: 128Mi
    requests:
      cpu: 10m
      memory: 128Mi
  rollOutPods: true
prometheus:
  enabled: true
  serviceMonitor:
    enabled: true
    namespace: kube-system
resources:
  limits:
    cpu: null
    memory: 256Mi
  requests:
    cpu: 10m
    memory: 256Mi
rollOutCiliumPods: true
routingMode: native

Cilium Version
cilium-cli: v0.16.20 compiled with go1.23.2 on linux/arm64
cilium image (default): v1.16.3
cilium image (stable): v1.16.4
cilium image (running): 1.16.4

Kernel Version
Linux apollo 6.8.0-1015-raspi #17-Ubuntu SMP PREEMPT_DYNAMIC Mon Nov 11 14:12:16 UTC 2024 aarch64 aarch64 aarch64 GNU/Linux

Kubernetes Version
Client Version: v1.30.6+k3s1
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.30.6+k3s1

Regression
No response
Sysdump
No response
Relevant log output

Anything else?
No response
Cilium Users Document

[ ] Are you a user of Cilium? Please add yourself to the Users doc

Code of Conduct

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

add annotation "io.cilium/lb-ipam-ips" to your service maybe helpful

Thanks but that is not the question scope, also the annotation is there:
  annotations:
    io.cilium/lb-ipam-ips: 192.168.4.30
    meta.helm.sh/release-name: vmks
    meta.helm.sh/release-namespace: kube-system

Removing the ingressClassName fixes the issue:
server:
  ingress:
    ingressClassName: cilium

authors: FouoF fmunteanu


text:
tests: update complexity check script to include new calls

Fixes #10100

This change is

Coverage increased (+0.02%) to 44.635% when pulling f4af0a9425937a7946ef92cc5567b5c21ffa336d on fristonio/fix-10100 into 629db726d96d8dddc5206b021170510d8ab0fc6d on master.

test-me-please

test-me-please

test-me-please

authors: coveralls fristonio joestringer


text:
Use RLock while reading data in DNS proxy

This change changes locking behaviour in GetRules and CheckAllowed
DNSProxy methods to read lock, which should allow concurrent DNS traffic
handling goroutines to get the data they need from the proxy faster.

/test

Since all failures are flakes that don's seem to be connected to the change, seems safe to mark as ready-to-merge

Added a release note. User has confirmed that this resolves their memory "leak" issue. Nice work!

authors: christarazi nebril


text:
cilium-health: fix probing for IPv6-only clusters

Due some refactoring done in ed934cb958c5, a bug was introduced in the
code that overwrote the IP address used to perform health checks.
In IPv6-only clusters, this IP address was overwritten by an empty IPv4
address which would then be used to perform health checks. Obviously
failing to perform such health checks since the address was "",
Cilium would report that cilium-health-ep controllers were failing.
Fixes: ed934cb958c5 ("health: Move endpoint IP to node package")
Signed-off-by: André Martins andre@cilium.io
Fixes https://github.com/cilium/cilium/issues/20796
Fix regression with cilium-health-probe controller in IPv6-only clusters

/test


text:
hive: Add Validate to CellConfig

To support validation and modification of configurations, this commit
extends CellConfig interface with Validate() and adapts the cell
constructors to allow implementing Validate() as a pointer receiver.
CellFlags is renamed to just Flags.
I tried bunch of approaches to getting Validate as pointer receiver
working. Using a type parameter works, but requires rather ugly
constraints: [T CellConfig, P interface { *T; CellConfigValidator }],
so I ended up with just passing the value in and reflecting on it to
figure out whether it's a struct or pointer to a struct.
This way we can implement a mutating Validate that populates private
fields by implementing it as pointer receiver and passing a pointer to the config:
type MyConfig struct {
  Foo int
  twoFoo int
}
func (*MyConfig) Flags(flags *pflag.FlagSet) { ... }

func (m *MyConfig) Validate() error {
  m.Foo += m.Foo
  m.twoFoo = m.Foo * 2
  return nil
}

func (m *MyConfig) TwoFoo() int {
  return m.twoFoo
}

var myConfigCell = hive.NewConfigCell(&MyConfig{})

This is useful when pflags does not provide the parsing primitives out of the box
for a specific option.

Closing to rethink whether Validate should be allowed to mutate config...


text:
Fix RKE doc link

Fixed link to broken anchor in RKE doc
Fixed link to broken anchor in RKE doc

Signed-off-by: Raphaël Pinson raphael@isovalent.com

Note: there's still a weird redirection on the RKE docs side even with the right URL. I've reported it to SuSE.


text:
bpf: improve trace events in SNAT path

Enhance trace events from the outbound SNAT path, to report the pre-SNAT IP address and the interface index of the egress interface.

/test

@julianwiedmann should we add a note in tail_handle_snat_fwd_ipv6 to also populate saddr and use send_trace_notify6 when egress gateway IPv6 support is implemented? If yes, maybe here?

@julianwiedmann should we add a note in tail_handle_snat_fwd_ipv6 to also populate saddr and use send_trace_notify6 when egress gateway IPv6 support is implemented? If yes, maybe here?

I'll need to refresh my memory, but pretty sure that we can wire up the IPv6 path even today :).
We'll populate the orig-saddr for all kinds of SNAT operations, not just EgressGW. So for instance plain IPv6 Masquerading in BPF, where we would report the pod's IP in the trace event.

/test

@kaworu I rebased and added the IPv6 handling. Also making sure that the orig_addr is only set when we actually SNAT the packet.
Would you have a moment to double-check this still works as expected?

/test

:wave:
Seems like we might need some extra commits from https://github.com/cilium/cilium/pull/26674 for 1.14 backport.

authors: julianwiedmann kaworu sayboras


text:
policy: Fix MapState.Equals()

Compare the entries of 'msA' and 'msB' rather than 'msB' against itself.
Simplify the body of the comparison function for readability.
Fixes: #28352

/test


text:
srv6: Move SRv6 encap after internal traffic handlings

Currently, SRv6 encap takes precedense over local delivery, so when configured egress rules overlapping with the local PodCIDR, it takes over the local Pod-to-Pod traffic. Prevent this issue by moving SRv6 encap logic after the local Pod-to-Pod delivery traffic.
srv6: Move SRv6 encap after internal traffic handlings

/test

I'd think that when a VRF is configured and a VPN network overlaps with a local network, you prefer the VPN delivery (for security reasons).

I concluded that at least for now, we should prioritize local delivery because users cannot always control the route imported from upstream, so if the imported route unexpectedly overlaps with Cilium's internal route, then we call it route hijacking. So, for me, always prioritizing the internal route is a safer option.
Also, it's inconsistent with the regular native routing mode (without SRv6). For example, users can install Bird on the node and import the route from an external router. The route will be installed into the kernel's routing table, so it will be evaluated later than local delivery.

Okay I understand the reasoning.

Conformance Ginkgo: https://github.com/cilium/cilium/issues/31336
Conformance Ingress: https://github.com/cilium/cilium/issues/30281

Sorry, but I noticed this doesn't work under endpointRoutes.enabled=true. Let me close this and think about another way around.

authors: YutaroHayakawa ldelossa


text:
Tests: Update Nightly to kubernetes 1.9

Updating Nightly test to Kubernetes 1.9 instead of 1.8
Signed-off-by: Eloy Coto eloy.coto@gmail.com

test-nightly

test-nightly

test-me-please


text:
pkg/labels: use LabelHealth constant across the code

Signed-off-by: André Martins andre@cilium.io

This change is

test-me-please


text:
daemon: add parameter indicating why TriggerPolicyUpdates is called

This function triggers regeneration for all endpoints, which can potentially be
very costly. To get more visibility into why it is called, add a string
parameter to it for callers to indicate the reason for calling it.
Signed-off by: Ian Vernon ian@cilium.io

This change is

test-me-please

Sigh, forgot to fix the unit test Daemon mocks. Will fix and reupload...

test-me-please