SSL encrypted connection configuration does not work

I tried some combinations based on the configuration options available in this client to use SSL encrypted connection. Unfortunately, nothing worked. Here are results of investigation:
Here is what they say for configuration of SSL encrypted connection in official documentation:

For untrusted networks you can open an SSL encrypted connection to api.logentries.com, port number 20000.

However, default secure port is 443, what makes it impossible just to use secure option without specifying port. Same for host, where the value here is data.logentries.com, but has to be api.logentries.com.
So I tried some options.
Does not send anything to LE:
secure: true,
host: 'api.logentries.com',
port: '20000'

Fails and crashes application:
host: 'api.logentries.com',
port: '20000'

Also, observed some issues with configuration of ports in general. Ignores the port:
host: 'data.logentries.com:12345'

Fails to start:
host: 'data.logentries.com',
port: '12345'

I won’t be able to investigate this until the weekend, but I can say that the use of 443 with data.logentries.com is intentional. This change came from @StephenHynes7  and I believe it’s what we’re moving towards, though I don’t know the details of why.
But it clearly should also work with the api / 20000 endpoint, and the host option is supposed to allow shorthand for setting port, too, so regardless of the above, both of those look like bugs.

Hi, any news on this?

Yup -- well, as of just now, since you reminded me, thanks :)
It looks like both problems are the consequence of really simple mistakes:
opts.port || opts.secure ? defaults.portSecure : defaults.port;

should be:
opts.port || (opts.secure ? defaults.portSecure : defaults.port);

And:
this[$host] = url.hostName || defaults.host;

should be:
this[$host] = url.hostname || defaults.host;

I’m not 100% sure that explains what you encountered, but I'm going to push these fixes up now since they’re at least clearly part of the story.

1.0.10 is live now with those fixes

I tried with a new version, SSL connection to api.logentries.com:20000 still does not work.

@voitau is there any particular reason why you want to send to api.logentries.com instead of data? data.logentries.com would be the perfered endpoint for Token TCP connections.

@voitau -- specifically what happens? Does it still throw? Does the host/port not get set correctly? Does it appear to work in node, but nothing shows up in the logentries app?

@StephenHynes7, api.* is mentioned in the official documentation as the proper endpoint for SSL (https://logentries.com/doc/input-token/):

For untrusted networks you can open an SSL encrypted connection to api.logentries.com, port number 20000.

@bathos, it does not show up in logentries app.
Configuration is:
winston.add(winston.transports.Logentries, {
  token: 'SOME_TOKEN,
  secure: true,
  host: 'api.logentries.com',
  port: '20000'
});

I tried with and without secure, api.* and data.*. Does not work unless everything removed besides token, which will make it work without SSL.

Hello everyone, any news here? Thanks!

I'll give this another go, voitau.  The old endpoint is still active and accepts the same format as the new one, so it must be an le_node-specific issue. Maybe I'm overriding the custom port when secure == true, will check it out now.

Not sure why this was so mysterious. Port was only accepting numbers, not strings. I'll have it parseInt first.

Fixed in 1.0.14

Maybe I'm doing something wrong, but I'm trying to test this and the message is not displayed in logentries admin UI. What is the configuration you are using to test this? What is the secure flag value, how do you define host and port?

I used the configuration above and everything was being set correctly (it was dropping port, if it was a string, previously). However I didn’t get to test it against a real account, so it remains possible that there’s another issue, though it seems pretty unlikely since it was sending to the right port & host in the mock. You definitely got 1.0.14?

Yes, it is 1.0.14 and this is my test code:
var Logger = require('le_node');
var winston = require('winston');

winston.add(winston.transports.Logentries, {
  token: 'SOME_TOKEN',
  secure: true,
  host: 'api.logentries.com',
  port: '20000'
});

winston.log('info', { key: 'value' });

Should this one be reopened? Thanks.

Yeah, I'll investigate again, though at this point I'm pretty confused since in tests the host & port were correct for me. I'll try it against a real account.

Hello, any updates here? Thanks.

Does using data.lognetries.com and port 443 work at all? I wouldn't advise using api.logentries.com as an endpoint versus data.logentries.com since it is not optimised for Token based forwarding.

@StephenHynes7 I tried all possible combinations I mentioned above and none of them worked for me. However, the official documentation says https://logentries.com/doc/input-token/:
"For untrusted networks you can open an SSL encrypted connection to api.logentries.com, port number 20000."

@voitau that part is likely out of date, i will look to change it. Really it should be data.logentries.com and not api.

Confirming that it works with secure: true and no other additional settings. Ticket can be closed, thanks!

Okay, found the root cause. It did not work with node v0.10, but works with v0.12.


[BUG]

Description
I don't know that if it's happening in my device or not.
But when I make it in responsive mode in the small screen then the dark mode button is not working 😄
Maybe it's a glitch from my side or maybe it's a bug
Please check this in your browser too @rupali-codes @CBID2 @ujjawaltyagii
I'm giving ss here
Screenshots

Though it is impossible to understand from the ss
Any additional information?
If it is a bug then I want to try to fix it
What browser are you seeing the problem on?
Chrome
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

It's working fine for almost every screen size 👀 and the only screen size where it doesn't work is screen size less than 310 and phone for that screen doesn't exist Ig XD

@Soumyajit2825 is dark mode not working? could you please elaborate more on the issue?

@Soumyajit2825 is dark mode not working? could you please elaborate more on the issue?

Actually it was not working in the small screen mobiles

@Soumyajit2825 is dark mode not working? could you please elaborate more on the issue?

Actually it was not working in the small screen mobiles

Which resolution? bcz its working fine..i have tested it :)

5 days ago when I had opened the issue then it was not working
Maybe someone has fixed it

5 days ago when I had opened the issue then it was not working Maybe someone has fixed it

Could you please check now :D

closing this issue because of inactivity


[Bug]: LogMessageWaitStrategy erroneously assumes each log frame contains a complete log line

Module
Core
Testcontainers version
1.17.3
Using the latest Testcontainers version?
Yes
Host OS
Linux CentOS
Host Arch
x86_64
Docker version
Docker version 20.10.18, build b40c2f6

What happened?
LogMessageWaitStrategy uses a LogContainerCmd to follow logs.
The regex is applied to every OutputFrame that is processed by FrameConsumerResultCallback.
FrameConsumerResultCallback consumes STDOUT or STDERR stream types via processOtherFrame.
Problem:
processOtherFrame passes the content of the com.github.dockerjava.api.model.Frame 1-to-1 to the consumer without doing any processing of log lines (like processRawFrame > normalizeLogLines).
This would only work if the Frame contains a complete log message and not a partial log message.
We reproduced this multiple times. See the log output for one example.
When using a WaitAllStrategy with more LogMessageWaitStrategy, this is more likely to happen because the second log wait strategy only attaches to the log after the first finished, and then of course gets a lot more data at once.
Relevant log output
2022-09-13 00:34:58,297 DEBUG - STDOUT: 2022-09-12 22:34:56,408 INFO  - Extension "HiveMQ Enterprise Extension for Kafka" version 4.9.0-SNAPSHOT started succes
2022-09-13 00:34:58,313 DEBUG - STDOUT: sfully.

from this log statement: https://github.com/testcontainers/testcontainers-java/blob/de1324ed2800eff4da326d0c23d281399d006bc0/core/src/main/java/org/testcontainers/containers/output/WaitingConsumer.java#L89
Additional Information
No response

Related: #643


How to set the password？

When I log in the web page by IP:4040，How to set the password？

The standalone version of Scope doesn't supply any authentication mechanism (you will have to supply it yourself).
To resolve this we suggest you to use Weave Cloud, which has a lot more functionalities than Scope standalone, including authentication. See https://cloud.weave.works


[FEATURE REQUEST] Upgrade K8s CNI plugins to the latest available versions

Is your feature request related to a problem? Please describe.
New versions are released, so need to upgrade Flannel, Calico and Canal
Describe the solution you'd like
Upgrade CNI plugins
Describe alternatives you've considered
No
Additional context

https://github.com/flannel-io/flannel
https://github.com/projectcalico/calico

DoD checklist

[ ] Changelog updated
[ ] COMPONENTS.md updated / doesn't need to be updated
[ ] Schema updated / doesn't need to be updated
[ ] Feature has automated tests
[ ] Automated tests passed (QA pipelines)

[ ] apply
[ ] upgrade

[ ] Idempotency tested
[ ] Documentation added / updated / doesn't need to be updated
[ ] All conversations in PR resolved
[ ] Solution meets requirements and is done according to design doc
[ ] Usage compliant with license
[ ] Backport tasks created / doesn't need to be backported

@atsikham Few questions:
1: This is not something needed for the K8s 1.22.4 upgrade right?
2: Would it make sense to split this in 3 separate tasks? On for each plugin.
3: Maybe this is redundant as we might want to disconinue support for 3 plugins? @erzetpe Maybe something consider for this deprecation plan you are working on?

Right
Might be 2 - flannel and calico/canal
May be

@atsikham and @erzetpe, as there is plan to create a spike about Calico deprecation do we still need to include Calico plugin in this upgrade ticket?

@cicharka Yes, as this is not planned for 1.3 or 2.0 release.


Error '_get_unpatched' from 'setuptools.dist'

Hi all,
I'm a data manager. I'm trying to install yxf on Windows. I'm getting this error:
C:\Windows\System32>python -m pip install yxf
Collecting yxf
Using cached yxf-0.3.0-py3-none-any.whl (9.1 kB)
Collecting grapefruit (from yxf)
Using cached grapefruit-0.1a4.tar.gz (13 kB)
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [62 lines of output]
C:\Users\User\AppData\Local\Programs\Python\Python312\Lib\site-packages\setuptools\dist.py:498: SetuptoolsDeprecationWarning: Invalid dash-separated options
!!
          ********************************************************************************
          Usage of dash-separated 'description-file' will not be supported in future
          versions. Please use the underscore name 'description_file' instead.

          This deprecation is overdue, please update your project and remove deprecated
          calls to avoid build errors in the future.

          See https://setuptools.pypa.io/en/latest/userguide/declarative_config.html for details.
          ********************************************************************************

  !!
    opt = self.warn_dash_deprecation(opt, section)
  C:\Users\User\AppData\Local\Programs\Python\Python312\Lib\site-packages\setuptools\dist.py:498: SetuptoolsDeprecationWarning: Invalid dash-separated options
  !!

          ********************************************************************************
          Usage of dash-separated 'requires-dist' will not be supported in future
          versions. Please use the underscore name 'requires_dist' instead.

          This deprecation is overdue, please update your project and remove deprecated
          calls to avoid build errors in the future.

          See https://setuptools.pypa.io/en/latest/userguide/declarative_config.html for details.
          ********************************************************************************

  !!
    opt = self.warn_dash_deprecation(opt, section)
  C:\Users\User\AppData\Local\Programs\Python\Python312\Lib\site-packages\setuptools\__init__.py:80: _DeprecatedInstaller: setuptools.installer and fetch_build_eggs are deprecated.
  !!

          ********************************************************************************
          Requirements should be satisfied by a PEP 517 installer.
          If you are using pip, you can try `pip install --use-pep517`.
          ********************************************************************************

  !!
    dist.fetch_build_eggs(dist.setup_requires)
  Traceback (most recent call last):
    File "<string>", line 2, in <module>
    File "<pip-setuptools-caller>", line 34, in <module>
    File "C:\Users\User\AppData\Local\Temp\pip-install-gndvqsgk\grapefruit_b46a4ba7f1bc48c9a649100bba33b205\setup.py", line 24, in <module>
      setup(
    File "C:\Users\User\AppData\Local\Programs\Python\Python312\Lib\site-packages\setuptools\__init__.py", line 103, in setup
      return distutils.core.setup(**attrs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    File "C:\Users\User\AppData\Local\Programs\Python\Python312\Lib\site-packages\setuptools\_distutils\core.py", line 147, in setup
      _setup_distribution = dist = klass(attrs)
                                   ^^^^^^^^^^^^
    File "C:\Users\User\AppData\Local\Programs\Python\Python312\Lib\site-packages\setuptools\dist.py", line 303, in __init__
      _Distribution.__init__(self, dist_attrs)
    File "C:\Users\User\AppData\Local\Programs\Python\Python312\Lib\site-packages\setuptools\_distutils\dist.py", line 283, in __init__
      self.finalize_options()
    File "C:\Users\User\AppData\Local\Programs\Python\Python312\Lib\site-packages\setuptools\dist.py", line 680, in finalize_options
      ep(self)
    File "C:\Users\User\AppData\Local\Programs\Python\Python312\Lib\site-packages\setuptools\dist.py", line 700, in _finalize_setup_keywords
      ep.load()(self, ep.name, value)
    File "c:\users\user\appdata\local\temp\pip-install-gndvqsgk\grapefruit_b46a4ba7f1bc48c9a649100bba33b205\.eggs\d2to1-0.2.12.post1-py3.12.egg\d2to1\core.py", line 30, in d2to1
      from setuptools.dist import _get_unpatched
  ImportError: cannot import name '_get_unpatched' from 'setuptools.dist' (C:\Users\User\AppData\Local\Programs\Python\Python312\Lib\site-packages\setuptools\dist.py). Did you mean: 'get_unpatched'?
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
I don't know what to do.
Thanks for the help.

I'm using
C:\Windows\System32>python --version
Python 3.12.0
C:\Windows\System32>pip --version
pip 23.2.1 from C:\Users\User\AppData\Local\Programs\Python\Python312\Lib\site-packages\pip (python 3.12)

Hi, thanks for reporting this! It looks like there's a problem while installing grapefruit, one of the libraries that yxf depends on. Sorry about that... multiple people had problems with it; I'll try to find an alternative.
In the meantime: Some people were able to fix installation errors by running pip install wheel before installing yxf. Could you try that and let me know if it works?

Hi again,
I've just published version 0.4.0, which no longer depends on grapefruit. This one should install without problems, just using pip install yxf.
Please let me know if this works. If you have any other feedback about yxf, don't hesitate to reach out.

It works perfectly.
Thank you.


Why make all the signature validation logic internal?

I wanted to use this library to validate ES256 signatures, since all the cryptographic code is there. And there isn't many solutions. Sadly, I'll have to find another library because of the extensive use of internal in this one, for pretty much all ES256 code. The only exposed method requires my data to be in the JWSHeader and Payload format, which it isn't.

Hi @yspreen, this library is intended for use with JOSE data types only. If your data is not in this format — sadly this isn't the right library for you.


error_version 1.5.... of Java not installed on Mac OSX

I installed the newest version of java and I restarted multiple times... still doesn't work.. any ideas? :)

If you open a terminal window and run java -version what does the output say?

java -version
No Java runtime present, requesting install.

Try installing the recent JDK for Mac from here:

I have the same problem. Actually, I have upgraded java to 1.8.0_161, and set .bash_profile , and now java -version shows the right version 1.8.0_161, but bolero reported that java version should be greater than 1.8.0_151.

I had the same issue.
Having the java runtime installed didn't seem to work for me.
You actually have to have the JDK installed, and then this seems to work.
JDK takes more diskspace though, so it would be better if only runtime would be enough.


Don't bother returning more than 1 hit

BlockchainIndex returns 10 hits, but only the first one is observed and returned. Simplify the code, return first hit only, and remove the weird for loops that returns on the first element.

Hey can I work on this?

Sure thing. Send a pull request when you feel good about it.

Thank you so much.  I will start right away, but I apologize ahead of time if I need to reach out with questions.  I am new to open source contribution. Thanks again.

Any help is welcome. For this one, start here:
https://github.com/apache/incubator-tuweni/blob/257e37a1a8439197412fbc5c8e74d0fa5dcb420a/eth-repository/src/main/kotlin/org/apache/tuweni/eth/repository/BlockchainIndex.kt#L763


Suggestion: List user's own pastes with links

The user should be able to list their own pastes with more info than the regular list, such as filenames and links to the pastes.

Apparently this was actually an issue with #29


Webdocs experiment

Please view this solely as a conversation-starter.

Please first read the "Rationale" section: https://github.com/single-cell-data/TileDB-SingleCell/blob/kerl/web-docs-experiment/README-docs.md#rationale
Link to rendered content: https://tiledb-singlecell-docs.s3.amazonaws.com/docs/overview.html
Setup notes: https://github.com/single-cell-data/TileDB-SingleCell/blob/kerl/web-docs-experiment/README-docs.md

Intentionally, no CI is set up yet for this experiment

This PR is biggish as it introduces a framework for API+examples, while also introducing example/vignette content

The examples are overlong and could benefit from getting some bio-user eyes on them

Please note all the Python API and examples content is provisional -- we can make this what we want it to be
Note R API and examples are still hosted at https://github.com/TileDB-Inc/tiledbsc

@aaronwolen thanks! Here's the deal about R docs:

For now of course they're in the separate repo https://github.com/TileDB-Inc/tiledbsc and well-hosted at https://tiledb-inc.github.io/tiledbsc/
R docs already do API+vignette marvelously
GitHub Pages is fine with single-language stuff (R or Python) but I haven't yet found a way to get GitHub Pages to handle multi-language content within a single repo -- which is why I went around that for this PR and chose S3 static hosting for the moment
We of course (independent of R/Python) need to set up CI, and also decide if we want S3 static hosting or something else
But to your question about R docs: I think we'll be best-served if we have CI render the R docs as now, and have this doctree point to that (and vice versa)

Basically letting the R-document framework do what it already does well right out of the box, and trying to get an almost-as-good setup for Python as well here, & having them each point to the other.


FixedPointMath 100% coverage

Description
Removed infeasible branch condition in calculateWeightedAverage() per Certora report. This should get fixed point math to 100% coverage.

[ ] Testing

[ ] Are there new or updated unit or integration tests?
[ ] Do the tests cover the happy paths?
[ ] Do the tests cover the unhappy paths?
[ ] Are there an adequate number of fuzz tests to ensure that we are
covering the full input space?

coverage: 93.166% (+0.04%) from 93.125%
when pulling 47ad8240542304570ff867db87503325c05b4c37 on fp-coverage
into<PHONE_NUMBER>c433c725bd06b2daa491af65b2dcc2 on main.


ENH: Rewrite JoblibParallelization

Based on the discussion here
this PR rewrites JoblibParallelization to improve reliability
and consistency of using joblib.

@blankjul Does this PR require any modifications?


Duplicate filter can offer unpredictable results in folders with multiples copies of a single file

I have used organize to sort and rename files. After several runs, I created a few duplicates. As I tried to remove the dupes with the built-in filter, the simulations worried me a bit. I created a simplified example to demonstrate this behavior.
Initial configuration
You have a folder with several copies of the same file. There is an original file and several copies of the same file with numbers appended at the end. For this example, I created a testfile.txt under testdupes folder. Content of the file is "Hello World!". Then I copied the file 5 times using the Finder (on Mac).
Organize rule
This is the organize rule I used to match the duplicate files:
- folders: ~/testdupes/**/*
    filters:
      - duplicate
    actions:
      - echo: "{path} is a duplicate of {duplicate}"

Initial test
 ~ % vi testdupes/tesfile.txt
(Files copied on Finder)
 ~ % cd testdupes
testdupes % ls -la
total 48
drwxr-xr-x   8 user  staff   256 27 ene 11:11 .
drwxr-xr-x+ 53 user  staff  1696 27 ene 11:10 ..
-rw-r--r--   1 user  staff    13 27 ene 11:10 tesfile_copy_1.txt
-rw-r--r--   1 user  staff    13 27 ene 11:10 tesfile_copy_2.txt
-rw-r--r--   1 user  staff    13 27 ene 11:10 tesfile_copy_3.txt
-rw-r--r--   1 user  staff    13 27 ene 11:10 tesfile_copy_4.txt
-rw-r--r--@  1 user  staff    13 27 ene 11:10 tesfile_copy_5.txt
-rw-r--r--   1 user  staff    13 27 ene 11:10 tesfile_orig.txt

Then I run the organize sim command:
testdupes % organize sim
False
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ SIMULATION ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Folder /Users/user/testdupes:
  File tesfile_copy_2.txt:
    - [Echo] /Users/user/testdupes/tesfile_copy_2.txt is a duplicate of /Users/user/testdupes/tesfile_copy_1.txt
  File tesfile_copy_3.txt:
    - [Echo] /Users/user/testdupes/tesfile_copy_3.txt is a duplicate of /Users/user/testdupes/tesfile_copy_2.txt
  File tesfile_copy_4.txt:
    - [Echo] /Users/user/testdupes/tesfile_copy_4.txt is a duplicate of /Users/user/testdupes/tesfile_copy_3.txt
  File tesfile_copy_5.txt:
    - [Echo] /Users/user/testdupes/tesfile_copy_5.txt is a duplicate of /Users/user/testdupes/tesfile_copy_4.txt
  File tesfile_orig.txt:
    - [Echo] /Users/cesargarciasaez/testdupes/tesfile_orig.txt is a duplicate of /Users/cesargarciasaez/testdupes/tesfile_copy_5.txt
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ SIMULATION ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Issues detected

Organize only looks for the first duplicate. If you have several duplicates of each other, they won't be detected as dupes of the original.
The order of the sort appear to be on last modified, not on creation date. This can create issues if you ask Organize to delete the duplicates. Instead of ending up with the original file, that file could be deleted and you could end up with one of the copies (but it is not easy to determine the ending name/number).
In this particular example, if I deleted instead of echoing the name, I would have keep just: tesfile_copy_1.txt
Note: I assume you will always retain one of the copies, but I am not sure this could not lead to a circular deletion of all duplicates, including the original file too.

Additional tests
To verify that last modified date is the default sort order, I rename all files but the original one.
testdupes % mv tesfile_copy_1.txt 'testfile_orig 1.txt'
testdupes % mv tesfile_copy_2.txt 'testfile_orig 2.txt'
testdupes % mv tesfile_copy_3.txt 'testfile_orig 3.txt'
testdupes % mv tesfile_copy_4.txt 'testfile_orig 4.txt'
testdupes % mv tesfile_copy_5.txt 'testfile_orig 5.txt'

Then running organize sim again, would leave testfile_orig.txt untouched.
testdupes % organize sim
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ SIMULATION ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Folder /Users/user/testdupes:
  File testfile_orig 1.txt:
    - [Echo] /Users/user/testdupes/testfile_orig 1.txt is a duplicate of /Users/user/testdupes/tesfile_orig.txt
  File testfile_orig 2.txt:
    - [Echo] /Users/user/testdupes/testfile_orig 2.txt is a duplicate of /Users/user/testdupes/testfile_orig 1.txt
  File testfile_orig 3.txt:
    - [Echo] /Users/user/testdupes/testfile_orig 3.txt is a duplicate of /Users/user/testdupes/testfile_orig 2.txt
  File testfile_orig 4.txt:
    - [Echo] /Users/user/testdupes/testfile_orig 4.txt is a duplicate of /Users/user/testdupes/testfile_orig 3.txt
  File testfile_orig 5.txt:
    - [Echo] /Users/user/testdupes/testfile_orig 5.txt is a duplicate of /Users/user/testdupes/testfile_orig 4.txt
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ SIMULATION ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

This test might suggest that last modified file is not tested for duplication. Is this expected behavior?
Additional run, renaming the original file. This would leave us again with testfile_orig 1.txt as the only remaining copy.
testdupes% mv tesfile_orig.txt testfile_true_orig.txt
testdupes % organize sim
False
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ SIMULATION ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Folder /Users/user/testdupes:
  File testfile_orig 2.txt:
    - [Echo] /Users/user/testdupes/testfile_orig 2.txt is a duplicate of /Users/user/testdupes/testfile_orig 1.txt
  File testfile_orig 3.txt:
    - [Echo] /Users/user/testdupes/testfile_orig 3.txt is a duplicate of /Users/user/testdupes/testfile_orig 2.txt
  File testfile_orig 4.txt:
    - [Echo] /Users/user/testdupes/testfile_orig 4.txt is a duplicate of /Users/user/testdupes/testfile_orig 3.txt
  File testfile_orig 5.txt:
    - [Echo] /Users/user/testdupes/testfile_orig 5.txt is a duplicate of /Users/user/testdupes/testfile_orig 4.txt
  File testfile_true_orig.txt:
    - [Echo] /Users/user/testdupes/testfile_true_orig.txt is a duplicate of /Users/user/testdupes/testfile_orig 5.txt
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ SIMULATION ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Final notes
I am aware that I can use the regex filter (and maybe even the filename filter) to deal with duplicates with similar names, but I am not quite sure that the duplicate filter won't cause any data loss without any extra attention in folders with several copies of the same file.

Hey, thank you for the detailed problem description.
First off, a circular deletion of all duplicates should not be possible, one file is always left behind. The only scenario where I can see this happen is if subsequent actions overwrite / delete the left over file because the location of the file detected as the original is unexpected.
Right now the duplicate filter considers whatever file it sees first as the original file. This kind of depends on the filesystem and the python glob.glob() function. I see that this is not ideal so I'm open for ideas. The created date seems like a good option.
To be really flexible, I'm thinking about a ordering parameter like this:
filters:
  - duplicate:
      ordering: "created" # ("lastmodified", "name", "path", custom python function?)
actions:
  - trash

What do you think about this?

Thanks Thomas for your fast and clear response!
Your proposal would be very useful to cover several use cases involving duplicates. I am not sure about how hard would be to implement it. Checking the glob.glob() function, I see it returns a list of paths in arbitrary order. I assume you would pass that list to pathlib or os to check the creation or modification dates. Leaving this open to custom functions would make it more extensible, but I am sure most people would use the 2-3 built-in.
As I first step, may I create a pull request to update the documentation of the duplicate function and make the current behavior explicit in the description? I am looking to contribute documenting practical use cases

Yes, that would be much appreciated! The new ordering parameter will be available in the next version of organize.

Hey, I thought maybe you want to beta test this new feature?
pip3 install git+https://github.com/tfeldmann/organize.git@v2

The documentation for v2 is here:
https://tfeldmann.github.io/organize/filters/#duplicate

Forgot to mention: For v2 slight adjustments in your config are needed:
https://tfeldmann.github.io/organize/updating-from-v1/

The new version is released and has the new detection method selection in the duplicate filter (docs). Feedback would be much appreciated!

Yes you should probably uninstall the brew version. Maybe you can try with python3 -m organize in the meantime.


Make it easier to run service tests against Heroku review apps

Heroku automatically creates review apps for PRs, but it takes a little bit of work to run the service tests against those PRs and, right now, requires a bit of inside knowledge about how the .env configuration files should be used and how to pluck the relevant settings out of Heroku for those tests. It would be great to make it easier to run the tests against these instances to ensure the tests are run before deploying. The bare minimum is to docoument how to run the tests, but perhaps we could also update the wrapper script to provide a picker of instances against which tests can be run.
In an ideal world, the tests would be automatically run on GitHub Actions and would serve to gate PRs, but it's unclear to me how this interacts with Heroku which tries to provide it's own paid-for CI infrastructure. Perhaps this more comprehensive approach might have to come hand-in-hand with a switch to a different hosting service (see #74).

Closing as invalid. Since we want to move to using Docker, the whole setup is likely going to look quite different and be more self contained from the get-go.


IL UP Onboarding Validate Test Data

Problem statement
As a developer, I want to compare the messages generated from the Covid and Universal pipelines, so that I can successfully onboard Illinois to the UP
What you need to know
[Links to documents, workflows, or list of key people and contact info]

Document 1
Document 2
...

Acceptance criteria

[ ] ...

To do

[ ] ...

Duplicate. Closing


Not allowed to upload

I'm attempting to upload a project called "jibe", which does not appear to exist, but it fails with 404 user not allowed. Can you please help?

The project exists, but has no releases. Please file a PEP 541 request at https://github.com/pypa/pypi-support/issues/new/choose

Thanks for looking that up. The PEP 541 submitted: https://github.com/pypa/pypi-support/issues/230


No parenthesis auto-complete w/ es6 intellisense

Expected Behavior

When making a function declaration, after typing function foo I want to type ( and have a pair of parenthesis be inserted, resulting in function foo()

Actual Behavior

Only a single ( gets inserted, leaving me with function foo(

Visual Studio Setup:

NTVS Version: 1.2
Visual Studio Version: Community 2015 w/ Update 3
Node.js Version: 4.4.4
Node es6 intellisense selected

I get a pair of parenthesis inserted when I tried nodejstools on VS2013, and also with VS2015 when I changed the node intellisense to es5. I thought about changing the intellisense mode because of here: https://github.com/Microsoft/nodejstools/issues/1142
I had problems with auto brace completion as well, but the thread didn't talk about paranthesis, so I'm wondering if the issue also has to do with the typescript-powered es6 intellisense the editor uses?

Yes, I believe you are correct that this is another manifestation of #1142. The Typescript bug tracking this feature mentions parens and other elements besides braces.
I'm following up with the Typescript team to verify this and see if there is any pre-release TS build for VS2015 that have this feature enabled.

I'm following up with the Typescript team to verify this and see if there is any pre-release TS build for VS2015 that have this feature enabled.

Will this be possible without TS 2.0.1 or will the feature have to wait 'till after TS 2.0.1 comes out?

@plushy1anthony999 I don't think so. To get brace completion today, you can try NTVS on the VS15 preview. That should have brace completion enabled. Other than that, I'm still trying to work with the Typescript team to get this feature into VS2015 as soon as possible.
I'm closing this as a duplicate of #1142 so that everything is tracked in a single location.
Thanks.


Move spirv

Closes #7

Update with main, please. Linux CI shows that scipy is not installed for testing. It should be fixed in main branch.


Fixed link for provisioner API

1. Issue, if available:

https://github.com/aws/karpenter/issues/1159

2. Description of changes:

fixed broken link.

3. How was this change tested?
4. Does this change impact docs?

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

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

FYI: https://github.com/aws/karpenter/pull/1161

Though yours looks more comprehensive.

Locations of some of the files changed, so I used this information to create PR 1172 I believe we can close this now.

@chrisnegus
I see. Thanks


Add missing AppVeyor config

To avoid it failing all the time. Also added 4.09 in Travis

Will be fixed by an ocurrent Windows CI   (#32 )

Will be fixed by an ocurrent Windows CI   (#32 )


ADS Releases for Mac and Linux are failing due to not being able to find source.tar.gz

ADS Releases for Mac and Linux are failing due to not being able to find source.tar.gz (which we previously stopped building).

Changes were made in the Linux and mac build definitions, and the source artifact is now published. Verified that the insiders build was published on the last successful build.


Update output-clause-transact-sql.md

Add a note to the parallelism section about table variables.

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

@erikdarlingdata Thanks for your contribution. This article was in need of a refresh anyway, so I've incorporated your change into an internal PR. When that one merges, this will automatically close, and you'll see the updated article in the next day or so.


Update sql-machine-learning-services-windows-install-sql-2022.md

I've found people very confused as to how to use named instances with this command. This should clarify by providing an example for the most common option of SQLEXPRESS

@WilliamDAssafMSFT

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thanks @greglow-sdu, we're reviewing internally and will treat these two PRs as one. We may need to bring your commits into an internal PR. Will update.

Thanks @greglow-sdu for taking the time to submit this PR! Good catch, this is an important addition to document behavior that is unintuitive. There are a few other edits needed for your changes and this article as a whole, so I'll bring this commit into a PR from the internal repo. Your commit will be merged and this article update is in flight. After the updated article is published, GitHub will list you as an article contributor.
Thanks again for this feedback, your suggestions help improve our documentation.


Complexity from closures is counted in method complexity

Consider the following code:

public function foo() {
    for ($i = 0; $i < 100; $i++) {

    }

    $x = function() {
        if ($whatever) {

        }
    };
        return $x;
}

The NPath complexity and Cyclomatic Complexity are both calculated as both the foo function's complexity and the anonymous function's complexity. For the purposes of any statistical analysis these should be considered as separate functions. Other tools such as http://jsmeter.info/ correctly handle closures/anonymous functions.

I don't think so, you're case is an exception because it returns 1 closure so we may exclude this closure. But in many case, this would just hide actual complexity:
With the following code:
public function foo() {
    for ($i = 0; $i < 100; $i++) {
        if ($i % 2) {
            echo '.';
        }
    }
}

Changing it using closure this way:
public function foo() {
    $echo = function ($i) {
        if ($i % 2) {
            echo '.';
        }
    };

    for ($i = 0; $i < 100; $i++) {
        $echo($i);
    }
}

Obviously, this does not reduce the complexity of foo() so you should get at least an as big NPath.
The only right way to reduce complexity is to use non-anonymous functions:
private function echo($i) {
    if ($i % 2) {
        echo '.';
    }
}

public function foo() {
    for ($i = 0; $i < 100; $i++) {
        $this->echo($i);
    }
}

I vote for not supporting this change.

Most of the time anonymous functions are used, they are passed into a different method to be called. Your example above would be in the minority, at least where I've seen anonymous functions used. Defining them and calling them in the same location rather defeats the purpose of having them.
Consider:
array_map(function($array) {
//...
}, $array);

It can also be expressed as
array_map([$this, 'funcName'], $array);

but is often less clear. The result is identical and the logic separated in the same way. The anonymous function is a separate code block and should be counted separately. As I noted in the original issue, in JavaScript, where anonymous functions are everywhere, they are counted as their own functions in metrics.

"less clear"? You find anonymous function more clear than named methods?
Returning a closure could be an example of good exception, but your example is exactly the kind of thing the complexity count try to help you to avoid.
If the complexity is counted by method, it's because a method/function is an atomic process you can test on its own, you can give it a name that explain what it does and a PHPDoc that gives even more details about this enclosed process, input and output.
When using a closure in things like array_map, you can't test this closure alone, and this function has no name by definition.
See those both:
$list = [];

foreach ($input as $item) {
  $list[] = $item % 1 ? 'odd' : 'even';
}

And
$list = array_map(function ($item) {
  return $item % 1 ? 'odd' : 'even';
}, $input);

They are obviously the same algorithm, and logically the same complexity, you do not reduce complexity by using array_map over foreach, if you use a tool that tells you it does, sorry but it's wrong, you're just hiding complexity under the carpet.
By doing:
/**
 * Get "odd" or "even" according to given number.
 */
function getOddOrEven($item) {
  return $item % 1 ? 'odd' : 'even';
}

$list = array_map('getOddOrEven', $input);

Here you really reduce complexity by creating a pure reusable and documented function dedicated to an atomic purpose, then calling it in an other function/method that did not take a responsibility it can delegate.
This way, you better respect both Single-responsibility Principle (dedicated function) and Open-closed Principle from the SOLID best-practices.
And it's certainly not less clear. I can't agree your point on this example.

An other example that makes it clear closures do not reduce complexity is nesting:
function parseItems($items) {

return array_map(function ($item) {
  if (has_stuff($item)) {
    do_stuff($item);
  }

  return array_map(function ($subItem) {
    if (has_other_stuff($subItem)) {
      do_other_stuff($subItem);
    }
      return array_map(function ($subSubItem) {
        if (has_other_stuff($subSubItem)) {
          do_other_stuff($subSubItem);
        }

        return $subSubItem;
    }, $subItem);
  }, $item);
}, $items);

}

Are you sure parseItems complexity should be 1? I don't think so.

I see ESLint and JSHint does not count it this way. I don't see the point of using complexity metric if replacing foreach with array_map (or for..in with .map in JS) make your function magically fall under the expected maximum. For me it's completely artificial.
The following:
function foo(items) {
  if (!items) {
    console.log('no items');
  }

  if (items) {
    console.log('items');
  }
}

gives me 3, while:
function foo(items) {
  if (!items) {
    console.log('no items');
  }

  (function () {
    if (items) {
      console.log('items');
    }
  })();
}

gives me 2. It's absurd, I did not reduce the complexity by encapsulating the second statement in an anonymous function. And it's not the definition of the metric
I can't tell you why those tools count it this way, it does not makes sense to me. But as for as PHPMD is concerned regarding its purpose to improve the code, it would be more harmful than helpful in the majority of cases.

I agree with @kylekatarnls here. Complexity of a method is calculated based on its method's body. An anonymous function declared inside a method's body is indeed part of it.
Otherwise the anonymous' function complexity wouldn't be accounted for anywhere. You'd be able to game the system to get CC of 1 by enclosing complex methods in anonymous functions.


smarthome_media_msgs_java: 0.1.81-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository smarthome_media_msgs_java to 0.1.81-0:

upstream repository: https://github.com/rosalfred/smarthome_media_msgs_java.git
release repository: https://github.com/rosalfred-release/smarthome_media_msgs_java-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.21
previous version for package: 0.1.80-0

smarthome_media_msgs_java
* Change package generation with genjava
* Contributors: Erwan Le Huitouze

Check of #11684


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/opstest2/docs-archive-test-source.git

Docs Build status updates of commit 5fe1023:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: RepositoryUnauthorizedWithSourceCommit] Git data access to 'https://github.com/opstest2/docs-archive-test-target' (branch: Release_Archive_main_2022-02-17-01-39-39) is not authorized.. It looks like this might be the first time user: v-alji submitted a pull request to this repo. Before user can run builds in a pull request, he/she can enable 'v-alji' GitHub account in Docs by visiting Docs Portal (https://ops.microsoft.com). After enabling his/her account (v-alji), close and then reopen this pull request to start another build.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Rename Left and Right to A and B

Hi! This is a completely crazy idea and is almost centrally a won't fix, because either is 1.0, but I decided to log it anyway.
I've recently saw an enum Either<T, U> { A(T), B(U) } in warp and now I think that Haskell's left/right terminology is a mistake :D Here are the benefits of A/B

Equal length of variants
Less typing
Nice symmetry between .0, .1 for pairs and ::A, ::B for co-pairs
"Either A or B" sounds more natural then "Either left or right". Like, we have A/B testing and such.
A and B have less semantic meaning than Left, Right.
Naturally extends to more varians: enum ABC<T, U, V> { A(T), B(U), C(V) }.

EDIT: apparently, there's no ab crate yet :thinking:

I like the idea + I wanted to fix a few things here and there for this repo but it seems like this repo is not very active recently.
Wonder if you/we should fork/create a new repo and work on the new ideas? wdyt? @matklad and also @bluss?

You can already rename it on import, at least: use either::Left as A; etc

The crate could also export it as an alternative form.

Nice symmetry between .0, .1 for pairs and ::A, ::B for co-pairs
A and B have less semantic meaning than Left, Right.
Naturally extends to more varians: enum ABC<T, U, V> { A(T), B(U), C(V) }.

For these points, the either_n crate is another possibility, with numeric variants: One, Two, ...

It seems like a matter of taste and not pressing to change after 1.0. Either<L, R> is consistent because the Left parameter is to the left of the right parameter.
Left and Right have more semantic meaning than A, B, in a good way. Left has the antonym Right and you can identify that there are conventionally just two possible outcomes in this space.

@phungleson I don't think this repo should be active. I oppose building an advanced Either framework, this should be a microcrate.


Feat: providing hover effect in bottom sectiion

Describe the feature
hovering on any section like about us will changes it colors to give best user experience
Describe the solution you'd like
on hovering we can change the color
Describe alternatives you've considered
i have no any alternative solutions
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Can you label it as gssoc


Issue with env variables with Docker

I'm not sure why but for some reason when I use this to run code inside a Docker container, it doesn't have access to the env variables I'm passing to Docker. The issue seems to exist only when executed inside a container.
Here's how I set the env for test: docker run -it --env-file ./.env -p 4000:4000 local/tstest
Any ideas?

Thanks for reporting this issue. I can't determine if it's this package or something else broken, could you provide a minimal reproducible example?

Ok, had time to debug this further. Turns out this is a very weird issue with dotenv package. Envs work perfectly with node + ts-directly when run local, and with Bun in docker container also, but with node + ts-directly in docker container dotenv suddenly doesn't get env variables and I need to read from process.env.
Anyways, false alarm and this is resolved :)


92C899EF-4420-92BE-55CA-7BC44042F3D3

[https://github.com/FW4248/axiom.co-vercel-C5BDDDDCF299A249003424DD34C33CBE5DFB277F/blob/3d0ad80ea63a3607d39e7e37691ff1f412bac0e3/yarn.lock
]
(Note: the socket.io.js file is the generated
output of make socket.io.js, and should not be
manually modified.)

The kind of change this PR does introduce

[x] a bug fix

[x] a new feature
[x] an update to the documentation
[x] a code change that improves performance
[x] other

[- [#1  ] (### _matrix Badge  |  active =) [passing]
]

New behaviour
Other information (e.g. related issues)

https://github.com/johnjbarton/socket.io-client/pull[📌=label+](https://github.com/johnjbarton/socket.io-client/pull/1#issue-1277141741) /1#issue-1277141741
#1 (comment)

**> #1 (comment) newhttps://github.com/johnjbarton/socket.io-client/pull/1#issuecomment-1160649320
https://github.com/johnjbarton/socket.io-client/pull/1#pullrequestreview-1012530816**

###  #@**__**

#1 (comment) newhttps://github.com/johnjbarton/socket.io-client/pull/1#issuecomment-1160649320


Example for MS-SQL does not retrieve values properly from SQL Server 2022

sql.query ( )  generates a call to sp_executesql and '$1' is not a valid parameter.
link to sp_executesql documentation:
https://learn.microsoft.com/en-us/sql/relational-databases/system-stored-procedures/sp-executesql-transact-sql?view=sql-server-ver16

Btw,  the create table and insert  statements work just right :-).


Active links in government navigation are hard to read

The active link colour in the government navigation feels hard to read.

Despite feeling hard to read the contrast ratio passes our accessibility tests, with a ratio of 5.8. The blue is significantly lighter than govuk-blue (#1d8feb vs #005ea5) presumably to meet contrast ratios.
http://leaverou.github.io/contrast-ratio/#%230b0c0c-on-%231d8feb
 #1d8feb
 #005ea5

Related thread: https://twitter.com/heydonworks/status/897711844303024130
https://ux.shopify.com/in-plain-sight-5639c9afb4c6 (edited)
http://designworkplan.com/design/signage-and-color-contrast.htm

Closing as header on GOV.UK has been rebuilt.


Implement "dockerd --dns-xxx" based dns mode

This PR implements "dockerd --dns" based DNS as an alternative to dnsmasq.
It can be enabled by settings dns_mode to docker_kubedns. dns_mode is a replacement for skip_dnsmasq and setup_dns.
As I understand it, dnsmasq was initially introduced to make hostNetwork PODs have proper cluster DNS configured. Maybe there were other reasons that I missed, but my feeling is that the solution got too complex for what it tries to solve.
An alternative is to use the docker daemon options --dns, --dns-search and --dns-opt. The docker_kubedns mode will add these to the docker service and skip all other DNS related roles/tasks. This means, that the hosts /etc/resolv.conf and dhclient conf is not touched at all.
Kubelet gets the kubedns/skydns IP as --cluster_dns, so all normal (non-hostNetwork) PODs will generate their own resolv.conf that has proper cluster DNS set up. PODs that have 'hostNetwork: true' set, will rely on docker providing a correct resolv.conf, which will contain proper cluster DNS settings as well (due to --dns-xxx).
The docker_kubedns mode also parses the hosts resolv.conf to figure out which nameservers and search domains are used. These are added as proper --dns-xxx args as well. These nameservers come after the kubedns entry, which makes them backup nameservers. This basically makes the early_dns logic unnecessary, as the hosts resolv.conf is untouched and only docker containers are affected (lookup still works, but takes longer) by the not running cluster DNS server.
The long term goal of this PR is to maybe completely remove dnsmasq and all the modifications to resolv.con and dhclient.conf and then leave only the docker_kubedns mode. This would simplify a lot of things and remove a big source of error.
The default dns_mode is set to dnsmasq_kubedns, which is the same as skip_dnsmasq=false and setup_dns=true.
I still need to update the documentation. Will do this after review.

The docker_kubedns mode also parses the hosts resolv.conf to figure out which nameservers and search domains are used. These are added as proper --dns-xxx args as well. These nameservers come after the kubedns entry, which makes them backup nameservers.

The dnsmasq ds should still be an option to leave with the docker_mode. There is a limitation for max nameservers in resolv.conf, and one is taken vy k8s cluster dns ip, either kubedns/skydns or dnsmasq ds.
So I suggest to allow users use both docker_mode and  leave skip_dnsmasq: False as is.

As already asked on Slack: Are there really use cases for more then 3 nameservers in resolv.conf? Even if it's reduced to 2 (due to skydns entry), why would anyone need more then 2?
Supporting rkt later should not be a problem as it supports the same dns flags.

I just figured out that host networks with --dns only work with docker >= 1.12. https://github.com/docker/docker/pull/22408 added a the needed bit to make it work.
The strange thing is, that kargo always installed 1.12 for me even though docker_version is set to 1.12. I'm on CentOS 7 but I'd expect the same is true for RedHat. CI already passed for coreos-calico-sep and centos7-flannel. ubuntu-weave-sep is currently running and I expect it to also pass as the logs already show that docker 1.12 is installed on the nodes.
So...there seems to be a bug that ignores the docker_version variable. What should we do about it? And also, which version should we actually install?

Another thing: When https://github.com/kubernetes/kubernetes/pull/29378 gets merged (I'd expect it for k8s 1.6.) all this dns stuff in kargo may get obsolete as it would allow to specify "dnsPolicy: ClusterFirstAlways" for hostNetwork PODs.

Just committed the changes. I'm still testing out if all combinations work as expected.

Please document the changes to the docs/dns-stack.md. Describe how removed dns_setup and skip_dnsmasq map to the new dns_mode and resolvconf_mode and hostnet_dns_mode

I'll document it after I have retested everything.

As asked by @bogdando, here is an updated table of supported modes:

dns_mode
resolvconf_mode
what it does

dnsmasq_kubedns (default)
docker_dns (default)
hosts' unconfigured, docker defines host containers' DNS, dnsmasq is prim resovler

kubedns
docker_dns (default)
hosts' unconfigured, docker defines host containers' DNS, skydns is prim

dnsmasq_kubedns (default)
host_resolvconf
hosts' resolv.conf configured, docker doesn't, dnsmasq is prim resovler for pods

kubedns
host_resolvconf
hosts' resolv.conf configured, docker doesn't, skydns is prim resovler

none
ignored
DNS stack is not configured nor for hosts, not docker, nor kubelet and there is no kubedns rc or dnsmasq ds as well

Perfect now, thanks!

Rerun CI. Please squash when done.

I've just updated the documentation to match the new modes. I hope I covered everything.
I removed the dns.jpeg figure as I'm not able to update the figure. @bogdando I'm not able to update this figure as I don't know how you've created it. Should we actually keep it up to date? To be honest, it was pretty hard to understand the DNS stack from the figure, the documentation and code was actually of much more help.

CI has kind of passed now. I had to slightly change .gitlab-ci.yml to use the classic host_resolvconf mode for the current CoreOS stable builds as those don't have docker 1.12 yet. This can be removed when CoreOS beta is promoted to stable.
To make sure this works as expected I also manually started the coreos-alpha-weave-ha build which uses the new docker_dns mode. Unfortunately, it turned out that this build is broken on master already. But it looks like docker dns related stuff worked as expected.
I'd say this PR is ready to be merged finally.

ci check this

Thank you @codablock for a great work done. I liked the docs update and I agree the drawing may be omitted. Let's keep it as an exercise for those who would like to do some reverse engineering.

Please do not merge. I got strange behavior right now and need to figure out what's wrong

@codablock please also rebase for the rkt support. Do you think we should prohibit to use any of the kubelet_deployment_type: rkt, etcd_deployment_type: rkt  with the resolvconf_mode=docker_dns?

I'd expect that rkt would have no influence on the rkt mode, as it does only deploy kubelet and etcd with rkt and these do not require working cluster DNS. This would be a completely different story if kubernetes would be deployed in the form of rktnetes (where kubelet actually uses rkt to spawn PODs).
Or am I wrong here and the rkt mode already switches kubelet into rktnetes mode?

ci check this

ci test this

Ok, the problem I found is fixed now. PODs were not able to lookup non-cluster names anymore on Azure because dnsmasq was forwarding these queries to <IP_ADDRESS> (default_resolver). This did not occur on GCE/CI as there is some logic in kargo to use the GCE nameserver as default_resolver.
I added a commit on top that changes dnsmasq to not use the default_resolver when the resolvconf_mode is not host_resolvconf. It will also only set no-resolv option in dnsmasq for the host_resolvconf case.
As I understand it, no-resolv was needed to avoid a loop in lookup as in the past the resolv.conf inside the dnsmasq container was using the hosts resolv.conf, which contained dnsmasq itself.
Now that the hosts resolv.conf is not touched with docker_dns, dnsmasq can rely on the hosts resolv.conf for non-cluster name lookup.
@bogdando Can you review the the commit on top? I also rebased on master, so rkt is included now. All tests passed as expected.


JS files in /assets/bootstrap.zip seems to be broken

The file:
/js/bootstrap.js
In this zip file: http://twitter.github.com/bootstrap/assets/bootstrap.zip is giving me this two errors when I included in my html.
http://cl.ly/3l1k3H363F0Y0W0F2X0d (with jQuery)
http://cl.ly/3G3U0N3O2x0G1Z321v3E (no jQuery)
PD. The bootstrap.min.js is giving some error too.

Nevermind the problem was the version of jQuery, that required 1.7.1


Bring back "Force refresh all my feeds" in 3.0

Also, seems like API request /rest/feed/refreshAll doesn't trigger the refreshing

Done! I also added a websocket connection to be able to notify the client that there are new entries and that the tree has to be reloaded. This way, the tree reloads immediately instead of waiting for the next polling event (up to 30 seconds).

Thanks!


RuntimeError: Dimension out of range (expected to be in range of [-2, 1], but got 2)

I am trying to use the Dynamic Pair encoder for some task, I am facing some issue.
In modules/pair_encoder/cells.py, the new_input that is generated has dimension 2
new_input = torch.cat([inputs, attention_vector], dim=-1)
but in modules/dropout.py the RNNDropout expects the dimension of 3
mask = inputs.new_ones(inputs.size(0), 1, inputs.size(2), requires_grad=False)
Am I giving wrong input somewhere?

Thanks for the reply,
torch.nn.Dropout worked for me, and I didn't modify the code. RNNDropout is used in Gate Module   which is used in _PairEncodeCell Module
 new_input = self.gate(new_input)
and this cell is a part of DynamicPairEncoder
super().__init__(memory_size, input_size, hidden_size, attention_size,
                         bidirectional, dropout, PairEncodeCell, batch_first=batch_first)

I think I put gate in a wrong location. Could you pull the latest code and try again?
BTW, Dynamic Pair encoder is still a work in progress....so it might not work as we expect.


[Issue]: clang: error: cannot find HIP runtime

Problem Description
Just installed ROCM and it doesn't work:
user@host :~/gpu/snippets/asm1$ /opt/rocm/bin/hipcc --save-temps -o test1 test1.cpp
clang: error: cannot find HIP runtime; provide its path via '--rocm-path', or pass '-nogpuinc' to build without HIP runtime
clang: error: cannot find HIP runtime; provide its path via '--rocm-path', or pass '-nogpuinc' to build without HIP runtime
user@host :~/gpu/snippets/asm1$

another attempt:
user@host :~/gpu/snippets/asm1$ /opt/rocm/bin/hipcc --rocm-path=/opt/rocm -o test1 test1.cpp
clang: error: cannot find HIP runtime; provide its path via '--rocm-path', or pass '-nogpuinc' to build without HIP runtime
clang: error: cannot find HIP runtime; provide its path via '--rocm-path', or pass '-nogpuinc' to build without HIP runtime
user@host :~/gpu/snippets/asm1$

I just downloaded amdgpu install and ran:
 amdgpu-install --usecase=graphics,rocm
didn't do anything extraordinary
 user@host :~/gpu/snippets/asm1$ /opt/rocm/bin/hipcc --version
HIP version: 5.7.0-0
AMD clang version 17.0.0 (https://github.com/RadeonOpenCompute/llvm-project roc-6.0.2 24012 af27734ed982b52a9f1be0f035ac91726fc697e4)
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /opt/rocm/llvm/bin
Configuration file: /opt/rocm-6.0.2/lib/llvm/bin/clang++.cfg

Operating System
Ubuntu 22.04.4 LTS
CPU
Ryzen 7 3700X
GPU
AMD Radeon RX 7900 XT
ROCm Version
ROCm 6.0.0
ROCm Component
HIP
Steps to Reproduce
Follow official installation guide
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

additional note:
I have installed aomp-19 , it compiled clean and runs without any issues, so i was able to compile examples with this command:
 DEVICE_LIB_PATH=/home/user/rocm/aomp/amdgcn/bitcode /home/user/rocm/aomp/bin/hipcc --save-temps test1.cpp  -o test1 -std=c++11 -I/home/user/rocm/aomp/include -O1
So, I thought, maybe I will run the same command with default ROCm installation that is located in /opt? Nope, it doesn't compile:
 user@host :~/gpu/snippets/asm1$ ROCM_PATH=/opt/rocm DEVICE_LIB_PATH=/opt/rocm-6.0.2/lib/llvm/lib/clang/17.0.0/lib/amdgcn/bitcode /opt/rocm-6.0.2/bin/hipcc --save-temps test1.cpp  -o test1 -std=c++11 -I/opt/rocm-6.0.2/include -O1
clang: error: cannot find HIP runtime; provide its path via '--rocm-path', or pass '-nogpuinc' to build without HIP runtime
clang: error: cannot find HIP runtime; provide its path via '--rocm-path', or pass '-nogpuinc' to build without HIP runtime
user@host :~/gpu/snippets/asm1$

With AOMP it works, with default ROCm it doesn't.

Internal ticket has been created for investigation.

@nuliknol, thanks for getting back.  I will close the issue now.

Can you give more light on what environment variables are those?

Got same error here, my .bashrc has:
export PATH=$PATH:/opt/rocm/bin
export HIP_PLATFORM=amd
export USE_ROCM=1
export ROCM_PATH=/opt/rocm
export HIP_CLANG_PATH=/opt/rocm/llvm/bin
export DEVICE_LIB_PATH=/opt/rocm/lib
export LD_LIBRARY_PATH=/opt/rocm/lib:$LD_LIBRARY_PATH
export HIP_PATH=$ROCM_PATH/hip
export ROCM_DEVICE_LIB_PATH=$ROCM_PATH/lib
and hipconfig --cpp_config:
-D__HIP_PLATFORM_HCC__= -D__HIP_PLATFORM_AMD__= -I/opt/rocm/hip/include -I/opt/rocm-6.1.3/lib/llvm/lib/clang/17
Is there anything wrong here?


[BUG] : DcsBios::Potentiometer min & max values are not limiting the travel of the in-game lever

DCS-BIOS version
DCS Unified 2.9.3

DCS-BIOS Control
PLT_FLAPS_LEVER when using DcsBios::Potentiometer

Describe the bug

Using a 100k linear potentiometer on a Leonardo board, connected to Analog port A0
DcsBios::Potentiometer function being used for the PLT_FLAPS_LEVER and a min & max value of 0 & 500 for F14
In-game, the flaps lever starts to move from the closed position to the open position. When the open (max travel in-game) is reached, the flap lever quickly snaps to the closed position - the function seems to causing the lever to continuously loop back to the closed position each time the in-game max is reached

To Reproduce

Arduino code :

#define DCSBIOS_DEFAULT_SERIAL
#include "DcsBios.h"

void setup() {
// the flaps lever at 100% or 65535 is up (flaps up)
  DcsBios::Potentiometer pltFlapsLever("PLT_FLAPS_LEVER", A0,false,700,1023);
  DcsBios::setup();
}

void loop() {
  DcsBios::loop();
}

fire up DCS-Bios Bridge or the socat cmd file to communicate with the Leonardo port
Run DCS, Instant Action, F14 tomcat anywhere, in-cockpit, move pot whilst looking at flaps lever

Expected behavior

once the max value is reached, the movement should stop & not loop around
Screenshots

n/a
Additional context

Tested with bort and appears to behave as expected. Moving this to the arduino repo

not sure what you mean "behave as expected"?
Is the function, when supplied with a min/max, supposed to reset to the minimum when maximum is reached?
Or do you mean the PLT_FLAPS_LEVER call does move the flaps so the call behaves as expected?


Avoid quadratic complexity when splitting output into lines

When searching for newlines in a process output keep track which part of
buffer was already examined to avoid processing the same data again and
again.

r? @alexcrichton
(rust-highfive has picked a reviewer for you, use r? to override)

Thanks for this! Can you describe how you saw this arise as well? I'm curious why this hasn't been caught before or if it's only a niche-ish situation where the perf matters.
Additionally would you be up for separating out this structure to a separate thing which can be unit-tested? I fear that the internals of the management of lines here has grown with complications to the point that it's probably best to have unit tests to ensure we don't accidentally regress things.

I encountered this while debugging an issue in rustc, more than once in fact. Each time I had a logging enabled which generated very long lines. The end result was a cargo stuck while using 100% CPU without making any visible progress.
I would rather not take a job of refactoring this to the point where it can be unit tested.

Hm ok, well unfortunately I'm not personally confident enough in this code to sign off on it. I would be much more confident with some unit tests showing what's happening.
I'd be more ok landing this if it were a pressing problem, but if it's mostly just debugging rustc then while I think this is important to fix it's best to fix it in a way to that doesn't run the risk of regressing other use cases in Cargo.

I introduced an intermediate refactoring, so that there is only a single place that drains the data buffer. With that in place, the optimization itself is straightforward.
Could you take another look? If you consider that too risky still, that's fair of course.

@bors: r+

:pushpin: Commit 8df771f3815138ed7423e767ff5c8ce332bcc395 has been approved by alexcrichton

:hourglass: Testing commit 8df771f3815138ed7423e767ff5c8ce332bcc395 with merge 9a3c162dc3b912cde1a0f436958adb2cb8f21e5e...

:sunny: Test successful - checks-actions
Approved by: alexcrichton
Pushing 9a3c162dc3b912cde1a0f436958adb2cb8f21e5e to master...


ListSatus should check LIST ACL

What changes were proposed in this pull request?
ozone cli

[hadoop@presto-test-006 ~]$ om-current/bin/ozone sh vol addacl -a user:test:r vol-1-29165
ACL user:test:r[ACCESS] added successfully.
[hadoop@presto-test-006 ~]$ om-current/bin/ozone sh  bucket addacl -a user:test:r vol-1-29165/bucket-188-54163
ACL user:test:r[ACCESS] added successfully.
[hadoop@presto-test-006 ~]$ export HADOOP_USER_NAME=test
[hadoop@presto-test-006 ~]$ om-current/bin/ozone sh key list --prefix=h vol-1-29165/bucket-188-54163/
PERMISSION_DENIED User test doesn't have LIST permission to access bucket Volume:vol-1-29165 Bucket:bucket-188-54163 Key:h

[hadoop@presto-test-006 ~]$ om-current/bin/ozone sh bucket list --prefix=b vol-1-29165/
PERMISSION_DENIED User test doesn't have LIST permission to access volume Volume:vol-1-29165

<EMAIL_ADDRESS>~/hadoop-2.7.2-5504-ozone-client]$ export HADOOP_USER_NAME=test
<EMAIL_ADDRESS>~/hadoop-2.7.2-5504-ozone-client]$ ./bin/hadoop fs -ls /vol-1-29165/bucket-188-54163/h*
-rw-rw-rw-   3 test test        243 2023-05-08 14:13 /vol-1-29165/bucket-188-54163/hosts

hadoop cli

listKeys(){
	if (isAclEnabled) {
        checkAcls(ResourceType.BUCKET, StoreType.OZONE, ACLType.LIST,
            bucket.realVolume(), bucket.realBucket(), keyPrefix);
      }
}

listStatus() {
	if (isAclEnabled) {
        checkAcls(getResourceType(args), StoreType.OZONE, ACLType.READ,
            bucket.realVolume(), bucket.realBucket(), args.getKeyName());
      }
}

In the listKeys method，ACLType is LIST, but in the listStatus method, ACLType is READ
So it causes inconsistent return information.
I think listStatus should also check LIST.
https://issues.apache.org/jira/browse/HDDS-8566

@swamirishi could you please help to review it?


Test-ComputerSecureChannel not working in 7.1.0

Steps to reproduce
PWSH 7.1.0  ---  PS C:\Program Files\PowerShell\7> Test-ComputerSecureChannel

Testing in 5.1 comes back True

Expected behavior
Result: True or False

Actual behavior
Test-ComputerSecureChannel: The term 'Test-ComputerSecureChannel' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Environment data

Name                           Value
----                           -----
PSVersion                      7.1.0
PSEdition                      Core
GitCommitId                    7.1.0
OS                             Microsoft Windows 10.0.19042
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

This command is loaded here just fine in 7.1.
This command is in the Microsoft.PowerShell.Management module, V1, and is NOT in the V <IP_ADDRESS> version (the version shipped with PowerShell 7. So by default you won't see the cmdlet or get tab completion. However, if you spell it out fully, PowerShell loads the V1.0 module (via compatibility), so you get a proxy function of that same name.
Here is what I see (in 7.1):
PS C:\Foo> gmo

ModuleType Version    PreRelease Name                                ExportedCommands
---------- -------    ---------- ----                                ----------------
Manifest   <IP_ADDRESS>               Microsoft.PowerShell.Management     {Add-Content, Clear-Content, Clear-Item, Clear-It…
Manifest   <IP_ADDRESS>               Microsoft.PowerShell.Utility        {Add-Member, Add-Type, Clear-Variable, Compare-Ob…

PS C:\Foo> gcm -mod Microsoft.PowerShell.Management | where name -match secure
PS C:\Foo> test-computersecurechannel
True
PS C:\Foo> gmo

ModuleType Version    PreRelease Name                                ExportedCommands
---------- -------    ---------- ----                                ----------------
Manifest   <IP_ADDRESS>               Microsoft.PowerShell.Management     {Add-Content, Clear-Content, Clear-Item, Clear-It…
Script     1.0                   Microsoft.PowerShell.Management     {Add-Computer, Checkpoint-Computer, Clear-EventLo…
Manifest   <IP_ADDRESS>               Microsoft.PowerShell.Utility        {Add-Member, Add-Type, Clear-Variable, Compare-Ob…

PS C:\Foo> gcm -mod Microsoft.PowerShell.Management | where name -match secure

CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Function        Test-ComputerSecureChannel                         1.0        Microsoft.PowerShell.Management

/cc @anmenaga for information.

so in order to use it, we have to use:
import-module Microsoft.PowerShell.Management -UseWindowsPowerShell
?
Thanks,
Dennis Fair  |  I.T Systems Admin
Four Peaks Landscape Management
p:<PHONE_NUMBER>tel:480-967-0376  |  f:<PHONE_NUMBER>
m:<PHONE_NUMBER>tel:480-319-5345
e: it<EMAIL_ADDRESS>[http://fourpeakslandscape.com/images/four-peaks-landscape-management.png]

From: Thomas Lee<EMAIL_ADDRESS>Sent: Wednesday, November 18, 2020 10:54 AM
To: PowerShell/PowerShell<EMAIL_ADDRESS>Cc: Ninja RMM<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [PowerShell/PowerShell] Test-ComputerSecureChannel not working in 7.1.0 (#14123)
This command is loaded here just fine in 7.1.
This command is in the Microsoft.PowerShell.Management module, V1, and is NOT in the V <IP_ADDRESS> version (the version shipped with PowerShell 7. So by default you won't see the cmdlet or get tab completion. However, if you spell it out fully, PowerShell loads the V1.0 module (via compatibility), so you get a proxy function of that same name.
Here is what I see (in 7.1):
PS C:\Foo> gmo
ModuleType Version    PreRelease Name                                ExportedCommands

Manifest   <IP_ADDRESS>               Microsoft.PowerShell.Management     {Add-Content, Clear-Content, Clear-Item, Clear-It…
Manifest   <IP_ADDRESS>               Microsoft.PowerShell.Utility        {Add-Member, Add-Type, Clear-Variable, Compare-Ob…
PS C:\Foo> gcm -mod Microsoft.PowerShell.Management | where name -match secure
PS C:\Foo> test-computersecurechannel
True
PS C:\Foo> gmo
ModuleType Version    PreRelease Name                                ExportedCommands

Manifest   <IP_ADDRESS>               Microsoft.PowerShell.Management     {Add-Content, Clear-Content, Clear-Item, Clear-It…
Script     1.0                   Microsoft.PowerShell.Management     {Add-Computer, Checkpoint-Computer, Clear-EventLo…
Manifest   <IP_ADDRESS>               Microsoft.PowerShell.Utility        {Add-Member, Add-Type, Clear-Variable, Compare-Ob…
PS C:\Foo> gcm -mod Microsoft.PowerShell.Management | where name -match secure
CommandType     Name                                               Version    Source

Function        Test-ComputerSecureChannel                         1.0        Microsoft.PowerShell.Management
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/PowerShell/issues/14123#issuecomment-729850279, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQ6MMPNU2PBVHCRJBYOUYW3SQQC5RANCNFSM4T2FYWPQ.


Auto-login option for cosmic-greeter module

I use NixOS with LUKS encryption, meaning I have to decrypt my drive on boot. It would be nice to have the option to skip the password prompt before logging into cosmic, if possible, so I don't have to enter two passwords every reboot.

The normal NixOS autologin options work with greetd, which cosmic-greeter uses. Feel free to re-open if those aren't working for some reason!

Apologies, but if I may ask, how would I set up the autologin? I know there's a section for it on the wiki but that seems to be specifically for setups where the user is using greetd as the sole login system instead of it being the underlying one.

This is a bit of a weird one. On my laptop with auto login set like so:
services = {
  displayManager = {
    autoLogin = {
      enable = true;
      user = "adamc";
    };
  };
};

auto login doesn't work, but with auto login set the same way on the COSMIC Live ISO I build it works. The only difference is that my laptop is pinned to NixOS 24.11 and the ISO's are pinned to NixOS Unstable.

Oh I think I dreamed yesterday evening that greetd followed those options like every other greeter in NixOS (this is what I get for working while sick). Apologies for that...
@drakon64 The ISO profile probably overrides greeter to use one which does have working autologin
Is there any reason to use cosmic-greeter at all if you're gonna just be doing autologin?
Also,

where the user is using greetd as the sole login system instead of it being the underlying one.

I'm not sure what this means, but all greetd greeters more or less do logins the same way and instruct which session command to start to the greetd daemon. The autologin config would be on greetd daemon if anywhere (and you'd have to use session command start-cosmic)


Adds support for nested components

TODO:

[ ] Get path dynamically
[ ] Add tests for this scenario
[ ] Verify that we don't break everything else
[ ] Add support for pods

@MiguelMadero rebase at will my friend - just pulled in the very latest v0.1.7 code that solves part of this :)


vsmarketplacebadge.apphb.com

Blocking apphb.com blocks badges in VS Code.
Connected to: AppHarbor
AppHarbor is a fully hosted .NET Platform as a Service. AppHarbor can deploy and scale any standard .NET application to the cloud.
Reference: https://github.com/Microsoft/vscode/issues/46993

I've seen strange scripts on some shopify shops. But, in general, you are right, apphb should not be blocked as whole platform.


[BUG] system-probe trying to mount /usr/src

Agent Environment

version: 7.45.0
Describe what happened:
system-probe container is failing with failed to generate container \"6c1f3280449a05abc7aaabdb5048cc9a7b6782473100b515fbf42f83b8a0a14a\" spec: failed to generate spec: failed to mkdir \"/usr/src\": mkdir /usr/src: read-only file system error message
Describe what you expected:
system-probe should start
Steps to reproduce the issue:
i noticed that the host does not have /usr/src dir, which is what the agent is trying to mount, that might be the reason.
Additional environment details (Operating System, Cloud provider, etc):
gke 1.24.12-gke.1000

found fix here: https://github.com/DataDog/helm-charts/issues/829


GEOMESA-796 Enable user-specific roles in GeoServer

A trait to mix into a GeoServerRoleService to create a GS role for each user on the fly,
which can be used to secure layers.

Closing - we need to remove all geoserver-specific code due to IP issues.


enhancement enble exception logic in pipeline execute thread

Proposed changes
Pipeline engine's code is exception safe, we could enable exception now.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Remove "clear history" entry

Hi,
I suggest inserting an option in the program preferences to be able to choose whether to display the key in the menu.
thanks

What is the reason to?

Thanks for the reply. It would be useful to insert this possibility of choice because it could be convenient for someone to reduce the number of entries in the menu and the possibility of deleting the history by mistake.
in fact, I suggest inserting an option in the settings to be able to choose whether or not to display the entry to the main menu for cleaning the history. By default this option remains as it is now, but it will only be activated by those who deem it useful.
Thanks to the availability


OrderBy/ThenBy clause incorrect with expanded properties

I'm trying to order by result set by a couple of expanded properties
 var result = await Client.For<RoleMember>()
                 .Expand(x => new { x.Role, x.Party })
                 .OrderBy(x => x.Role.Name)
                 .ThenBy(x => x.Party.Name)
                 .FindEntriesAsync();

what gets requested is
 GET: http://localhost/odata/RoleMember?$expand=Role,Party,Asset&$orderby=Role/Name,Party/Name&$orderby=Role/Name,Party/Name

which generates a 404 probably due to the duplicated orderby clause.
If I remove the ThenBy clause I get
GET: http://localhost/odata/RoleMember?$expand=Role,Party&$orderby=Role/Name

which returns the data correctly

This is entirely true.
Same here also.
This should be resolved ASAP.

I heavily use Simple.OData.Client in my windows desktop application and faced the same issue. I resolved the issue for my case, you can try using my version with the fix from https://github.com/haharoi/Simple.OData.Client.
Pull request created: https://github.com/object/Simple.OData.Client/pull/354.

I heavily use Simple.OData.Client in my windows desktop application and faced the same issue. I resolved the issue for my case, you can try using my version with the fix from https://github.com/haharoi/Simple.OData.Client.
Pull request created: https://github.com/object/Simple.OData.Client/pull/354.

I suppose this is fixed in 4.25.0.


why not support for cpu cluster?

I dont't have a gpu cluster but cpu, and why not support for cpu cluster?

It can also support cpu cluster. You don't have to specify --gpu when you don't need GPU.

But in cpu cluster we couldn't see any cpu information(total/allocated) in top command :scream:

Thanks for reporting it. It will support later. And your contributions are also welcome!

Hi @cheyang , I have deployed arena successfully on my small cpu cluster, but the tf-job always failed on my cluster, here is the reproduce step:
1. check the cluster status
# ./arena top node
INFO[0000] Use specified kubeconfig file /etc/kubernetes/admin.conf
NAME      IPADDRESS     ROLE    GPU(Total)  GPU(Allocated)
k8s-mst   <IP_ADDRESS>  master  0           0
k8s-nod1  <IP_ADDRESS>  worker  0           0
k8s-nod2  <IP_ADDRESS>  worker  0           0

2. run one tf job
# ./arena submit tf --name=tf-git5 --image=tensorflow/tensorflow:1.5.0-devel --syncMode=git --syncSource=https://github.com/cheyang/tensorflow-sample-code.git "python code/tensorflow-sample-code/tfjob/docker/mnist/main.py --max_steps 10000"
INFO[0000] Use specified kubeconfig file /etc/kubernetes/admin.conf

NAME:   tf-git5
LAST DEPLOYED: Tue Aug  7 16:31:56 2018
NAMESPACE: default
STATUS: DEPLOYED

RESOURCES:
==> v1alpha2/TFJob
NAME           AGE
tf-git5-tfjob  0s

3. check the job status
./arena get tf-git5
INFO[0000] Use specified kubeconfig file /etc/kubernetes/admin.conf
NAME     STATUS  TRAINER  AGE  INSTANCE                NODE
tf-git5  FAILED  TFJOB    0s   tf-git5-tfjob-worker-0  N/A

4. can't see the detailed logs because the pod status is failed.
# ./arena logs tf-git5
INFO[0000] Use specified kubeconfig file /etc/kubernetes/admin.conf
pod 'tf-git5-tfjob-worker-0' has not been started.

it seems that arena hasn't implemented describe command yet, so I can't get concrete failing reason. :cry:
I don't konw how to solve this probelm, any help will be appreciated!

Try kubectl describe po tf-git5-tfjob-worker-0 to see why the pod is not started successfully.

I guess it's failed to download source code from git.
State:          Terminated
      Reason:       Error
      Exit Code:    1

You can check logs by using kubectl logs tf-git5 -c init-code.

I have the same problem as @xiechengsheng , my k8s version is 1.11.0.
# kubectl get po -n default

NAME                    READY     STATUS       RESTARTS   AGE
tf-git-tfjob-worker-0   0/1       Init:Error   0          41m

# arena get tf-git

NAME    STATUS  TRAINER  AGE  INSTANCE               NODE
tf-git  FAILED  TFJOB    0s   tf-git-tfjob-worker-0  N/A

But later I deployed arena on playground and it worked fine! The only difference is that the playground's k8s version is 1.10.0, and maybe the network of the playground is better than my cluster?

It seems the reason failing to download source code from git is right 😢
# kubectl logs tf-git-tfjob-worker-0 -c init-code -n default
I0808 07:18:17.469610   32274 main.go:164] starting up: ["/git-sync"]
E0808 07:29:46.803093   32274 main.go:171] error syncing repo: error running command: exit status 128: "Cloning into '/code'...\nerror: RPC failed; curl 18 transfer closed with outstanding read data remaining\nfatal: The remote end hung up unexpectedly\nfatal: early EOF\nfatal: index-pack failed\n"

Looks like it's connection issue between your machine and github.com. You can try https://www.katacoda.com/courses/kubernetes/playground  . I think there will be no connection issue.

Yes, after fixing my kube-dns component problem, tf job runs successfully, thanks!

Ok, I will close this issue first, and track with #5


naive fix for debian 9

The mongo package does not exist for debian 9.
I changed the regexp to use 71 (as for version 7 and 8) instead of legacy

There is a "81" version for Debian 8.1+

Looks good to me, thanks. This bit me when I upgraded to Debian 9, let's hope your PR gets merged soon ;)

Added finer release name resolution.
Still i get error because the library appear to download from fastdl.mongodb.org instead of downloads.mongodb.org


Add types file #8

Hi, I found the issue here and thought that I'd give it a try. Nothing too different there, but including this in a typescript project should work now :).

Awesome job @kristapsstrals, this is great! Thanks so much for this! Thank you for contributing and hope to see you around for more open source 🎉

Thanks, @joshghent! I'll definitely Will look around other open source issues! Might look at other issues in this repo as well 👍


Personal portfolio Website Using Html & CSS

Simple and Impressive Single page portfolio

@iashupandey Provide the screenshots of your projects for overview

Sure.. I'll provide
On Mon, 4 Oct, 2021, 6:22 pm Abhishek Kumar, @.***>
wrote:

@iashupandey https://github.com/iashupandey Provide the screenshots of
your projects for overview
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ZeroOctave/ZeroOctave-Javascript-Projects/pull/87#issuecomment-933452309,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARGMS37735LP3GLSGUEIXGDUFGPPJANCNFSM5FGNVKDA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I had added the screenshot and also updated the code.


Add asserts for DeserializeSeed

Equivalent asserts that accept a seed to test stateful deserialization. Was requested in #2262.
Not sure about the examples for these methods. Any minimal example for DeserializeSeed is pretty big, so I decided not to add it. Let me know if this is ok.
Also not sure if it's better to take seed as the last argument rather than the first.

Could you please add a usage example to the documentation of each of the new functions?

@dtolnay Sure, added.

But nothing in there is using any private internals of serde

Deserializer is pub, but it's a private module, I can't use it in a third-party crate.


Investigate enabling idempotency for the core MLZ deployment

Benefit/Result/Outcome
So that an IT Admin can manage ongoing MLZ configuration changes from Bicep code.
Description
Deploying MLZ a second time without modifications results in a deployment error because at least one resource provider tries to destroy and re-create resources rather than performing an update.
For example, a second deployment of MLZ to the same subscription with no changes in parameters or Bicep code results in this error:
{
    "status": "Failed",
    "error": {
        "code": "InUseSubnetCannotBeDeleted",
        "message": "Subnet <subnet ID> is in use by /subscriptions/<subscription id>/resourceGroups/<resource group name>/providers/Microsoft.Network/networkInterfaces/<network interface name>/ipConfigurations/<private endpoint name> and cannot be deleted. In order to delete the subnet, delete all the resources within the subnet. See aka.ms/deletesubnet.",
        "details": []
    }
}

The goal of the spike is to determine if we can make simple changes to the structure of the Bicep code to enable idempotent deployments.
**Work on this spike is limited to: 16 hours
Possible Outcomes
The spike will result in either one of the following outcomes (not both):

Simple changes are made to the Bicep code that enable idempotent deployments.
We know that #658 and other work (to be defined) will be required to enable idempotent deployments.

Note: some ARM idempotency issues may be related to calculated names of the deployments. If the names change then ARM sees it as a new resource.

Can very likely be implemented in bicep using conditional resource deployment; ie:
@allowed([
  'new'
  'existing'
])
param newOrExisting string = 'new'

resource sa 'Microsoft.Storage/storageAccounts@2019-06-01' = if (newOrExisting == 'new') {
  name: storageAccountName

https://docs.microsoft.com/en-us/azure/azure-resource-manager/bicep/conditional-resource-deployment#new-or-existing-resource

This error showed up again from a customer implementation who is looking to use MLZ Bicep as the "root" stamp and deploy on top of it with new resources:
{
    ...
    "status": "Failed",
    "error": {
        "code": "InUseSubnetCannotBeDeleted",
        "message": "Subnet <subnet ID> is in use by /subscriptions/<subscription id>/resourceGroups/<resource group name>/providers/Microsoft.Network/networkInterfaces/<network interface name>/ipConfigurations/<private endpoint name> and cannot be deleted. In order to delete the subnet, delete all the resources within the subnet. See aka.ms/deletesubnet.",
        "details": []
    }
}

+@mikedzikowski

This is the resource that caused this error for my team. Relevant doc we found.
My best guess is that this is defined this way to avoid a circular dependency with the Firewall Private IP for the Subnet route table. Hard coding the Firewall Private IP is a workaround that lets this child resource be pulled into the parent resource definition.
https://github.com/Azure/missionlz/blob/f261f1d92edacde43c035cf6da2512e036c12655/src/bicep/modules/hubNetwork.bicep#L154-L172

Please consider the concerns in #700 as part of this work.

Closing due to consensus that some resource providers will not be idempotent, like networking and keyvault.


Can not abandon failed channel open

Background
An lnd node was getting an incoming channel open from a peer that unfortunately set the fee way too low - eventually it fell out of the mempool and is now long gone.
The original input has since been spent elksewhere, so there is no way this tx ever somehow miraculously comes back.
The nodes insiting on trying to negotiate the channel, though. As there is nothing on chain, there is nothing to close.
Even abanding the channel doesn't work:
$ lncli abandonchannel --funding_txid $TXID
[lncli] rpc error: code = Unknown desc = unable to find closed channel summary

We do want to establish this channel, which currently we are prohibited from as clightning does not allow redundant channels.
The opening party (running clightning) is cooperative and will cooperate if anything needs to be done on  their end, but in either case it seems like this scenario should have a path to reolution even from the perspective of a single node?

Just to clarify, are you sure you're not missing the --output_index flag? Is the number after the colon in the channel_point notation really 0?

Yeah, there's only one output (otherwise we would have bumped the fee when it was still in the mempool)

What state is the channel in? Could you provide more details from the listchannels/pendingchannels output?

The initiating clightning shows it as `CHANNELD_AWAITING_LOCKIN.
listchannels doesn't include it.
$ lncli pendingchannels
        {
            "channel": {
                "remote_node_pub": "...",
                "channel_point": "...:0",
                "capacity": "...",
                "local_balance": "0",
                "remote_balance": "...",
                "local_chan_reserve_sat": "...",
                "remote_chan_reserve_sat": "...",
                "initiator": "INITIATOR_REMOTE",
                "commitment_type": "STATIC_REMOTE_KEY"
            },
            "confirmation_height": 0,
            "commit_fee": "...",
            "commit_weight": "...",
            "fee_per_kw": "..."
        }

I don't see how your error above is possible if the channel shows up under pending_open_channels. Was the funding txid reversed by any chance?

I don't see how your error above is possible if the channel shows up under pending_open_channels. Was the funding txid reversed by any chance?

As I mentioned in the issue, the tx has low enough fee that it won't be confirmed, and there is no change output to do a CPFP or RBF. It won't ever go though.

Yes, but what @wpaulino is saying is: if you see the channel under pending_open_channels, the unable to find closed channel summary error shouldn't be possible. Are you 100% sure the --funding_txid flag is correct, without a colon or anything but otherwise looks the same as in channel_point of pending_open_channels?

So, it did work when I tried again - don't save my shell history so can't tell if I misissued it after all - I tried about a dozen separate times but was also pretty tired so it's not impossible I messed it up... But I have also issued a couple of rebuilds and restrats since.
What can I say, it's working now. Sorry for the confusion!


feat: link learn more block to search page.

Changes the following:

Links in the "Learn More" block will now go to the search page, with the clicked keyword as the pre-loaded search string.
Removed the tags template.
Change the search page so that it will match articles that contain any of the words in the search term.

@kunwarsaluja I wanted to give a quick overview of how search works.
Generally, it only searches based on an article's title or description (since that's all we have available in the index). Matching is not case sensitive. Before this change, searching for "computer accessories" would only match articles whose title or description contained the full phrase "computer accessories". After this change, it will match articles whose title or description contains either of the words "computer" or "accessories".
I saw in the video that someone tried surrounding the search with quotes. This is currently not supported, and would require some fairly complex parsing to support. We could certainly add it, but it's not there today.
Fix #136
Test URLs:

Before: https://main--channelco-crn-com--hlxsites.hlx.page/news/managed-services/painful-decision-slalom-consulting-layoffs-hit-900-workers
After: https://issue-136--channelco-crn-com--hlxsites.hlx.page/news/managed-services/painful-decision-slalom-consulting-layoffs-hit-900-workers

@mulderjoe See comment above from @mfrisbey on how the search works today. It's searching for title and description captured in the metadata of the article and other tiny things.
Let know if all looks good form search and the Learn-more block now being redirected to search page
See working on the page : https://issue-136--channelco-crn-com--hlxsites.hlx.page/news/managed-services/painful-decision-slalom-consulting-layoffs-hit-900-workers


4.7 relnotes

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/TOC.md
:white_check_mark:Succeeded
View

docs/release-notes/Index.md
:white_check_mark:Succeeded
View

docs/release-notes/NuGet-4.7-RTM.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Datepicker overflow, possible to set right: 0 on last-child?

Currently, the datepicker overflows offscreen.

I'd like to set right: 0; on the second datepicker box, but can't find a solution.

I would love this resolved. It's the only thing letting down a 10/10 library.


Add custom domain for BenSwap (SmartBCH)

Hi! What is your PR about?

[ ] Add or edit a skin
[x ] Add a custom domain for your space
[ ] Add an alias to migrate your space

yes.. could you check again? thanks

Hi, I checked https://vote.benswap.cash/#/  it goes to the snapshot homepage.
I already did https://ina9pk8175.execute-api.us-west-2.amazonaws.com/dev?domain=vote.benswap.cash
What have I missed? Thanks!


Crash related to non-NSObject class

App crashes if my view controller has a property that isn't NSObject.
The crash happens in ivar_getTypeEncodingSwift3 method.
Minimal code:
import UIKit

class Foo {
}

class ViewController: UIViewController {
    let foo = Foo()

    func injected() {
    }
}

Thanks for the report. I’m afraid this seems to be the case if members do not inherit from NSObject and I’m not in a position to fix this at the moment. Please see https://github.com/johnno1962/injectionforxcode/issues/177

Wait, so injected method is the problem? If that's the case, then maybe it's support should be removed? It's easy enough to register for notification.

Objective-C users find it useful so I’ll leave it in for now. I’ll try to work out what the problem is with Pure Swift instances if I get the time.


Serial dissappears after attempting to remove access code on Arch Linux

After receiving some keys with data already on them I tried to remove them by revoking the access key and writing over them (I didn't know about tools > delete at the time). Which resulted in the serial number information dissappearing until I delete the config via tools > delete at which point it returned.
This was pretty confusing and irritating as I thought I had borked my key on the first day.
Firmware: 3.4.6
Distro: Arch
Kernel: 4.4.1-2
Yubi GUI: 3.1.24

This sounds like the option "API call" under "Serial # Visibility Settings" in the Settings tab is unchecked. If it is the tool can not read the serial number from the key.

Hi Klali,
I was pointed this way by the support team and it was not turned off (I had previously made sure all the boxes were ticked). Also I could see the serial numbers then when I went to protected keep protected it removed the serial number information and wouldn't appear after replug/reboot or using in a vm.

Ok.
Exactly which view are we talking about here?
Can you reproduce this again? Can you lay that out in steps?

I'm struggling to reproduce it after the tools > delete option as I can't apply access codes on Arch, which is another issue I was going to raise after I had delved into the dependencies for the Ubuntu install and see if there is a difference.
No offence meant but linux support seems to mean works on Ubuntu.


:fireworks: Updates maintenance/license year to 2023

Proposed Changes
Happy New Year 🥂

It is not necessary to update or even specify the year in a copyright statement.
The Linux Foundation lawyers agree: https://www.linuxfoundation.org/blog/blog/copyright-notices-in-open-source-software-projects
The Amazon legal department agree: https://github.com/opendistro-for-elasticsearch/anomaly-detection-kibana-plugin/issues/365
The NTPsec project, after consultation with open source lawyers, agree: https://blog.ntpsec.org/2020/02/15/copyright-year.html
The following statement is sufficent to assert copyright to enable the license terms:  "Copyright Contributors to the XYZ project"


2021 Home DeHumidifier choice

Related https://github.com/130s/hut_10sqft/issues/556

Investigation on hold due to observation https://github.com/130s/30y-130s_life/issues/1228#issuecomment-1146510243.